使用迭代器从C ++中的向量末尾删除元素

时间:2014-04-12 23:19:34

标签: c++ vector iterator time-complexity invalidation

我想从最后一个元素开始以相反的顺序从向量中删除元素。据我所知,使用迭代器执行此操作的最简单方法是:

std::vector<int> data = ...
std::vector<int>::iterator iter;
for(iter = data.end() - 1; iter != data.begin() - 1; iter--) {
    data.erase(iter);
}

我的问题是,这些erase()调用中的每一个都是O(1),因为我们最后会删除,即使删除的一般复杂性是O(n)?此外,这段代码是否安全&#34;假设向量至少有1个元素?

3 个答案:

答案 0 :(得分:4)

不,你的算法有UB。 如果没有,它会有O(n)。
将身体更改为此将删除UB:iter = data.erase(iter);
为什么?因为vector.erase()的行为如下:

  • 效果:在擦除点或之后使迭代器和引用无效。
  • 复杂性:T的析构函数被称为等于元素数量的次数 擦除,但T的移动赋值运算符被称为等于数的次数 擦除元素后向量中的元素。

如果像这样重写循环,它甚至可以用于空容器:

for(auto it = data.end(); it!=data.begin(); it = data.erase(it-1)) /**/;

等于:

std::remove_if(data.rbegin(), data.rend(), [](data::reference x){return true;});

无论如何,使用data.clear()data.resize(0)或使用临时空向量进行交换很可能会更快。

答案 1 :(得分:3)

除非您真的需要以相反的顺序删除它们,否则最好只需拨打data.clear()

否则,不,它不是“安全”:形成data.begin() - 1未定义,并且在您调用iter之后递减.erase也是未定义的。 Deduplicator has posted a good fix for that.

答案 2 :(得分:0)

有点儿。 std :: vector erase()的复杂性是

  

删除的元素数量(destructions)加上最后一个元素删除(移动)后的元素数量。

基于此来源:http://www.cplusplus.com/reference/vector/vector/erase/

当你只删除一个元素及其最后一个元素时,它的线性与N = 1,可以解释为O(1)constantish。