1

i'm trying to do a simple istream operator overloading, but for some reason, once entering this function, the program enters an infinite loop. please help!

my code:

#include <iostream>
#include <string>
using namespace std;

 class date{

int m_day,m_month,m_year;

public:

date(int day=1,int month=1,int year=2000){    //constructor
    if (day>0 && day<32 && month>0 && month<13){
        m_day =day;
        m_month=month;
        m_year=year;
    }
}


friend ostream& operator<< (ostream& out, const date& d);
friend istream& operator>> (istream& in, const date& d);
};


istream& operator>> (istream& stream, const date& d){              //overload >>
stream >> d.m_day;
return stream;

}

void main(){  

date date1;

cin>>date1;                   //check istream

getchar();
}
4

2 回答 2

7

这段代码对我来说似乎是错误的，因为您正在尝试修改const对象 (d)。

istream& operator>> (istream& stream, const date& d){              //overload >>
    stream >> d.m_day;
    return stream;    
}
于 2013-05-17T13:45:27.533 回答
2

实际上，Marc 的回答并不能完全解决您的问题。发生的事情有点复杂：

您定义了接收const &. 那应该使编译器简单地拒绝编译您的函数。但问题就在这里：你的构造函数date对于每个参数都有默认值。因此，它可以只用一个参数调用。由于它没有被标记explicit，它可以作为 int 的隐式转换运算符（如date(i)）。这意味着编译器可以解释stream >> d.m_day为stream >> date(d.m_day). 因此，您的问题是：这是一个要求operator>>(istream& stream, const date& d)，您会得到无限递归。

因此，你不仅应该date& d在你的重载上使用非常量operator>>，你还应该标记构造函数explicit，比如

explicit date(int day=1, int month=1, int year=2000)

这样它就不会充当隐式转换运算符。实际上，这应该对几乎每个可能只接受一个参数的构造函数进行。

于 2015-09-28T18:56:00.337 回答