使用remove_if从向量中删除元素

Question

我试图删除矢量元素使用remove_if.但没有成功.我究竟做错了什么？

这是我的代码:

#include <iostream>
#include <string>
#include <vector>
#include <algorithm>

void printme(std::vector<int>& a){
    for(const auto& item: a)
    std::cout << item << std::endl;
}

int main()
{
    std::vector<int> a {1, 2, 3, 4, 5, 6};
    printme(a);  
    a.erase( (std::remove_if(a.begin(), a.end(), [](const int& x){
        return x == 2;
        }), a.end()));
    printme(a);
}

我的输出只是:

1 2 3 4 5 6

预期产量:

1 2 3 4 5 6 1 3 4 5 6

Answer 1

您正在使用std::vector::erase()成员函数的重载,该函数将单个迭代器作为参数.由于erase()您的参数是提供迭代器a.end(),因为以下表达式:

(std::remove_if(a.begin(), a.end(), [](const int& x){ return x == 2; }), a.end()))

评估为a.end()(即,因为逗号运算符).

传递给迭代器的迭代器erase()需要一个迭代器才能解除引用.但是,迭代器a.end()不能解除引用,因此,调用会erase()导致未定义的行为.

要使用带有两个迭代器的重载,请删除调用周围的括号std::remove_if:

a.erase(std::remove_if(a.begin(), a.end(), [](const int& x){
        return x == 2;
        }), a.end());

Answer 2

您正在添加多余的括号,将其更改为

a.erase( std::remove_if(a.begin(), a.end(), [](const int& x){
    return x == 2;
    }), a.end());

需要注意的是逗号操作只是返回最后一个操作数,这意味着你传递a.end()到erase,从而导致UB.

Answer 3

其他答案指出了问题所在.我想说,通过简化代码可以更容易地注意到这些问题.

我建议使用:

int main()
{
   std::vector<int> a {1, 2, 3, 4, 5, 6};
   printme(a);  

   auto it = std::remove_if(a.begin(), a.end(), [](const int& x){ return x == 2; });
   a.erase(it, a.end());

   printme(a);
}

Answer 4

你在函数调用中只有太多的括号.

a.erase(std::remove_if(a.begin(), a.end(), [](const int& x) {return x == 2;}), a.end());

只需std::remove_if在通话结束前和通话结束时删除一个括号即可.