从vector中移除非长度为2的字符串时未完全移除问题排查

嘿，我来帮你搞定这个bug！你遇到的问题其实是C++里遍历容器删除元素时的经典坑——迭代器失效或者索引错位，导致像"lol"这样的元素被跳过了。

为什么会漏删？ #

举个例子，假设你用了普通的下标循环：

// 错误示例：会导致漏删
for (size_t i = 0; i < vec.size(); ++i) {
    if (vec[i].size() != 2) {
        vec.erase(vec.begin() + i);
    }
}

当你删除第i个元素后，vector里后面的所有元素都会往前移动一位。但循环里的++i会让你直接跳到原来的i+1位置，而原来的i+1元素现在已经跑到i的位置了，这就导致这个元素被跳过检查，所以像"lol"这种元素就没被删掉。

如果是用普通迭代器循环但没处理返回值，也会出现类似问题：

// 错误示例：迭代器失效
for (auto it = vec.begin(); it != vec.end(); ++it) {
    if (it->size() != 2) {
        vec.erase(it); // erase后it已经失效，再++it会触发未定义行为
    }
}

正确的解决方案 #

这里给你三种靠谱的做法，优先推荐第一种：

1. 使用erase-remove惯用法（STL标准最优解）

这是C++处理容器元素删除最简洁高效的方式，利用STL算法把要删除的元素移到容器末尾，再一次性删除：

#include <algorithm> // 需要包含这个头文件

// 移除所有长度不为2的字符串
vec.erase(
    std::remove_if(vec.begin(), vec.end(), 
        [](const std::string& s) { return s.size() != 2; }
    ),
    vec.end()
);

• std::remove_if会把所有满足"长度不为2"的元素移到vector的末尾，返回指向第一个待删除元素的迭代器。
• 再用vec.erase()把末尾的这些元素一次性删掉，效率比逐个删除高很多。

2. 用迭代器遍历并正确处理erase的返回值

如果你一定要手动遍历迭代器，记得用erase的返回值更新迭代器（erase会返回下一个有效的迭代器）：

for (auto it = vec.begin(); it != vec.end();) {
    if (it->size() != 2) {
        it = vec.erase(it); // 用返回的迭代器继续遍历
    } else {
        ++it; // 只有不删除的时候才递增迭代器
    }
}

3. 反向下标遍历

从vector的末尾往前遍历，这样删除元素不会影响前面未遍历的元素位置：

for (int i = vec.size() - 1; i >= 0; --i) {
    if (vec[i].size() != 2) {
        vec.erase(vec.begin() + i);
    }
}

验证效果 #

用上面的任意一种方法，你的vector最终都会只保留长度为2的字符串，不会再出现漏删的情况啦！

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