Копировать значения карты в вектор в STL

86

В данный момент прорабатываю эффективный STL. Правило 5 предполагает, что обычно предпочтительнее использовать функции-члены диапазона вместо их одноэлементных аналогов. В настоящее время я хочу скопировать все значения на карте (т.е. мне не нужны ключи) в вектор.

Как лучше всего это сделать?

Гилад Наор
источник
Если ключи не нужны, то и вся карта может не понадобиться. В таком случае рассмотрите возможность перемещения значений с карты в вектор, как описано в этом вопросе .
Nykodym

Ответы:

61

Вы не можете легко использовать здесь диапазон, потому что итератор, который вы получаете от карты, ссылается на std :: pair, где итераторы, которые вы использовали бы для вставки в вектор, ссылаются на объект типа, хранящегося в векторе, который является (если вы сбрасываете ключ) не пара.

Я действительно не думаю, что это становится намного чище, чем очевидное:

#include <map>
#include <vector>
#include <string>
using namespace std;

int main() {
    typedef map <string, int> MapType;
    MapType m;  
    vector <int> v;

    // populate map somehow

    for( MapType::iterator it = m.begin(); it != m.end(); ++it ) {
        v.push_back( it->second );
    }
}

которую я, вероятно, переписал бы как функцию шаблона, если бы собирался использовать ее более одного раза. Что-то типа:

template <typename M, typename V> 
void MapToVec( const  M & m, V & v ) {
    for( typename M::const_iterator it = m.begin(); it != m.end(); ++it ) {
        v.push_back( it->second );
    }
}

источник
79
Python действительно избаловал меня :-(
Гилад Наор
1
Хороший шаблон. Может, дать ему итератор вывода вместо контейнера!
xtofl
Решение Скурмеделя еще лучше: используйте функцию 'transform' с функтором ap -> p.second.
xtofl
2
Я твердо верю в бритву Оккама - не вводите сущности без надобности. В случае решения преобразования нам нужна вспомогательная функция, которая не нужна для явного решения цикла. Итак, пока мы не получим безымянные функции, я буду придерживаться своего решения.
3
Остерегайтесь интерпретации бритвы Оккама. В конце концов, введение новой неконстантной переменной «it» может оказаться не самым безопасным решением. Алгоритмы STL уже давно зарекомендовали себя как быстрые и надежные.
Винсент Роберт
62

Вероятно, вы могли бы использовать std::transformдля этой цели. Я, возможно, предпочел бы версию Нилса, в зависимости от того, что более читабельно.


Пример от xtofl (см. Комментарии):

#include <map>
#include <vector>
#include <algorithm>
#include <iostream>

template< typename tPair >
struct second_t {
    typename tPair::second_type operator()( const tPair& p ) const { return p.second; }
};

template< typename tMap > 
second_t< typename tMap::value_type > second( const tMap& m ) { return second_t< typename tMap::value_type >(); }


int main() {
    std::map<int,bool> m;
    m[0]=true;
    m[1]=false;
    //...
    std::vector<bool> v;
    std::transform( m.begin(), m.end(), std::back_inserter( v ), second(m) );
    std::transform( m.begin(), m.end(), std::ostream_iterator<bool>( std::cout, ";" ), second(m) );
}

Очень общий, не забудьте отдать должное, если сочтете это полезным.

Скурмедель
источник
что мне нравится даже больше, чем у Нила. Тренировка, тренировка!
xtofl
Я бы посоветовал использовать лямбда в качестве последнего параметра.
varepsilon
@varepsilon: Вероятно, хорошая идея (если она используется на современном компиляторе C ++), но я больше не уверен в C ++, я вроде как чувак на C. Если кто-то хочет его улучшить и думает, что может это сделать, пожалуйста, продолжайте :)
Skurmedel 01
29

Старый вопрос, новый ответ. В C ++ 11 появился новый модный цикл for:

for (const auto &s : schemas)
   names.push_back(s.first);

где схемы - это, std::mapа имена - это std::vector.

Это заполняет массив (имена) ключами из карты (схем); измените s.firstна, s.secondчтобы получить массив значений.

Сет
источник
3
Должно бытьconst auto &s
Слава
1
@Slava, чтобы прояснить для любого нового в диапазоне на основе: способ, которым я написал, это работает, однако версия, предложенная Славой, быстрее и безопаснее, поскольку позволяет избежать копирования объекта итератора с помощью ссылки и указывает константу, поскольку это будет опасно изменять итератор. Спасибо.
Сет,
4
Кратчайшее и чистое решение. И, вероятно, самый быстрый (проверено, что он быстрее, чем принятое решение, а также быстрее, чем решение @Aragornx). Добавьте, reserve()и вы получите еще один прирост производительности. С появлением C ++ 11 это теперь должно быть приемлемым решением!
Adrian W
3
Разве это не должно быть names.push_back (s.second); поскольку вопрос требует значений, а не ключей в векторе?
Дэвид
24

Если вы используете библиотеки boost , вы можете использовать boost :: bind для доступа ко второму значению пары следующим образом:

#include <string>
#include <map>
#include <vector>
#include <algorithm>
#include <boost/bind.hpp>

int main()
{
   typedef std::map<std::string, int> MapT;
   typedef std::vector<int> VecT;
   MapT map;
   VecT vec;

   map["one"] = 1;
   map["two"] = 2;
   map["three"] = 3;
   map["four"] = 4;
   map["five"] = 5;

   std::transform( map.begin(), map.end(),
                   std::back_inserter(vec),
                   boost::bind(&MapT::value_type::second,_1) );
}

Это решение основано на сообщении Майкла Гольдштейна в списке рассылки boost .

ОК.
источник
24
#include <algorithm> // std::transform
#include <iterator>  // std::back_inserter
std::transform( 
    your_map.begin(), 
    your_map.end(),
    std::back_inserter(your_values_vector),
    [](auto &kv){ return kv.second;} 
);

Извините, что я не добавил никаких объяснений - я думал, что код настолько прост, что не требует никаких объяснений. Так:

transform( beginInputRange, endInputRange, outputIterator, unaryOperation)

эта функция вызывает unaryOperationкаждый элемент из inputIteratorдиапазона ( beginInputRange- endInputRange). Значение операции сохраняется в outputIterator.

Если мы хотим работать со всей картой - мы используем map.begin () и map.end () в качестве диапазона ввода. Мы хотим сохранить наши ценности карты в векторе - поэтому мы должны использовать back_inserter на нашем векторе: back_inserter(your_values_vector). Back_inserter - это специальный outputIterator, который подталкивает новые элементы в конец данной (как параметр) коллекции. Последний параметр - unaryOperation - принимает только один параметр - значение inputIterator. Таким образом, мы можем использовать лямбда : [](auto &kv) { [...] }, где & kv - это просто ссылка на пару элементов карты. Поэтому, если мы хотим вернуть только значения элементов карты, мы можем просто вернуть kv.second:

[](auto &kv) { return kv.second; }

Думаю, это объясняет любые сомнения.

Арагорнкс
источник
3
Привет, добавьте немного пояснений вместе с кодом, так как это помогает понять ваш код. Ответы только на код не одобряются.
Бхаргав Рао
1
Да! этот фрагмент кода может решить вопрос, включая объяснение, которое действительно помогает улучшить качество вашего сообщения. Помните, что вы отвечаете на вопрос читателей в будущем, и эти люди могут не знать причины вашего предложения кода.
J. Chomel
Я думаю, что это работает только начиная с C ++ 14, поскольку до этого auto не поддерживался в лямбдах. Явная подпись функции все равно будет работать.
turoni
19

Используя лямбды, можно сделать следующее:

{
   std::map<std::string,int> m;
   std::vector<int> v;
   v.reserve(m.size());
   std::for_each(m.begin(),m.end(),
                 [&v](const std::map<std::string,int>::value_type& p) 
                 { v.push_back(p.second); });
}

источник
1
Я не думаю, что вам нужно v.reserve (m.size ()), потому что v будет расти по мере того, как вы нажимаете новые элементы.
Dragan Ostojić
11
@ DraganOstojić .reserve () вызывает только одно перераспределение. В зависимости от количества элементов .push_back () может выполнять несколько распределений, чтобы получить один и тот же размер.
mskfisher
8

Вот что бы я сделал.
Также я бы использовал функцию шаблона, чтобы упростить построение select2nd.

#include <map>
#include <vector>
#include <algorithm>
#include <memory>
#include <string>

/*
 * A class to extract the second part of a pair
 */   
template<typename T>
struct select2nd
{
    typename T::second_type operator()(T const& value) const
    {return value.second;}
};

/*
 * A utility template function to make the use of select2nd easy.
 * Pass a map and it automatically creates a select2nd that utilizes the
 * value type. This works nicely as the template functions can deduce the
 * template parameters based on the function parameters. 
 */
template<typename T>
select2nd<typename T::value_type> make_select2nd(T const& m)
{
    return select2nd<typename T::value_type>();
}

int main()
{
    std::map<int,std::string>   m;
    std::vector<std::string>    v;

    /*
     * Please note: You must use std::back_inserter()
     *              As transform assumes the second range is as large as the first.
     *              Alternatively you could pre-populate the vector.
     *
     * Use make_select2nd() to make the function look nice.
     * Alternatively you could use:
     *    select2nd<std::map<int,std::string>::value_type>()
     */   
    std::transform(m.begin(),m.end(),
                   std::back_inserter(v),
                   make_select2nd(m)
                  );
}
Мартин Йорк
источник
1
Хороший. А почему make_select2nd нет в stl?
Николай Голубев
select2nd - это расширение STL в версии SGI (так неофициально). Добавление шаблонов функций в качестве утилит теперь просто вторая натура (см. Make_pair <> () для вдохновения).
Мартин Йорк
2

Один из способов - использовать функтор:

 template <class T1, class T2>
    class CopyMapToVec
    {
    public: 
        CopyMapToVec(std::vector<T2>& aVec): mVec(aVec){}

        bool operator () (const std::pair<T1,T2>& mapVal) const
        {
            mVec.push_back(mapVal.second);
            return true;
        }
    private:
        std::vector<T2>& mVec;
    };


int main()
{
    std::map<std::string, int> myMap;
    myMap["test1"] = 1;
    myMap["test2"] = 2;

    std::vector<int>  myVector;

    //reserve the memory for vector
    myVector.reserve(myMap.size());
    //create the functor
    CopyMapToVec<std::string, int> aConverter(myVector);

    //call the functor
    std::for_each(myMap.begin(), myMap.end(), aConverter);
}
aJ.
источник
Я бы не стал заморачиваться с переменной aConverter. просто создайте временный файл в for_each. std :: for_each (myMap.begin (), myMap.end (), CopyMapToVec <std :: string, int> (myVector));
Мартин Йорк
предпочитаю «преобразовать», поскольку именно это вы и делаете: преобразование карты в вектор с помощью довольно простого функтора.
xtofl
2

Почему нет:

template<typename K, typename V>
std::vector<V> MapValuesAsVector(const std::map<K, V>& map)
{
   std::vector<V> vec;
   vec.reserve(map.size());
   std::for_each(std::begin(map), std::end(map),
        [&vec] (const std::map<K, V>::value_type& entry) 
        {
            vec.push_back(entry.second);
        });
    return vec;
}

Применение:

auto vec = MapValuesAsVector (anymap);

Ян Вильманс
источник
Я думаю, ваш vec будет в два раза больше карты
dyomas
спасибо dyomas, я обновил функцию, чтобы сделать резервную копию вместо изменения размера, и теперь она работает правильно
Ян
2

Я думал так должно быть

std::transform( map.begin(), map.end(), 
                   std::back_inserter(vec), 
                   boost::bind(&MapT::value_type::first,_1) ); 
ОДЖМАН
источник
2

Мы должны использовать функцию преобразования из алгоритма STL, последним параметром функции преобразования может быть объект функции, указатель функции или лямбда-функция, которая преобразует элемент карты в элемент вектора. Эта карта случаев имеет элементы с парой типов, которые необходимо преобразовать в элемент с типом int для вектора. Вот мое решение, в котором я использую лямбда-функцию:

#include <algorithm> // for std::transform
#include <iterator>  // for back_inserted

// Map of pair <int, string> need to convert to vector of string
std::map<int, std::string> mapExp = { {1, "first"}, {2, "second"}, {3, "third"}, {4,"fourth"} };

// vector of string to store the value type of map
std::vector<std::string> vValue;

// Convert function
std::transform(mapExp.begin(), mapExp.end(), std::back_inserter(vValue),
       [](const std::pair<int, string> &mapItem)
       {
         return mapItem.second;
       });
Loc Tran
источник
-3

Удивлен, что никто не упомянул наиболее очевидное решение - использовать конструктор std :: vector.

template<typename K, typename V>
std::vector<std::pair<K,V>> mapToVector(const std::unordered_map<K,V> &map)
{
    return std::vector<std::pair<K,V>>(map.begin(), map.end());
}
Franjesus
источник
4
Это потому, что ваше решение не соответствует вопросу. Вектор должен состоять только из значений.
ypnos 05