调用非虚拟成员函数时发生访问冲突

Question

我有一段代码运行了很多次，突然报告访问冲突。

for(std::list<Projectile>::iterator it = projectiles.begin(); it != projectiles.end();) {
    bool finished = ...

    // try to print address of collides(), prints always 1
    std::cout << &Projectile::collides << std::endl; 
    // here it crashes:
    if(it->collides(&hero)) { 
        std::cout << "Projectile hits " << hero.getName() << std::endl;
        finished = it->apply(&hero);
    }

    // ...
    if(finished) {
        it = projectiles.erase(it);
    } else {
        ++it;
    }
}

因此，VS 调试堆栈跟踪显示，在 if(it->collides(&hero)) { 行中，程序尝试调用 cdcdcdcd() 处的方法，这会导致访问违规。

根据 VS，it、*it 和 hero 是有效对象。

所以我假设cdcdcdcd()实际上应该是collides()。由于 Collides 是一种非虚拟方法，因此它的地址基本上不应该改变，或者？

问题是方法 collides() 在成功之前执行了几次，但突然它不再起作用了。

难道是地址改了？我是否覆盖了它？

请帮我！我也很欣赏有关此代码的任何问题的信息:)

Answer 1

0xcdcdcdcd 是 Win32 调试 CRT 堆使用的填充模式；假设您正在 Windows 上的调试器中运行，这可能是一个重要的线索。这里有一个不错的堆填充模式的解释。

我的猜测是，您以某种方式使其他地方的迭代器无效，或者存在其他缓冲区溢出或悬空指针或其他问题。

应用程序验证程序可能有助于诊断此问题。您可能还想研究问题如何调试堆损坏错误中提到的其他内容？ 或任何原因中的一些技术重载全局 new 和删除？ （免责声明：目前我对两者都有最高评价的答案）。

如果您的 STL 库具有调试功能，它也可能有助于解决这个问题。例如，Metrowerks（现为 Freescale）Codewarrior 具有定义 _MSL_DEBUG，可用于构建标准库的版本（包括 std::list），该版本将检测迭代器失效等常见问题（需要一定的运行时成本）。

看起来 Visual Studio 有 调试迭代器支持< /a> 如果您正在使用它，这可能符合要求。

Answer 2

你的终止条件似乎是错误的[编辑：实际上它看起来是正确的，即使它仍然让我害怕。请参阅评论。]。当finished 变为 true 时，您将擦除当前的射弹，然后将迭代器设置为...某物。没有理由认为循环将在此时终止。它很可能最终要么开始指向列表之外，要么指向一个无效的对象（调试器并不总是这样标记）。

当你擦除射弹时，你应该使用“break”显式地离开循环。