迭代std::list时移除Bullet对象报错，如何解决？

我正在创建一个存储屏幕上抛射物对象的列表，这些对象执行不同方法的功能正常，但尝试移除屏幕外的对象时持续报错。代码如下：

std::list<Bullet> bullets;
//bullets are added on mouse click

//start iterating through objects to move
std::list<Bullet>::iterator mover;
for (mover = bullets.begin(); mover != bullets.end(); mover++){
    //Checks if bullet is off screen
    if (mover->x < 0 || mover->x > 800 || mover->y < 0 || mover->y > 800) {
        /*
          The deconstructor works, but when bullets.erase(mover); is done, a hard exception occurs.
          I believe it may be due to it iterating to a place with nothing, but I am unsure,
          and more unsure on how to fix it
        */
        mover->~Bullet();
        bullets.erase(mover);
        std::cout << "Kablooey" << std::endl;
        //an attempted fix that doesn't seem to do anything
        if (mover == bullets.end()) {
            break;
        }
    }
    //makes current object move afterwards, could also be source of error?? 
    //Should I make it check if there is anything there first?
    mover->Move();
    std::cout << "nyoom" << std::endl;
}

错误提示：Expression: List iterators incompatible

请问如何正确从列表中删除对象而不触发错误？

• 迭代器失效：调用bullets.erase(mover)后，原迭代器mover会失效，后续代码中mover++和mover->Move()访问无效迭代器，直接触发迭代器不兼容的错误。
• 手动析构多余：std::list::erase会自动调用对象的析构函数，手动执行mover->~Bullet()会导致对象被重复析构，引发未定义行为。

方案一：利用erase返回值更新迭代器 #

std::list::erase会返回指向被删除元素下一个位置的有效迭代器，用它更新循环迭代器即可避免失效问题：

std::list<Bullet>::iterator mover = bullets.begin();
while (mover != bullets.end()) {
    if (mover->x < 0 || mover->x > 800 || mover->y < 0 || mover->y > 800) {
        // 无需手动调用析构，erase会自动处理
        mover = bullets.erase(mover);
        std::cout << "Kablooey" << std::endl;
    } else {
        mover->Move();
        std::cout << "nyoom" << std::endl;
        mover++;
    }
}

改用while循环更直观：删除元素时直接用返回的迭代器继续遍历；未删除时才手动递增迭代器，避免for循环中mover++的冲突。

方案二：使用erase-remove惯用法（C++11及以上） #

如果编译器支持C++11或更高版本，用std::remove_if结合erase的erase-remove惯用法，代码更简洁高效：

// 标记需要删除的元素
auto new_end = std::remove_if(bullets.begin(), bullets.end(), [](const Bullet& b) {
    bool off_screen = b.x < 0 || b.x > 800 || b.y < 0 || b.y > 800;
    if (off_screen) {
        std::cout << "Kablooey" << std::endl;
    }
    return off_screen;
});
// 真正从容器中删除元素
bullets.erase(new_end, bullets.end());

// 遍历剩余元素执行移动操作
for (auto& bullet : bullets) {
    bullet.Move();
    std::cout << "nyoom" << std::endl;
}

这种方式将删除逻辑和移动操作分离，彻底避免迭代器失效问题，代码可读性也更强。

内容的提问来源于stack exchange，提问作者DAG3223