映射/设置迭代器不兼容 - 检查对象是否在映射中

Question

我正在尝试搜索我的地图以查看 Person 接收的对象（消息）是否已包含在他们的地图中。

这是地图容器：

std::map<MessageContent, unsigned int>  messageList;

这是我对 find() 函数的调用：

if(listOfPersons.at(y).getMessageList().find(*tempMessage.getMessageContent()) 
                                    != listOfPersons.at(y).getMessageList().end())

这里是 getMessageContent()：

MessageContent* Message::getMessageContent() const
{
    return mc;
}

最后是 getMessageList() 函数

std::map<MessageContent, unsigned int> Persons::getMessageList() const
{
    return messageList;
}

也与手头的话题有关，我认为我使用 return messageList; 不正确，因为我认为这种类型的返回返回的是 messageList 的实际副本，因此最好使用引用类型。

非常感谢任何帮助。 谢谢你

Answer 1

当您return messageList 时，您确实做错了什么：您正在按照您所说的那样制作副本，这不仅是性能问题（可能），而且在您的情况下，它是一个严重的错误，正如您所说getMessageList() 两次并在其中一个上执行 find() 并在另一个上执行 end() 。这会调用未定义的行为，因为您正在比较来自两个不同容器的两个迭代器。按照你的建议让函数返回一个引用，这个问题就会得到解决。