为什么std string ==崩溃?

时间:2012-06-02 22:02:10

标签: c++

当我在调试模式下运行我的应用程序时,有时我会在此函数中遇到运行时错误:

void ChatListHandler::seatOccupancyChanged( const std::string& userName, bool occupied, bool isSelf, bool isSelfTable, int tableNo, int seatNo, int numPlayersAtTable )
{
    if(!isSelf && (isInGroup(userName,IN_GAME_GROUP) || isInGroup(userName,IN_LOBBY_GROUP)))
    {
        if(occupied)
        {
            movePlayer(userName,IN_GAME_GROUP);
        }
        else
        {
            movePlayer(userName,IN_LOBBY_GROUP);
        }
    }
}

bool ChatListHandler::isInGroup( const std::string& name, GroupTypeEnum group )
{
    for(size_t i = 0; i < m_groups.size(); ++i)
    {
        if(m_groups[i].second == group)
        {
            if(m_groups[i].first->getList())
            {
                for(agui::ListItem::iterator it = 
                    m_groups[i].first->getList()->getItemsBegin(); it !=
                    m_groups[i].first->getList()->getItemsEnd(); ++it)
                {
                    if((*it).first.text == name)
                    {
                        return true;
                    }
                }
            }

            break;
        }
    }

    return false;

    m_list->repositionGroups();
}

它崩溃了:

if((*it).first.text == name)

我明白了:

Unhandled exception at 0x5fd1942c (msvcp90d.dll) in my.exe: 0xC0000005: Access violation reading location 0x00000040.

调用堆栈如下所示: enter image description here

由于

1 个答案:

答案 0 :(得分:4)

如果没有看到所有代码,我无法确定,但我的猜测是错误与这些行有关:

for(agui::ListItem::iterator it =
                m_groups[i].first->getList()->getItemsBegin(); it !=
                m_groups[i].first->getList()->getItemsEnd(); ++it)

除非您对getList()的调用每次都返回完全相同的列表(即指向同一列表的指针,而不是副本),否则您可能会在不同的列表上获取迭代器。这意味着检查it != m_groups[i].first->getList()->getItemsEnd()始终为false,因为迭代器来自不同的列表。在这种情况下,您的迭代器可能会离开列表的末尾,因此取消引用会导致崩溃。

希望这有帮助!