修改地图对的值
有人可以解释为什么每当我尝试增加对时什么都没有发生?我试着调试它,尽管它发生在递增行注意到。修改地图对的值
编辑:这里是全功能
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";
}
}
从我所看到的,您的代码不工作,因为你的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";
}
}
是的,它修复了它。似乎咖啡不够:/非常感谢你! –
你不需要检查元素是否已经存在,但你应该使用'std :: map :: operator []'来代替:'auto&pair = ageVoters [age]; if(vote ==“leave”)++ pair.first; ...' – Slava
@Slava小心,它与'insert'具有完全不同的语义。当没有现有元素时使用括号运算符将首先默认构造一个元素,然后将操作数赋给该默认构造元素**。这几乎肯定会对用户定义的类型有一些性能影响。 –
请编辑您的问题包含一个[MCVE] – NathanOliver
既然你不显示什么'hasAge'是我怀疑任何人可以帮助很大。我猜'hasAge'实际上是你想要修改的副本。 – nwp
@NathanOliver你有MCVE的自动标记,如果是的话我怎样才能导入它? – Slava