c++ - 对于具有相同调用堆栈的同一对象，从同一线程调用函数两次

Question

背景

我正在调试一个开源项目，我愿意加入并学习 C++。然后我尝试扩展其中一个问题所需的一些功能，我偶然发现了一个有趣的案例(我一生中从未见过这样的事情)。

源代码

QuestSet Player::GetQuestForEvent(uint16 eventId) const
{
    QuestSet eventQuests; // QuestSet is typedef for std::set<uint32>

    for (uint8 i = 0; i < MAX_QUEST_LOG_SIZE; ++i)
    {
        uint32 questId = GetQuestSlotQuestId(i);
        if (questId == 0)
            continue;

        QuestStatusMap::const_iterator qs_itr = m_QuestStatus.find(questId);
        if (qs_itr == m_QuestStatus.end())
            continue;

        QuestStatusData const& qs = qs_itr->second;

        Quest const* qinfo = sObjectMgr->GetQuestTemplate(questId);
        if (!qinfo)
            continue;

        if (qinfo->GetEventIdForQuest() == eventId)
            eventQuests.insert(questId);
    }

    return eventQuests; <--- breakpoint here
}

我注意到我的服务器崩溃了

Unhandled exception at 0x00007FFE1BE6A388 (KernelBase.dll) in worldserver.exe: 0xC0000005: Access violation.

接着是来 self 服务器的这个日志

C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.20.27508\include\xtree(240) : Assertion failed: map/set iterators in range are from different containers

所以我在上述位置放置了断点，我注意到该函数被调用了两次。我知道上传图片可能不是最佳选择，但我认为这是对这种情况的最佳解释，所以请看一看。

截图

• 第一个断点命中

• 第二次断点命中(诊断工具窗口中的通知事件窗口)

这是项目中唯一调用此函数的地方。

void OnLogin(Player* player, bool /*firstLogin*/) override
{
    QuestSet eventQuests = player->GetQuestForEvent(1);
    /*std::for_each(eventQuests.begin(), eventQuests.end(), [&player](uint32 questId)
    {
        player->AbandonQuest(questId);
    });*/
}

我知道这听起来很傻，但我没有建议，请你指出正确的方向，理解为什么这个断点被击中两次很可能也会修复异常。

故障排除:

• 项目是在调试配置中构建的
• 调用堆栈、线程和this 对象在两次命中中都是相同的。

Answer 1

很可能该函数实际上并未被调用两次。两次击中同一个断点只是得出了错误的结论。

Visual Studio 允许您检查断点附近的汇编代码，默认值为 Ctrl+Alt+D我相信。在那里您会看到混合的汇编指令(您的 CPU 实际执行的操作)和“以下指令源自哪一行代码”的调试信息。

我希望您在那里看到的是 return eventQuests; line 与两个汇编指令 block 相关联，另一个指令 block 介于两者之间。您将首先在第一组指令中命中断点，然后执行第二组指令(例如与循环相关联，或右括号)，然后由于第二组指令再次命中断点。是的，这听起来很愚蠢，但 MSVC 输出的调试信息并不是我见过的最好的。

另一种简单的检查方法是在返回之前插入打印语句(如 std::cout << "Hi" << std::endl )并检查您是否得到一个或两个打印输出。除非你得到两份打印输出，否则你实际上多次输入函数的结论是错误的。