我想从最后一个元素开始以相反的顺序从向量中删除元素。据我所知,使用迭代器执行此操作的最简单方法是:
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个元素?
答案 0 :(得分:4)
不,你的算法有UB。
如果没有,它会有O(n)。
将身体更改为此将删除UB:iter = data.erase(iter);
为什么?因为vector.erase()
的行为如下:
如果像这样重写循环,它甚至可以用于空容器:
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。