充满垃圾记录的地图

Question

我正在使用这样的地图：

map<int,CUser*> m_mUsers;

...

for ( i = m_mUsers.begin(); i != m_mUsers.end(); i++ )
{
    if( (*i).second->GetUserID() == pUser->GetUserID() )
        (*i).second->OnDeviceLogout( pUser );
}

...

添加到地图：

m_mUsers[ sd ] = pUser;

从地图中删除：

i = m_mUsers.find( sd );
m_mUsers.erase( i );

当我运行它时，大多数情况下它都按我的预期工作。但很少有垃圾记录留在地图中，所以当我尝试遍历预期为空的地图时，我遇到垃圾记录，并在 i->second->GetUserID() 上崩溃...什么我做错了吗？

Answer 1

您没有检查i != m_mUsers.end()。这是不正确的，除非 保证 sd 在您的情况下存在。

i = m_mUsers.find( sd );

//You should do the check before erasing
if (i != m_mUsers.end()) {
    m_mUsers.erase( i );
}

这应该使代码正常工作。顺便说一句，我建议你使用 std::shared_ptr 或 std::unique_ptr 用于地图的 mapped_type。 如果您没有可用的 c++11，可以尝试technical report 或boost libraries， 其中包括智能指针。 如果你只是想有一个参考而不关心在std::map 中管理CUsers， 您也可以这样做，这意味着您没有为CUsers 管理内存（仅在 c++11 中）：

//No room for the question: Who is managing CUser memory in this map?
std::map<int, std::reference_wrapper<CUser>> m_mUsers;

如果你不修改用户，你甚至可以这样做：

//Note I added const to CUser
std::map<int, std::reference_wrapper<CUser const>> m_mUsers;