c++ - 检查错误时重载 >> 运算符中的无限循环

Question

std::istream & operator >>(std::istream & ins, Rational & target)
{
    int num, den;
    char symb;          
    std::cout << "Please enter a rational number: ";
    ins >> num >> symb >> den;
    std::cout << std::endl;

    if(validateInput(num, symb, den)){
        target = Rational(num, den);            
        return ins;
    }
    else{
        std::cin >> target;
    }
}

bool validateInput(int num, char symb, int den)
{
    if(symb != '/'){
        std::cout << "Error: Illegal format. Please use '2/4'." << std::endl;
        return false;
    }
    if((static_cast<int>(num) != num) && (static_cast<int>(den) != den)){
        std::cout << "Error: Not a valid rational number." << std::endl;
        return false;
    }
    if(den == 0){
        std::cout << "Error: Cannot divide by 0." << std::endl;
        return false;
    }

    return true;
}

它接受格式为“x/y”的有理数，例如 2/4。 如果我输入正确，它工作正常。如果我输入 2p4，它会给出正确的错误(我遗漏了一个“/”)，然后要求输入一个新号码。如果分母是0，也会报错，要求重新取数。

但检查它是否为有效数字似乎不起作用。如果我输入“a/4”，它将无限循环直到崩溃。我不知道为什么。检查调试器，它返回到 ins >> 语句，但不要求用户提供任何信息。

我假设我的逻辑在某处是错误的。请注意，我对 C++ 还很陌生，还在学习中。我之前尝试过异常处理，但仍然没有正确学习，所以我又选择了我更熟悉的东西。

谢谢!

Answer 1

问题的基本要点是，如果流的状态变坏，C++ 流格式化提取运算符将停止工作，您必须重置状态才能使它们再次工作。

您还有其他问题。

首先，您的验证功能表明您缺乏经验:static_cast<int>(intval) == intval将永远是真实的并且不会证实任何事情。其次，您无法验证您确实已经成功地从流中提取值(这是无限循环的原因:您所做的就是一遍又一遍地验证失败。)

因此，当您提取值时，您应该验证一切正常，如下所示:

int num, den;
char symb;
// Remember to flush unfinished lines
std::cout << "Please enter a rational number: " << std::flush; 
if (std::cin >> num >> symb >> den)
    // you extracted an integer, a character and an integer succesfully
    // perhaps check that the character is '/' and denominator is non-zero
else
    // there was an error: what should we do?

“我们应该做什么”部分远非显而易见:您可以只是重置流并从中删除第一个有问题的字节，然后重试，如果您认为这是合理的(并且直观的) .然而，由于数字太大，提取也可能失败，在这种情况下，这可能会导致奇怪的行为:考虑以下输入(在具有常见 long 大小的实现上):

3111111111111111111111111111111111111111111111111/3

如果您没有任何实际规范，需要考虑的是面向行的输入:首先读取一行并尝试解析它；如果它看起来不正常，请忽略它并尝试下一个。