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

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() 的方法，这会导致访问冲突。

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

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

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

会不会是地址变了？我覆盖了吗？

请帮帮我！另外，我感谢有关此代码不合适的任何信息的信息:)

Answer 1

0xcdcdcdcd 是 Win32 调试 CRT 堆使用的填充模式；假设您在 Windows 上的调试器中运行，这可能是一个重要的线索。有一个不错的explanation of the heap fill patterns here。

我的猜测是你要么在其他地方使迭代器无效，要么有其他缓冲区溢出或悬空指针或其他问题。

Application Verifier 可能有助于诊断。您可能还想查看问题How to debug heap corruption errors? 中提到的其他内容或Any reason to overload global new and delete? 中的一些技术（免责声明：目前我对这两个问题都有最高评价）。

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

看起来 Visual Studio 有 debug iterator support，如果你正在使用它，它可能符合要求。

Answer 2

你的终止条件似乎错误[编辑：实际上它看起来是正确的，即使它仍然让我害怕。见 cmets。]。当finished 变为真时，你会擦除当前的弹丸，然后将迭代器设置为 ... 一些东西。没有理由认为循环将在此时终止。 it 很可能最终会开始指向列表之外，或者指向其他无效的对象（您的调试器不会总是这样标记）。

当你擦除弹丸时，你应该明确地使用“break”离开循环。