我有一个包含活动或非活动项目的向量。我希望此向量的大小保持较小以解决性能问题，因此我希望从向量中删除已标记为非活动的项目。我在迭代时尝试这样做，但出现错误“向量迭代器不兼容”。
vector<Orb>::iterator i = orbsList.begin();
while(i != orbsList.end()) {
bool isActive = (*i).active;
if(!isActive) {
orbsList.erase(i++);
}
else {
// do something with *i
++i;
}
}
【问题讨论】：
从向量中间删除项目将使该向量的所有迭代器无效，因此您不能这样做（更新：不求助于 Wilx 的建议）。
另外，如果您担心性能，从矢量中间擦除项目无论如何都是一个坏主意。也许您想使用std::list？
【讨论】：
您可以这样做，但我认为您将不得不重新调整您的while()。 erase() 函数返回一个迭代器，指向擦除后的下一个元素：iterator erase(iterator position);。引用 23.1.1/7 的标准：
a.erase(q) 返回的迭代器 立即指向元素 在元素存在之前跟随 q 抹去。如果不存在这样的元素， a.end() 被返回。
虽然也许您应该改用Erase-remove idiom。
【讨论】：
i=orbsList.erase(i) 而不是orbsList.erase(i++) 可能就足够了
正如他们所说，无论您使用哪种形式的迭代器增量，vector 的迭代器都会在 vector::erase() 上失效。请改用整数索引。
【讨论】：
您可能需要考虑为您的数据结构使用std::list 而不是std::vector。将擦除与迭代结合使用会更安全（不易出错）。
【讨论】：
过去我最易读的方法是使用std::vector::erase 和std::remove_if。在下面的示例中，我使用此组合从向量中删除任何小于 10 的数字。
(对于非 c++0x，您只需将下面的 lambda 替换为您自己的谓词：)
// a list of ints
int myInts[] = {1, 7, 8, 4, 5, 10, 15, 22, 50. 29};
std::vector v(myInts, myInts + sizeof(myInts) / sizeof(int));
// get rid of anything < 10
v.erase(std::remove_if(v.begin(), v.end(),
[](int i) { return i < 10; }), v.end());
【讨论】：
std::remove_if lambda 看起来不错，但如果我只想删除一个符合条件的元素，它仍然会遍历每个元素直到最后。因此，我更喜欢手动迭代，这样我就可以随时跳出循环。
我同意 wilx 的回答。这是一个实现：
// curFiles is: vector < string > curFiles;
vector< string >::iterator it = curFiles.begin();
while(it != curFiles.end()) {
if(aConditionIsMet) {
it = curFiles.erase(it);
}
else ++it;
}
【讨论】：
如果有人需要处理索引
vector<int> vector;
for(int i=0;i<10;++i)vector.push_back(i);
int size = vector.size();
for (int i = 0; i < size; ++i)
{
assert(i > -1 && i < (int)vector.size());
if(vector[i] % 3 == 0)
{
printf("Removing %d, %d\n",vector[i],i);
vector.erase(vector.begin() + i);
}
if (size != (int)vector.size())
{
--i;
size = vector.size();
printf("Go back %d\n",size);
}
}
【讨论】：
--i nad i-- 之间的区别
erase 返回指向下一个迭代器值的指针（与 Vassilis 相同）：
vector <cMyClass>::iterator mit
for(mit = myVec.begin(); mit != myVec.end(); )
{ if(condition)
mit = myVec.erase(mit);
else
mit++;
}
【讨论】：
*mit访问迭代器位置的值