-1

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

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.

调用堆栈如下所示: 在此处输入图像描述

谢谢

4

1 回答 1

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,因为迭代器来自不同的列表。在这种情况下,您的迭代器可能会离开列表的末尾,因此取消引用会导致崩溃。

希望这可以帮助!

于 2012-06-02T22:05:16.917 回答