c++ - 充满垃圾记录的 map

Question

我正在使用这样的 map :

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 );
}

...

添加到 map :

m_mUsers[ sd ] = pUser;

从 map 中删除:

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

当我运行它时，它大部分都按我预期的那样工作。但很少有垃圾记录留在 map 中，所以当我尝试遍历预期为空 map 的内容时，我遇到了垃圾记录，并在 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 用于 map 的 mapped_type。 如果你没有可用的 c++11，你可以尝试 technical report或 boost libraries , 其中包括智能指针。 如果你只是想有一个引用而不关心管理 std::map 中的 CUser， 你也可以这样做，这意味着你没有为 CUser 管理内存(仅在 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;