2016-07-13 29 views
0

有人可以解释为什么每当我尝试增加对时什么都没有发生?我试着调试它,尽管它发生在递增行注意到。修改地图对的值

编辑:这里是全功能

void VoteCollector::resultsBasedOnAge(std::vector<Voter>& voters) 
{ 
    std::map<int,std::pair<int,int>> ageVoters; 
    std::map<int,std::pair<int,int>>::iterator hasAge = ageVoters.begin(); 


    for(unsigned i = 0; i < voters.size(); i++) 
    { 
     if(ageVoters.find(voters.at(i).getAge()) != ageVoters.end()) 
     { 
      if(voters.at(i).getVote() == "leave") 
      { 
       hasAge->second.first++; 
      } 
      else if(voters.at(i).getVote() == "stay") 
      { 
       hasAge->second.second++; 
      } 
      hasAge++; 
     } 
     else 
     { 
      if(voters.at(i).getVote() == "leave") 
      { 
       ageVoters.insert(std::make_pair(voters.at(i).getAge(),std::make_pair(1,0))); 
      } 
      else if(voters.at(i).getVote() == "stay") 
      { 
       ageVoters.insert(std::make_pair(voters.at(i).getAge(),std::make_pair(0,1))); 
      } 
      hasAge++; 
     } 
    } 

    for(std::map<int,std::pair<int,int>>::iterator it = ageVoters.begin(); it != ageVoters.end(); it++) 
    { 
     std::cout << it->first << " years -- " << it->second.first << " leave.\t" << it->second.second << " stay\n"; 
    } 
} 
+1

请编辑您的问题包含一个[MCVE] – NathanOliver

+0

既然你不显示什么'hasAge'是我怀疑任何人可以帮助很大。我猜'hasAge'实际上是你想要修改的副本。 – nwp

+0

@NathanOliver你有MCVE的自动标记,如果是的话我怎样才能导入它? – Slava

回答

1

从我所看到的,您的代码不工作,因为你的hasAge指向,我不知道在什么地方你可能不是故意的。你想分配给它std::map::find的结果。

假设你正在使用C++ 11的代码也可以简化为:

void VoteCollector::resultsBasedOnAge(const std::vector<Voter>& voters) 
{ 
    std::map<int, std::pair<int, int>> ageVoters; 

    for (const auto& v: voters) 
    { 
     int age = v.getAge(); 
     const auto& vote = v.getVote(); 

     auto it = ageVoters.find(age); 
     if (it != ageVoters.cend()) 
     { 
      if (vote == "leave") 
      { 
       ++it->second.first; 
      } 
      else if (vote == "stay") 
      { 
       ++it->second.second; 
      } 
     } 
     else 
     { 
      if (vote == "leave") 
      { 
       ageVoters.insert(std::make_pair(age, std::make_pair(1, 0))); 
      } 
      else if (vote == "stay") 
      { 
       ageVoters.insert(std::make_pair(age, std::make_pair(0, 1))); 
      } 
     } 
    } 

    for (const auto& v: voters) 
    { 
     std::cout << v.first << " years -- " 
        << v.second.first << " leave.\t" 
        << v.second.second << " stay\n"; 
    } 
} 
+0

是的,它修复了它。似乎咖啡不够:/非常感谢你! –

+0

你不需要检查元素是否已经存在,但你应该使用'std :: map :: operator []'来代替:'auto&pair = ageVoters [age]; if(vote ==“leave”)++ pair.first; ...' – Slava

+0

@Slava小心,它与'insert'具有完全不同的语义。当没有现有元素时使用括号运算符将首先默认构造一个元素,然后将操作数赋给该默认构造元素**。这几乎肯定会对用户定义的类型有一些性能影响。 –