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; }

Answer 3

您应该使用const个操作数编写运算符。看看here。

错误：将'const T'作为'bool T :: operator＆lt;（T）'的'this'参数传递'丢弃限定符

3 个答案: