删除vector末尾元素后迭代器失效引发循环异常的问题排查

你的代码出现无限循环和垃圾值的核心原因是vector的erase操作会导致迭代器失效，删除末尾元素时的问题尤为明显：

• 当执行v.erase(it)删除最后一个元素（值为7的元素）时，erase会返回指向被删除元素下一个位置的迭代器，但你没有用这个返回值更新it。此时原it已经变成失效迭代器——因为被删除的是最后一个元素，它原本指向的位置已不属于vector的有效内存范围。
• 后续循环执行++it时，对失效迭代器做自增操作属于C++标准里的未定义行为，会让迭代器指向内存中的随机区域，这就是你看到垃圾值的原因。同时，失效的迭代器永远无法等于v.end()，循环会无限运行下去。
• 至于删除其他位置元素时看似正常，那只是巧合：删除非末尾元素后，erase会把后续元素前移，原it之后的迭代器都会失效，但你的代码继续++it可能刚好跳过了失效区域，本质上依然是错误的写法。

必须利用erase的返回值更新迭代器，避免失效问题：

vector<int> v;
v.push_back(1);
v.push_back(2);
v.push_back(3);
v.push_back(4);
v.push_back(5);
v.push_back(6);
v.push_back(7);

for (vector<int>::iterator it = v.begin(); it != v.end(); )
{
    if (*it == 7)
        it = v.erase(it); // 用erase返回的有效迭代器更新it
    else
    {
        cout << *it << endl;
        ++it; // 仅未删除元素时才自增迭代器
    }
}

这个写法的关键逻辑：

• 把迭代器自增操作从循环头移到else分支中
• 删除元素时，erase返回的迭代器已经指向了下一个有效元素，无需额外自增
• 未删除元素时，正常自增迭代器遍历下一个元素

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