C++删除vector中负元素时遇段错误（末元素为负时）求助

你的代码出现段错误的核心原因是：调用vector::erase()后，当前迭代器会直接失效。

当你删除元素时，vector会重新排列后续元素的内存位置，原来指向被删除元素的迭代器就变成了无效的“野指针”。如果删除的是容器最后一个元素，iter会直接变成A.end()，之后循环里的iter++会让它越过容器的合法内存范围，触发段错误。

下面给两种靠谱的修复方案：

方案一：手动维护迭代器有效性 #

修改删除元素的循环，利用erase()的返回值更新迭代器——erase()会返回指向被删除元素下一个有效位置的迭代器：

// 正确删除小于1的元素
for (iter = A.begin(); iter != A.end();) {
    if ((*iter) < 1) {
        iter = A.erase(iter); // 用erase返回的有效迭代器更新
    } else {
        ++iter; // 只有不删除元素时才递增迭代器
    }
}

这样每次删除后，iter直接指向了下一个合法元素，不会出现失效问题；不需要删除时再正常推进迭代器。

方案二：使用STL标准的erase-remove惯用法（更推荐） #

这是C++里移除容器特定元素的标准写法，效率更高，代码也更简洁：

// 记得先包含<algorithm>头文件
#include <algorithm>

// ...其他代码...

// 移除所有小于1的元素
A.erase(std::remove_if(A.begin(), A.end(), [](int num) { return num < 1; }), A.end());

原理说明： #

• std::remove_if会遍历容器，把所有满足条件（小于1）的元素移到容器末尾，返回指向第一个被移到末尾的元素的迭代器。
• 接着erase()删除从这个迭代器到容器末尾的所有元素，完成最终的移除操作。
  这种方法避免了多次删除导致的频繁元素移动，效率比循环删除高很多，也从根源上避免了迭代器失效的问题。

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