我需要实现一个for_each
函数,如下所示。我知道std::for_each
可以将fn
应用于每个元素,但我们不能擦除std::for_each
中的元素。我需要扩展这个模板函数,以便在fn
中,调用者可以同时访问元素和擦除元素。有合适的方法吗?
#include <iostream>
#include <vector>
#include <algorithm>
using namespace std;
class A
{
public:
explicit A(){
mVec.clear();
}
~A(){}
template<class T> void for_each(T fn)
{
for(size_t i = 0; i < mVec.size(); ++i)
{
//fn can erase element or just visit element
fn(mVec[i]);
}
}
vector<int> mVec;
};
int main()
{
A test;
for(int i = 0; i < 8; ++i)
{
test.mVec.push_back(i);
}
test.for_each([&test](int i){
if (i % 2 == 0)
{
cout << i << " deleted" << endl;
test.mVec.erase(find(test.mVec.begin(), test.mVec.end(), i));
}
else
{
cout << i << " parse" << endl;
}
});
system("pause");
return 0;
}
编辑:在for_each
模板函数中,我们不知道调用方是否会擦除元素。擦除元素在fn
中完成
您能从函数中返回一个bool
值吗?其中true
的意思是"擦除元素"?然后你的for_each函数变成了类似的函数。
size_t i = 0;
for(size_t j = 0; j < mVec.size(); ++j) {
if (!fn(mVec[j])) {
// The element must be kept
if (i != j)
mVec[i] = std::move(mVec[j]);
i++;
}
}
mVec.resize(i);
优点还在于,无论擦除多少元素,这总是O(n)。
EDIT:上面的循环实际上只是std::remove_if(),所以@ChenOT的建议是最好的。或者
n = std::remove_if(mVec.begin(), mVec.end(), fn) - mVec.begin();
mVec.resize(n);