遍历C++ vector时触发段错误的原因及优化迭代方式咨询

你的代码在第6-7次循环时触发段错误，核心问题是用范围for循环遍历vector的同时执行erase操作，导致迭代器失效。范围for循环本质依赖迭代器实现，erase会使当前及之后的迭代器失效，继续遍历就会触发未定义行为（比如访问已释放的内存）。

最优解决方案：erase-remove惯用法 #

C++标准库提供了专门的批量删除元素的范式，既高效又避免迭代器问题，代码更简洁：

#include <vector>
#include <algorithm>  // 需要包含algorithm头文件
#include <synchapi.h>

class someClass{
public:
    int x;
    int y;
    int health = 0;
    someClass(int a, int b) : x(a), y(b) {}  // 用初始化列表更规范
};

std::vector<someClass> classList;

int main() {
    for (int i = 0; i < 7; ++i) {
        classList.emplace_back(i, i*3);  // emplace_back直接构造对象，比push_back更高效
    }

    while(true) {
        // erase-remove惯用法：移除所有health为0的元素
        classList.erase(
            std::remove_if(classList.begin(), classList.end(),
                [](const someClass& e) { return e.health == 0; }
            ),
            classList.end()
        );
        Sleep(50);
    }
    return 0;
}

原理说明：

• std::remove_if会把所有不满足删除条件的元素移到vector的前半部分，返回指向第一个待删除元素的迭代器
• vector::erase则删除从该迭代器到末尾的所有元素，完成批量删除

手动迭代器遍历（适合需要额外逻辑的场景） #

如果遍历过程中除了删除还要做其他操作，可以手动用迭代器控制，利用erase返回的有效迭代器更新循环变量：

while(true) {
    if(!classList.empty()) {
        for (auto it = classList.begin(); it != classList.end(); ) {
            if (it->health == 0) {
                it = classList.erase(it);  // erase返回下一个有效迭代器，不用手动++
            } else {
                ++it;  // 不删除时才移动迭代器
            }
            // 这里可以添加其他操作逻辑
        }
    }
    Sleep(50);
}

原代码崩溃原因 #

原代码用iter手动维护索引，但范围for循环的迭代器在erase后已经失效，继续循环时e会指向已被删除的元素或非法内存，最终触发段错误。手动维护索引的方式极易出错，尤其是容器元素数量变化时，索引和实际元素的对应关系会混乱。

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