c++:mapに格納された要素を全て表示したい

Question

c++のmapに関する質問です。

#include<iostream>
#include<map>

using namespace std;
int main(void){
    map<string,int>m;
    int n;
    string str;
    cin>>n;
    for(int i=0;i<n;i++){
      cin>>str>>m[str];
    }
    for(auto const&answer:m){
      cout<<answer.first<<" "<<answer.second<<endl;
    }
    
        
    return 0;
}

文字列とそれに対応する数字を入力し、それをmapで格納し最後にそれぞれの第一要素と第二要素を全て表示するプログラムを作るために上記のようなコードを書いたのですが、上手く全表示することが出来ません。
自分なりに調べてはみたのですが改善策が分からずじまいになってしまったため、どなたか分かる方に改善策を教えていただきたいです。

Answer 1

#include <iostream>
#include <map>
#include <string>

int main(void){
    using namespace std;

    map<string,int>m;
    int n;
    cin>>n;
    for(int i=0;i<n;i++){
      string key;
      int val;
      cin >> key >> val;
      m[key] = val;
    }
    for ( auto const& answer : m ) {
      cout << answer.first << " " << answer.second << endl;
    }

    return 0;
}

[追記]
cin >> str >> m[str] だと cin >> str によって str が書き換わる前に
m[str] が評価されてると推察されます。その検証:

#include <iostream>
#include <string>

std::string key;
int val;

int& foo(const char* str) {
  key = str; // 引き渡された文字列をコピーののち
  return val; // val の参照を返す
}

int main(void){
  using namespace std;
  char buffer[64] = "ABRACADABRA";
  cin >> buffer >> foo(buffer);
  cout << "key =[" << key << "]\n"; // foo() に引き渡された文字列
  cout << "val =[" << val << "]\n";
}

実行結果

abc 3       ← 入力
key =[ABRACADABRA]
val =[3]

たしかに。
※ Windows/Visual C++ 2019 での結果です。

Comments

[yusuke-fusegi]

全表示できました！ありがとうございました！

[episteme]

cin>>str>>m[str];
こいつを
cin >> str;
cin >> m[str];
とするんでもよかったかな...

[fana]

OK，それでは，私から問うておこう．
元のコードの書き方 (cin>>str>>m[str];）だと何故ダメなんでしょうか？

[yusuke-fusegi]

>>fana様
cin>>m[str]だとmapの第二要素をint型として格納できないからでしょうか？

[episteme]

いや、m[str] は int& を返すから cin >> m[str] に問題はないはず。

cin >> str >> m[str] だと cin >> str によって str が書き換わる
前に m[str] が評価されてると推察されます。

[fana]

> str が書き換わる前に m[str] が評価されてる

動作を見るとそんな感じですね．
(cin>>str)>>m[str];
とかしても変わらない模様．

[yohhoy]

C++17で仕様修正された、下記ページの内容が関係していますね。
https://cpprefjp.github.io/lang/cpp17/expression_evaluation_order.html
ある程度新しい GCC(g++) や Clang(Clang++) では、この修正が適用されるようです。

コンパイラの対応バージョンを気にするくらいなら、cin >> str; cin >> m[str]; と2文で書くのが無難ではあります。

[yusuke-fusegi]

バージョンによっては質問の際私が投稿したソースコードでも問題なく動かせるということでしょうか？
色々な方に教えていただいて、今後はmapの第一要素と第二要素を入力してからそれをmapとして定義するように書いた方が無難だと感じたのですが。

[yohhoy]

> バージョンによっては質問の際私が投稿したソースコードでも問題なく動かせるということでしょうか？
はい。新しいC++コンパイラ（正確にはC++17仕様に厳格に対応したコンパイラ）では、質問中のソースコードで期待通り動作します。

> 今後はmapの第一要素と第二要素を入力してからそれをmapとして定義するように書いた方が無難だと感じた
シンプルなコードを書いた方が無難というのは、同意見です。
　cin >> str;
　cin >> m[str];
とするか
　cin >> key >> val;
　m[key] = val;
とするかは好みの問題ですね。

[episteme]

[追記]した検証コード、Visual C++ 2019 で -std:c++17 オプション付けたら
期待(思惑?)通りの挙動になりました。

[yusuke-fusegi]

＞＞yohhoy様、episteme様
バージョンが変わっても動かせるようにコーディングする意識は今までなかったので、大変勉強になりました！お二方ともありがとうございます。