在 C++ 中重载日期类的 >> 运算符会导致无限循环答案

【问题标题】：Overloading >> operator for Date class in c++ causes infinite loop在 C++ 中重载日期类的 >> 运算符会导致无限循环
【发布时间】：2012-05-30 05:59:07
【问题描述】：

我正在尝试为 c++ 中的 Date 类重载 >> 运算符，但是当运行进入第一个 if 语句时，它会进入无限循环，你能帮帮我吗？

//operator
    istream& operator >>(istream& is,CustomDate& d){
        int day,month,year; 
        char ch1,ch2;
        string test;
        is>>skipws>>day>>ch1>>month>>ch2>>year;
        if(!is){
            is.clear(ios_base::failbit);
            return is;
        }
        if(ch1!='/' || ch2!='/')
            error("invalid date input");
        d = CustomDate(day,month,year);
        return is;
    }

这是调用它的函数

CustomDate Menu::inputDate(){
    CustomDate date;
    cout<<"Input your departure date"<<endl;
    cin>>date;
    if(!cin){
        error("invalid date format");
    }
    return date;
}

这是调用函数的循环

do{
    try{
        date = inputDate();
        validDate = true;
    }
    catch(runtime_error& e){
        cout<<"Date format not valid! must input as dd/mm/yyyy!"<<endl;
        validDate = false;
    }
}while(!validDate);

//customdate constructor
CustomDate::CustomDate()
    :day(1),month(1),year(2012){}

CustomDate::CustomDate(int day, int month, int year)
    :day(day),month(month),year(year){

    if(day<0 || day>30)
        error("Error: Date constructor");
    if(month<0 || month>12)
        error("Error: Date constructor");
    if(year<0)
        error("Error: Date constructor");
}

【问题讨论】：

不要尝试缩进添加了`的代码段，只需使用代码块按钮。
你能显示你的重载运算符>>而不是你的重载运算符
你输入的是什么？
他在出现错误时清除错误状态，这也有点令人惊讶。所以任何使用>> 运算符的人都会大吃一惊。
@pippin1289 不是变量，是操纵函数。真正的问题是他为什么要使用它，因为>> 默认会跳过前导空格。

标签： c++ date overloading infinite istream

【解决方案1】：

正如我在评论中所说：

“clear() 函数应该清除流”是什么意思？它不会丢弃流内容，因此如果流中有垃圾（例如无法解析为 int 的字符 'a'），它将永远不会“清除”该垃圾，只会继续重试。我认为问题在于 clear 并没有像你认为的那样做。

如果它无法提取整数或分隔符错误，则不要从流提取运算符中抛出异常（也可以尝试使用更多空格来帮助使代码更具可读性）：

istream& operator >>(istream& is, CustomDate& d){
    int day, month, year;
    char ch1, ch2;
    if (is >> day >> ch1 >> month >> ch2 >> year)
    {
        if (ch1 == '/' && ch2 == '/')
            d = CustomDate(day, month, year);
        else
            is.setstate(ios::failbit);
    }
    return is;
}

然后在inputDate处理失败的提取

CustomDate inputDate(){
    CustomDate date;
    cout << "Input your departure date" << endl;
    if (cin >> date)
      return date;

    // something went wrong
    if (cin.eof())
        error("No more data in stream");
    else // cin.fail() is true, probably invalid date format
    {
        // clear error and discard input up to next newline
        cin.clear();
        cin.ignore(numeric_limits<streamsize>::max(), '\n');
        error("invalid date format");
    }
}

【讨论】：

这可能有效，但它给了我一个编译错误 C2065 'numeri_limits' : undeclared identifier
is.clear(ios_base::failbit) 不会清除流的failbit。它设置failbit 并清除badbit 和failbit。

【解决方案2】：

您的代码有许多错误。 do ... try ... catch 循环可能是一个无限循环，它也可能不正确。

如果您在输入流上启用了异常并且流提取语句中的解析在重载的operator>> 中失败，您将获得一个无限循环。您的代码永远不会重置流，因此一旦出现解析错误，您的代码就会陷入循环。永远。

如果您没有启用异常并且输入被破坏到足以使流提取语句以某种方式将流标记为“坏”怎么办？未启用异常，因此不会引发异常。您的代码（如果在流提取调用之后立即执行 if 语句，则为 then 分支）将执行。这也不会引发异常。 do ... try ... catch 将成功。在这里，您的代码错误地将错误的输入视为有效输入。

【讨论】：

【解决方案3】：

我会修改您的示例，因为它依赖于能够输入多个项目，即 int char int char int。我会输入一个字符串。然后使用字符串流解析日期并检查其格式。

所以它可能是这样的：

istream& operator >>(istream& is,CustomDate& d){
    string dateStr;
    is>>dateStr;
    istringstream iss(dateStr);
    int day,month,year; 
    char ch1,ch2;
    if(!(iss>>day>>ch1>>month>>ch2>>year)){
        error("invalid date input");
        return is;
    }
    if(ch1!='/' || ch2!='/'){
        error("invalid date format use m/d/y");
        return is;
    }
    d = CustomDate(day,month,year);
    return is;
}

我不确定这是否会修复任何无限循环，但它可能是检查输入的更好方法。

【讨论】：