Есть ли способ легко обрабатывать функции, возвращающие std:: pairs?

В C++11 имеет функцию std::minmax_element который возвращает пару значений. Это, однако, довольно запутанно для обработки и чтения и создает дополнительную, позже бесполезную переменную, чтобы загрязнить область.

auto lhsMinmax = std::minmax_element(lhs.begin(), lhs.end());
int &lhsMin = *(lhsMinMax.first);
int &lhsMax = *(lhsMinmax.second);

есть ли лучший способ сделать это? Что-то вроде:

int lhsMin;
int lhsMax;
std::make_pair<int&, int&>(lhsMin, lhsMax).swap(
    std::minmax_element(lhs.begin(), lhs.end()));
33
c++ c++11 minmax std-pair tuples

6 ответов

это выглядит достаточно распространенным случаем, чтобы запросить вспомогательную функцию:

template <class T, std::size_t...Idx>
auto deref_impl(T &&tuple, std::index_sequence<Idx...>) {
    return std::tuple<decltype(*std::get<Idx>(std::forward<T>(tuple)))...>(*std::get<Idx>(std::forward<T>(tuple))...);
}

template <class T>
auto deref(T &&tuple)
    -> decltype(deref_impl(std::forward<T>(tuple), std::make_index_sequence<std::tuple_size<std::remove_reference_t<T>>::value>{})) {
    return deref_impl(std::forward<T>(tuple), std::make_index_sequence<std::tuple_size<std::remove_reference_t<T>>::value>{});
}

// ...

int lhsMin;
int lhsMax;
std::tie(lhsMin,lhsMax) = deref(std::minmax_element(lhs.begin(), lhs.end()));

index_sequence является C++14, но полная реализация можно сделать в C++11.

примечание: Я бы повторил decltype на derefвозвращает тип даже в C++14, так что SFINAE может применяться.

увидеть его жить на Coliru

13

С структурированные привязки из C++1z вы можете напрямую сделать

auto [lhsMinIt, lhsMaxIt] = std::minmax_element(lhs.begin(), lhs.end());
32

чтобы избежать загрязнения вашей области, вы можете заключить назначение в меньшую область:

int lhsMin, lhsMax;

{
    auto it = std::minmax_element(lhs.begin(), lhs.end());
    lhsMin = *it.first;
    lhsMax = *it.second;
}

кроме того, вы можете использовать лямбда

int lhsMin, lhsMax;

std::tie(lhsMin, lhsMax) = [&]{
    auto it = std::minmax_element(lhs.begin(), lhs.end());
    return std::make_tuple(*it.first, *it.second);
}();
23

Я бы просто был более прямым и написал свою собственную версию minmax_element:

template <class Iter, class R = typename iterator_traits<Iter>::reference>
std::pair<R,R> deref_minmax(Iter first, Iter last)
{
    auto iters = std::minmax_element(first, last);
    return std::pair<R,R>{*iters.first, *iters.second};
}

который тогда просто:

int lo, hi;
std::tie(lo, hi) = deref_minmax(lhs.begin(), lhs.end());

это ограничит вас только одной копией элементов (что не так уж важно с ints), также позволяют сохранить доступ к ссылкам в контейнер.

в C++17, для удовольствия, мы могли бы писать обобщенный dereferencer:

template <class Tuple>
auto deref(Tuple&& tup) {
    return std::apply([](auto... args) {
        return std::tuple <decltype(*args)...>(*args...);
    }, tup);
}

auto& [lo, hi] = deref(std::minmax_element(lhs.begin(), lhs.end()));

здесь lo и hi ссылки на сам контейнер.

4

нет никакого способа назначить два ссылки сразу в текущей редакции стандарта, если это то, что вы после. Обратите внимание, что ни один из других ответов не делает этого, кроме Барри, который требует C++17 и вспомогательный шаблон.

однако, если вы хотите получить доступ для чтения и записи к минимальным и максимальным элементам, почему бы просто не пойти с итераторами minmax_element предоставляет вам напрямую? Скорее всего, он будет генерировать идентичный машинный код, используя ссылки в любом случае, в если ваш lhs это ContiguousContainer но, возможно, и в других случаях.

вам нужно будет немного меньше полагаться на автоматический вычет типа, например,

decltype(lhs.begin()) lhsMinIt, lhsMaxIt;
std::tie(lhsMinIt, lhsMaxIt) = std::minmax_element(lhs.begin(), lhs.end());
/* now access your minimum and maximum as *lhsMinIt and *lhsMaxIt */

если вы знаете типа lhs будет одним из стандартных контейнеров, вы можете использовать немного чище обозначение типа decltype(lhs)::iterator.

2

В C++14 или больше

template<class=void, std::size_t...Is>
auto indexer( std::index_sequence<Is...> ) {
  return [](auto&&f){
    return f( std::integral_constant<std::size_t, Is>{}... );
  };
}
template<std::size_t N>
auto indexer() {
  return indexer( std::make_index_sequence<N>{} );
}
template<class F>
auto fmap_over_tuple( F&& f ) {
  return [f=std::forward<F>(f)](auto&& tuple) {
    using Tuple = decltype(tuple);
    using Tuple_d = std::decay_t<Tuple>;
    auto index = indexer< std::tuple_size< Tuple_d >::value >();
    return index(
      [&f, &tuple](auto&&...Is) {
        using std::get;
        return std::make_tuple(
          f( get<Is>( std::forward<Tuple>(tuple) ) )...
        );
      }
    );
  };
}

так fmap_over_tuple принимает объект функции. Он возвращает объект function, который при передаче кортежа продолжает вызывать объект function для каждого элемента кортежа и генерировать кортеж из него.

затем мы пишем кортеж разыменования:

auto dereference_tuple = fmap_over_tuple(
  [](auto&& e) { return *e; }
);

теперь в C++17 мы делаем:

auto[Min, Max] = dereference_tuple( std::minmax_element(lhs.begin(), lhs.end() );

и Боб твой дядя.

В C++11 просто сделайте то, что вы сделали. Чистый достаточно.

в C++14 живой пример.

2