std :: mapからすべてのキー(または値)を取得してベクターに入れる方法は?


246

これは私が出てくる可能な方法の1つです。

struct RetrieveKey
{
    template <typename T>
    typename T::first_type operator()(T keyValuePair) const
    {
        return keyValuePair.first;
    }
};

map<int, int> m;
vector<int> keys;

// Retrieve all keys
transform(m.begin(), m.end(), back_inserter(keys), RetrieveKey());

// Dump all keys
copy(keys.begin(), keys.end(), ostream_iterator<int>(cout, "\n"));

もちろん、別のファンクターRetrieveValuesを定義して、マップからすべての値を取得することもできます。

これを簡単に達成する他の方法はありますか?(なぜstd :: mapにそうするためのメンバー関数が含まれていないのか、いつも疑問に思っています。)


10
あなたのソリューションは最高です...
リネロ2013

4
これを追加すると思うのは、これだけですkeys.reserve(m.size());
Galik 2018

回答:


176

ソリューションは機能するはずですが、プログラマーのスキルレベルによっては読みにくい場合があります。さらに、機能を呼び出しサイトから移動します。これにより、メンテナンスが少し難しくなります。

あなたの目標がキーをベクターに入れることなのか、それともcoutに出力することなのかわからないので、両方を行います。あなたはこのようなものを試すかもしれません:

map<int, int> m;
vector<int> v;
for(map<int,int>::iterator it = m.begin(); it != m.end(); ++it) {
  v.push_back(it->first);
  cout << it->first << "\n";
}

または、Boostを使用している場合はさらに簡単です。

map<int,int> m;
pair<int,int> me; // what a map<int, int> is made of
vector<int> v;
BOOST_FOREACH(me, m) {
  v.push_back(me.first);
  cout << me.first << "\n";
}

個人的には、入力が少なく、何をしているのかが非常に明確なので、BOOST_FOREACHバージョンが好きです。


1
私のグーグル検索の後で私がここに戻ってしまう数字に行きなさい。あなたの答えが私が好む答えです :)
mpen '14

4
@ジェレ-実際に働いたことがありBOOST_FOREACHますか?ここで提案するコードは完全に間違っています
Manuel

2
@Jamie-これは別の方法ですが、ブーストのドキュメントでは、型にコンマが含まれている場合、BOOST_FOREACHの前に変数とその型を指定していることが示されています。彼らはそれをtypedefingすることも示します。だから、私は混乱しています、私のコードの何が問題になっていますか?
Jere.Jones 2010

17
好奇心が強い、サイズ変更の割り当てを防ぐためにベクターのサイズを事前に設定することは理にかなっていますか?
アラン(

2
v.reserve(m.size())転送中にベクターのサイズを変更しないようにすることを忘れないでください。
ブライアンホワイト

157
//c++0x too
std::map<int,int> mapints;
std::vector<int> vints;
vints.reserve(mapints.size());
for(auto const& imap: mapints)
    vints.push_back(imap.first);

4
いいね。忘れてくださいit = ...begin(); it != ...end。素敵なもちろん()メソッドのキーを備え、そのベクトルを返すのstd ::マップだろう...
masterxilo

2
@BenHymers:この答えは C ++ 11がC ++になったanswered Mar 13 '12 at 22:33数か月後のに出されたようです。
セバスチャンマッハ

37
@BenHymersしかし、それは今質問を読んでいる誰にとっても役に立ちます。それはSOがすべてについてである-質問者を助けるだけでなく、他のすべての人です。
Luchian Grigore

9
for(auto&imap)は、コピー操作がないため、より正確です。
HelloWorld、2015年

2
@StudentT、いっそのこと、for(auto const & imap : mapints)
cp.engr 2015年

61

この目的のためにブースト範囲アダプターがあります:

vector<int> keys;
// Retrieve all keys
boost::copy(m | boost::adaptors::map_keys, std::back_inserter(keys));

値を抽出するための同様のmap_values範囲アダプターがあります。


1
残念ながら、boost::adaptorsBoost 1.43までは利用できないようです。Debianの(スクイーズ)の現在の安定リリースは唯一の申し出は1.42ブースト
ミカエル・ルBaillif

2
それは残念だ。Boost 1.42は、Squeezeの2.5年以上前の2010年2月にリリースされました。
Alastair 2012年

この時点で、Squeeze UpdatesまたはバックポートリポジトリがBoost 1.44を提供しているのではないでしょうか?
Luis Machuca 2013年

それはどのブーストヘッダーが定義されていますか?
James Wierzba

1
リンクされたdocoを参照してください。定義されているのはboost/range/adaptor/map.hpp
Alastair、

46

C ++ 0xは、さらに優れたソリューションを提供してくれました。

std::vector<int> keys;

std::transform(
    m_Inputs.begin(),
    m_Inputs.end(),
    std::back_inserter(keys),
    [](const std::map<int,int>::value_type &pair){return pair.first;});

22
私の見解では、それについて優れたものは何もありません。std :: vector <int>キー; keys.reserve(m_Inputs.size()); for(auto keyValue:m_Inputs){keys.push_back(keyValue.first); }不可解な変換よりもはるかに優れています。パフォーマンスの面でも。これは良いです。
ジャガンナート

5
同等のパフォーマンスが必要な場合は、ここでもキーのサイズを予約できます。forループを避けたい場合は、変換を使用してください。
DanDan

4
追加したいだけ-[](const auto&ペア)を使用できます
ivan.ukr

@ ivan.ukrどのコンパイラを使用していますか?この構文はここでは使用できません
含める

4
@ ivan.ukrラムダの自動パラメーターはc ++ 14です
roalz '13年

16

C ++ 11を使用した@DanDanの答えは次のとおりです。

using namespace std;
vector<int> keys;

transform(begin(map_in), end(map_in), back_inserter(keys), 
            [](decltype(map_in)::value_type const& pair) {
    return pair.first;
}); 

およびC ++ 14を(@ ivan.ukrで述べたように)使用して、我々は交換することができdecltype(map_in)::value_typeauto


5
keys.reserve(map_in.size());効率を上げるために追加できます。
Galik

変換メソッドには、実際にはforループよりも多くのコードが必要です。
user1633272 2018年

タイプの後ろにconstを置くことができます!私はほとんどそれを忘れています。


10

あなたの解決策は素晴らしいですが、それを行うためにイテレータを使うことができます:

std::map<int, int> m;
m.insert(std::pair<int, int>(3, 4));
m.insert(std::pair<int, int>(5, 6));
for(std::map<int, int>::const_iterator it = m.begin(); it != m.end(); it++)
{
    int key = it->first;
    int value = it->second;
    //Do something
}

10

@ rusty-parksソリューションに基づくが、c ++ 17の場合:

std :: map <int、int> items;
std :: vector <int> itemKeys;

for(const auto&[key、ignoreed]:items)
{
    itemKeys.push_back(key);
}

std::ignoreこのように構造化バインディングで使用することはできないと思います。コンパイルエラーが発生します。通常の変数を使用するだけで十分です。たとえばignored、それは単に使用されません。
jb

1
@jbありがとう。実際、std::ignorestd::tie構造的バインディングでの使用を意図していますが、構造的バインディングでの使用は意図していません。コードを更新しました。
マディヤール

9

上記のBOOST_FOREACHは見栄えがよく、きれいだと思いますが、BOOSTを使用する別のオプションもあります。

#include <boost/lambda/lambda.hpp>
#include <boost/lambda/bind.hpp>

std::map<int, int> m;
std::vector<int> keys;

using namespace boost::lambda;

transform(      m.begin(), 
                m.end(), 
                back_inserter(keys), 
                bind( &std::map<int,int>::value_type::first, _1 ) 
          );

copy( keys.begin(), keys.end(), std::ostream_iterator<int>(std::cout, "\n") );

個人的には、このアプローチはこの場合のBOOST_FOREACHアプローチほどクリーンではないと思いますが、boost :: lambdaは他のケースでは本当にクリーンである可能性があります。



7

c ++ 11のビットを取る:

std::map<uint32_t, uint32_t> items;
std::vector<uint32_t> itemKeys;
for (auto & kvp : items)
{
    itemKeys.emplace_back(kvp.first);
    std::cout << kvp.first << std::endl;
}


5

これは、C ++ 11マジックを使用した、std :: map、std :: unordered_mapの両方で機能する素晴らしい関数テンプレートです。

template<template <typename...> class MAP, class KEY, class VALUE>
std::vector<KEY>
keys(const MAP<KEY, VALUE>& map)
{
    std::vector<KEY> result;
    result.reserve(map.size());
    for(const auto& it : map){
        result.emplace_back(it.first);
    }
    return result;
}

ここで確認してください:http : //ideone.com/lYBzpL


4

最適な非sgi、非ブーストSTLソリューションは、map :: iteratorを次のように拡張することです。

template<class map_type>
class key_iterator : public map_type::iterator
{
public:
    typedef typename map_type::iterator map_iterator;
    typedef typename map_iterator::value_type::first_type key_type;

    key_iterator(const map_iterator& other) : map_type::iterator(other) {} ;

    key_type& operator *()
    {
        return map_type::iterator::operator*().first;
    }
};

// helpers to create iterators easier:
template<class map_type>
key_iterator<map_type> key_begin(map_type& m)
{
    return key_iterator<map_type>(m.begin());
}
template<class map_type>
key_iterator<map_type> key_end(map_type& m)
{
    return key_iterator<map_type>(m.end());
}

そしてそれらを次のように使用します:

        map<string,int> test;
        test["one"] = 1;
        test["two"] = 2;

        vector<string> keys;

//      // method one
//      key_iterator<map<string,int> > kb(test.begin());
//      key_iterator<map<string,int> > ke(test.end());
//      keys.insert(keys.begin(), kb, ke);

//      // method two
//      keys.insert(keys.begin(),
//           key_iterator<map<string,int> >(test.begin()),
//           key_iterator<map<string,int> >(test.end()));

        // method three (with helpers)
        keys.insert(keys.begin(), key_begin(test), key_end(test));

        string one = keys[0];

1
必要に応じてconst_iteratorとリバースイテレータの作成もリーダーに任せます。
Marius

-1

原子マップの例

#include <iostream>
#include <map>
#include <vector> 
#include <atomic>

using namespace std;

typedef std::atomic<std::uint32_t> atomic_uint32_t;
typedef std::map<int, atomic_uint32_t> atomic_map_t;

int main()
{
    atomic_map_t m;

    m[4] = 456;
    m[2] = 45678;

    vector<int> v;
    for(map<int,atomic_uint32_t>::iterator it = m.begin(); it != m.end(); ++it) {
      v.push_back(it->second);
      cout << it->first << " "<<it->second<<"\n";
    }

    return 0;
}

-2

ここでの例の1つに少し似ており、std::map使用の観点から単純化されています。

template<class KEY, class VALUE>
std::vector<KEY> getKeys(const std::map<KEY, VALUE>& map)
{
    std::vector<KEY> keys(map.size());
    for (const auto& it : map)
        keys.push_back(it.first);
    return keys;
}

次のように使用します。

auto keys = getKeys(yourMap);

2
ねえ、私はこの答えが古いことを知っていますが、それも間違っています。サイズで初期化するとmap.size()、ベクターサイズが2倍に戻ります。誰か他の人の頭痛を救うために修正してください:(
thc

-3

(なぜstd :: mapにそうするためのメンバー関数が含まれていないのか、いつも疑問に思っています。)

それはあなたがそれをすることができるよりもそれをすることができないので。メソッドの実装がフリー関数の実装より優れていない場合は、一般にメソッドを記述しないでください。あなたは無料の関数を書くべきです。

それがとにかく有用である理由もすぐには明らかではありません。


8
ライブラリが「バッテリーを含む」機能や一貫性のあるカプセル化されたAPIなどのメソッドを提供する効率以外にも理由があります。確かに、これらの用語のどちらもSTLを特によく説明していませんが:) Re。なぜそれが有用なのかはっきりしない-本当に?利用可能なキーをリストすることがmap / dictで実行できる便利な理由であるのは明らかだと思います。それは、それを何に使用しているかに依存します。
andybuckley

4
この推論では、empty()として実装できるため、必要ありませんsize() == 0
gd1

1
@ gd1が言ったこと。クラスに多くの機能的な冗長性があるべきではありませんが、絶対にゼロを主張することは良い考えではありません。
einpoklum

1
古いバージョンのC ++では、empty()とsize()が合理的に異なるパフォーマンスを保証できるコンテナーがあり、仕様はこれを可能にするほど十分に緩いと思います(特に、一定時間のsplice()を提供するリンクリスト) 。そのため、それらを分離することは理にかなっています。しかし、この矛盾はもはや許されないと思います。
DrPizza

同意する。C ++はstd::map<T,U>ペアのコンテナーとして扱います。Pythonでは、a dictは反復されるとそのキーのようにd.items()動作しますが、C ++の動作を取得すると言うことができます。Pythonもを提供しますd.values()std::map<T,U>確かに提供することが可能keys()values()持つオブジェクトを返すメソッドbegin()end()キーと値を超える反復子を提供します。
ベン
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.