Why overloaded ' operator < ' should be const for class?

我是研究僧i 提交于 2019-12-31 02:15:14

问题


Can anybody explain this behavior in context of STL sort algorithm? If operator < is not defined const it gives error,

error: passing ‘const B’ as ‘this’ argument of ‘bool B::operator<(const B&)’ discards qualifiers [-fpermissive] while (__pivot < *__last)

Is sort algo lhs const object or sort is const method?

class B
{
 public:
    ...
    bool operator < (const B& b) const       // why const required here?
    {
        return (m_i < b.m_i);
    } 
    ...

 private:
    int m_i;
    int m_j;
};

int main()
{
  vector<B> Bvec2 {B(5), B(3), B(30), B(20), B(8)};
  std::sort(Bvec2.begin(), Bvec2.end());
  ...
}

回答1:


Marking the function as const promises that it will not change the object. So it can be used on const objects.

The STL almost certainly takes the arguments as const, because that is the smart thing to do.

It shouldn't hurt you to define operator< as const because I cannot imagine having a less-than operator that changes the object. That would just be silly.

If you want to know exactly where here is some code copied out of libstdc++ bits/stl_algo.h on a Fedora 20 machine:

  /// This is a helper function...
  template<typename _RandomAccessIterator, typename _Tp, typename _Compare>
    _RandomAccessIterator
    __unguarded_partition(_RandomAccessIterator __first,
                          _RandomAccessIterator __last,
                          const _Tp& __pivot, _Compare __comp)

const _Tp& __pivot, right there.




回答2:


The standard is a little unclear on this, but [alg.sorting] gives two hints as to why this failure to compile might be standard-conforming behaviour. The first is [alg.sorting]/2:

... It is assumed that comp will not apply any non-constant function through the dereferenced iterator.

Next, we are told that when no comparator is supplied [alg.sorting]/3:

... comp(*i, *j) != false defaults to *i < *j != false

since in your case, comp defaults to *i < *j != false, and this applies a non-const function to the dereferenced iterators. This invalidates the assumption given in [alg.sorting]/2, and so your code has undefined behavior. It is legal for code with undefined behavior to not compile.



来源:https://stackoverflow.com/questions/23927003/why-overloaded-operator-should-be-const-for-class

易学教程内所有资源均来自网络或用户发布的内容,如有违反法律规定的内容欢迎反馈
该文章没有解决你所遇到的问题?点击提问,说说你的问题,让更多的人一起探讨吧!