C ++重载运算符<错误

Question

当我没有将const 放入函数bool operator<(const Node& otherNode) //const 时，为什么会收到错误消息？

stl_algo.h:91: error: passing 'const Node' as 'this' argument of 'bool Node::operator<(const Node&)' discards qualifiers

所有重载的运算符都应该是常量吗？

class Node {
public:
    double coordinate;

    bool operator==(const Node& other) const{
        return coordinate == other.coordinate;
    }

    bool operator<(const Node& other) const{
        return coordinate < other.coordinate;
    }
};

Answer 1

不是所有的操作符，但== 和< 绝对应该设为const，是的。从逻辑上讲，它们不会修改任何被比较的对象。

错误可能来自于从const 调用非const 方法，例如：

bool isSmaller(const Node& other) const
{
   return *this < other;
}

在这种情况下，由于方法 isSmaller 是 const，this 隐含地是 const 对象，因此 operator < 也必须是 const 以便在该上下文中调用有效。

从错误消息来看，const 对象上似乎正在调用 Node::operator <，来自 stl_algo.h 中的函数 - 排序/排序函数、散列函数等。

Answer 2

比较运算符，例如<、>、<=、>=、==、!= 通常应该对 const 对象进行操作，因为如果任何对象被比较可以通过比较来改变。但是您可以将比较声明为非成员函数，以确保两个操作数之间的对称性。

class Node {
public:
    double coordinate;
};
inline operator<(const Node& lhs, const Node& rhs)
{
  return lhs.coordinate < rhs.coordinate;
}

Answer 3

您是否尝试删除方法的 const 修饰符？此外，正如@LuchianGrigore 所建议的，您可以使用this 关键字：

bool operator< (const Node& other) {
    return this.coordinate < other.coordinate;
}