错误写法
循环内直接erase
#include <iostream>
#include <vector>
#include <string>
using namespace std;
int main(int argc, char* argv[])
{
vector<string> nameList;
nameList.push_back("小明");
nameList.push_back("小亮");
nameList.push_back("小丽");
nameList.push_back("大壮");
for(auto it = nameList.begin(); it != nameList.end(); ++it)
{
cout << *it << endl;
if(*it == "小亮")
{
nameList.erase(it);
}
}
return 0;
}
如上输出结果为:
小明
小亮
大壮
漏掉了处理小丽
错误原因分析
执行erase后, 被erase的迭代器会自动指向下一元素, erase后继续执行迭代器++ 操作导致跳过了一个元素
正确写法
循环代码改写为如下, 首先for循环右边不执行每次迭代器++操作,
命中要erase的对象情况用it接收返回值,这个返回的迭代器即是erase后的下一元素,所以这里不需要执行迭代器自增了
其他情况执行迭代器自增即可
for(auto it = nameList.begin(); it != nameList.end(); )
{
cout << *it << endl;
if(*it == "小亮")
{
it = nameList.erase(it);
}
else
{
++it;
}
}