从函数返回时std :: pair的奇怪行为

时间:2016-06-29 11:39:26

标签: c++ stl unordered-map

#include<boost/unordered_map.hpp>
#include<string>
#include<iostream>
#include<boost/unordered_set.hpp>
using namespace std;

typedef boost::unordered_map<string, boost::unordered_map<string,     boost::unordered_set<string>>> nfa;
const boost::unordered_map<string, boost::unordered_set<string>>&    
get_second(const std::pair<string, 
           boost::unordered_map<string, boost::unordered_set<string>>>& p)
 {return p.second;}


int main()
{
   nfa a;
   a["A"]["0"] = {"B", "C"};
   a["A"]["1"] = {"B"};
   a["B"]["0"] = {"B"};
   a["B"]["1"] =  {"C"};
   cout << "Printing using direct reference" << endl;
   for (auto tr_table : a)
   {
     for (auto tr : tr_table.second)
      cout << tr_table.first << " " << tr.first << " " <<     tr.second.size() << endl;
  }
  cout << "Printing using function get_second" << endl;
  for (auto tr_table : a)
  {
    for (auto tr : get_second(tr_table))
      cout << tr_table.first << " " << tr.first << " " << tr.second.size() << endl;
  }
 return 0;
 }

对于相同的unordered_map,使用tr.second返回正确的行数,但使用get_second会返回一个没有元素的新map元素。 这种行为的原因是什么?

我在Ubuntu上使用g ++ 5.3.1。

PS:使用std :: unordered_map时行为相同。

3 个答案:

答案 0 :(得分:6)

get_second使用非const键的错误类型。

因此构造了转换后的临时值,并且您将返回对此临时值的引用。

之后所有的赌注都没有了。

答案 1 :(得分:3)

你的get_second方法参数在constness方面与循环迭代器不匹配...更新如下(注意对中的const字符串)并且它有效:

get_second( const std::pair<const string,
    unordered_map<string, unordered_set<string>>>& p )

答案 2 :(得分:2)

请注意,SWT.MENU_KEYBOARD&#39; std::unordered_mapvalue_type const键),因此您{{1} }&#39; s参数错误。

您只需更改为std::pair<const Key, T>即可获得正确的行为。