错误:将'const T'作为'bool T :: operator <(T)'的'this'参数传递,丢弃限定符

Question

#include <iostream>
#include <vector>
#include <algorithm>
class MyData
    {
    public:
        int m_iData;
        bool operator<(MyData rhs) { return m_iData < rhs.m_iData; }
    };

int main () 
{
    std:: vector <MyData> myvector(2, MyData() );
    myvector[0].m_iData=2; myvector[1].m_iData=4;

    std::sort(myvector.begin(), myvector.end()); 
}

试图编译它给出:

error: passing 'const MyData' as 'this' argument of 'bool MyData::operator<(MyData)'
discards qualifiers

Answer 1

比较运算符将在类实例的const-reference上调用,因此必须将其声明为const成员函数.

通过const-reference而不是value传递参数也是一种好习惯,尽管它对你的简单类没有多大影响:

bool operator<(const MyData & rhs) const { return m_iData < rhs.m_iData; }
//             ^^^^^^^^^^^^^^      ^^^^^
//             if you like         mandatory

通常强烈建议将所有成员函数声明为常量,这不会改变您的对象.这不仅可以传达您的意图和设计,而且也不可能在常量对象或引用上使用这些函数.

Answer 2

一个operator<应该是const两个参数:

    bool operator<(MyData const& rhs) const { return m_iData < rhs.m_iData; }