2015-07-20 94 views
1

我有一组整数对的,我想打印出来,所以我重载< <运营商设定和对类如:重载运算符<<为集

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) 

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

回答

5

auto it = s.begin()类型的itconst_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

我也建议你总是拿第二个参数为const &

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

  2. 您不应该在修改容器时输出,所以这使用C++类型系统来执行。

+0

谢谢,实际上,当我为矢量(而不是集合)写这个,它工作,这就是为什么我感到困惑。 –

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