重载运算符<<为了集

时间:2015-07-20 16:49:00

标签: c++ vector set operator-overloading

我有一组整数,我想打印它,所以我重载了<< set和pair类的运算符为:

template<typename T, typename U>
inline ostream& operator<<(ostream& os, pair<T,U> &p){
    os<<"("<<p.first<<","<<p.second<<")";
    return os;
}


template<typename T>
inline ostream& operator<<(ostream& os, set<T> &s){
    os<<"{";
    for(auto it = s.begin() ; it != s.end() ; it++){
        if(it != s.begin())
            os<<",";
        os<<*it;
    }
    os<<"}";
    return os;
}

当我创建一个集合并输出它时

set<pair<int,int>> s;
cout<<s<<endl;

它给出了错误:

cannot bind ‘std::ostream {aka std::basic_ostream<char>}’ lvalue to ‘std::basic_ostream<char>&&’
   os<<*it;

initializing argument 1 of ‘std::basic_ostream<_CharT, _Traits>& std::operator<<(std::basic_ostream<_CharT, _Traits>&&, const _Tp&) [with _CharT = char; _Traits = std::char_traits<char>; _Tp = std::pair<int, int>]’
     operator<<(basic_ostream<_CharT, _Traits>&& __os, const _Tp& __x)

我不知道是什么问题,错误非常神秘。此外,如果我创建一组整数并打印它,它可以正常工作。

2 个答案:

答案 0 :(得分:5)

it中的auto it = s.begin()类型为const_iteratorsource)。因此,当您致电os<<*it;时,您需要一个可以const的功能  对。如果您将代码更改为此代码,它将起作用:

#include <iostream>
#include <set>
#include <utility>

using namespace std;

template<typename T, typename U>
inline ostream& operator<<(ostream& os, const pair<T,U> &p){
    os<<"("<<p.first<<","<<p.second<<")";
    return os;
}


template<typename T>
inline ostream& operator<<(ostream& os, const set<T> &s){
    os<<"{";
    for(auto it = s.begin() ; it != s.end() ; it++){
        if(it != s.begin())
            os<<",";
        os<<*it;
    }
    os<<"}";
    return os;
}

int main()
{
    set<pair<int,int>> s {{1,2}};
    cout<<s<<endl;
}

Live Example

我还建议您始终将第二个参数作为<{p}的const &

  1. 您可以将临时绑定到const &(ex函数返回)

  2. 您不应该在输出期间修改容器,因此这使用C ++类型系统来强制执行该操作。

答案 1 :(得分:2)

流运算符并不意味着修改它们传递的值。因此,您应该接受const引用而不是可变引用:

template<typename T, typename U>
inline ostream& operator<<(ostream& os, const pair<T,U> &p){
    os<<"("<<p.first<<","<<p.second<<")";
    return os;
}


template<typename T>
inline ostream& operator<<(ostream& os, const set<T> &s){
    os<<"{";
    for(auto it = s.begin() ; it != s.end() ; it++){
        if(it != s.begin())
            os<<",";
        os<<*it;
    }
    os<<"}";
    return os;
}