使用STL std::remove时出现operator==编译错误的问题排查（vec为std::vector<std::string>类型）

哦，这个坑我之前也踩过！你现在的编译错误和代码逻辑有两个关键问题，咱们一步步拆解：

1. 错误根源：用错了STL算法 #

你现在用的std::remove的作用是移除容器中等于某个指定值的所有元素，它的第三个参数是「要匹配的目标值」。所以编译器会尝试把vector里的std::string元素和你传入的Compare对象做==比较——这俩类型完全不搭，自然找不到对应的operator==，直接报错。

而你的需求是「按自定义条件移除元素」，这时候应该用**std::remove_if，它的第三个参数是一个谓词（返回bool的函数/仿函数）**，用来判断哪些元素需要被移除。

2. 仿函数实现不符合谓词要求 #

你写的Compare类的operator()返回的是std::string，这完全不符合remove_if的要求：谓词必须返回bool值，告诉算法「这个元素是否需要被移除」。而且你原来的代码里，当子串匹配时没有返回值，这属于未定义行为。

修正后的代码 #

第一步：修复仿函数 #

把operator()改成返回bool，并且逻辑上明确：返回true表示该元素需要被移除（也就是和query不匹配的情况）。同时优化参数传递，避免不必要的拷贝：

class Compare {
private:
    size_t m_from;
    size_t m_to;
    std::string m_query;
public:
    // query用const引用传递，避免拷贝
    Compare(size_t from, size_t to, const std::string& query) 
        : m_from(from), m_to(to), m_query(query) {}
    
    // 谓词函数必须返回bool，参数用const引用避免拷贝，函数加const保证可以被const对象调用
    bool operator()(const std::string& s) const {
        // 需求是移除不匹配的，所以当子串不等时返回true（标记为要移除）
        return s.substr(m_from, m_to) != m_query.substr(m_from, m_to);
    }
};

第二步：用std::remove_if替代std::remove #

同时注意erase的参数：remove_if会返回指向第一个待移除元素的迭代器，你需要把这个迭代器到容器末尾的所有元素都删掉：

Compare compare(0, i, query);
// 传递remove_if的返回值和vec.end()给erase
vec.erase(std::remove_if(vec.begin(), vec.end(), compare), vec.end());

额外提醒 #

• 如果你用的是C++11及以上版本，也可以用lambda表达式替代自定义仿函数，代码会更简洁：

vec.erase(std::remove_if(vec.begin(), vec.end(), 
    [&](const std::string& s) {
        return s.substr(0, i) != query.substr(0, i);
    }), vec.end());

内容的提问来源于stack exchange，提问作者kobi