整数输入验证,怎么样?

时间:2012-11-03 18:12:25

标签: c++ error-handling integer infinite-loop

我遇到了一个非常简单的代码问题。我想接受1到3之间的整数,并进行错误检查。它适用于检查太大或太小的数字,但是当输入字母/数字组合时,它会陷入无限循环。建议?

#include <iostream>
using namespace std;

int main(int argc, char *argv[]){
    int input;

    cout << "\nPlease enter a number from 1 to 3:" << endl;
    cout << "-> ";
    cin >> input;

    while(input< 1 || input> 3){
        cout << "\n---------------------------------------" << endl;
        cout << "\n[!] The number you entered was invalid." << endl;
        cout << "\nPlease re-enter a number from 1 to 3" << endl;
        cout << "-> ";
        cin >> input;
    }

    cout << "You chose " << input << endl;
}

4 个答案:

答案 0 :(得分:7)

问题在于:

cin >> input;

当您尝试读取非数字值时,将导致设置坏位。在那之后,任何使用operator>>的尝试都会被忽略。

因此,纠正此问题的方法是测试流是否处于良好状态,如果没有,则重置状态标志并再次尝试读取。但请注意,输入错误(导致问题)仍然在输入上,因此您需要确保将其丢弃。

if (cin >> input)
{
    // It worked (input is now in a good state)
}
else
{
    // input is in a bad state.
    // So first clear the state.
    cin.clear();

    // Now you must get rid of the bad input.
    // Personally I would just ignore the rest of the line
    cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

    // now that you have reset the stream you can go back and try and read again.
}

为了防止它被卡住(这是由设置的错误位引起的)读入字符串然后使用字符串流来解析用户输入。我也更喜欢这种方法(用于用户交互式输入),因为它允许更容易地组合不同类型的阅读(即组合operator>>std::getline(),因为您可以在字符串流上使用这些)。

#include <iostream>
#include <sstream>
#include <string>
// using namespace std;
// Try to stop using this.
// For anything other than a toy program it becomes a problem.

int main(int argc, char *argv[])
{
    int          input;

    std::string  line;
    while(std::getline(std::cin, line))   // read a line at a time for parsing.
    {
        std::stringstream linestream(line);
        if (!(linestream >> input))
        {
             // input was not a number
             // Error message and try again
             continue;
        }
        if ((input < 1) || (input > 3))
        {
             // Error out of range
             // Message and try again
             continue;
        }
        char errorTest;
        if (linestream >> errorTest)
        {
             // There was extra stuff on the same line.
             // ie sobody typed 2x<enter>
             // Error Message;
             continue;
        }

        // it worked perfectly.
        // The value is now in input.
        // So break out of the loop. 
        break;
    }
}

答案 1 :(得分:2)

#include <iostream>
#include <string>

using namespace std;

int validatedInput(int min = 1, int max = 3)
{
    while(true)
    {
        cout << "Enter a number: ";
        string s;
        getline(cin,s);
        char *endp = 0;
        int ret = strtol(s.c_str(),&endp,10);
        if(endp!=s.c_str() && !*endp && ret >= min && ret <= max)
            return ret;
        cout << "Invalid input. Allowed range: " << min << "-" << max <<endl;
    }
}

int main(int argc, char *argv[])
{
    int val = validatedInput();
    cout << "You entered " << val <<endl;
    return 0;
}

答案 2 :(得分:1)

这些答案大多包括不必要的复杂性。

输入验证是使用do-while

的最佳时机
do{

   cout << "\nPlease enter a number from 1 to 3:" << endl;
   cout << "-> ";

   if(!cin){
      cout << "Invalid input"
      cin.clear()
      cin.ignore(numeric_limits<streamsize>::max(), '\n');
   }

}while(!(cin >> input))
  • 使用numeric_limits<streamsize>::max()完全清除 失败后cin缓冲。

  • 使用cin.clear()重置cin上的失败标记,以便!cin不会 总是评价错误。

cin.fail()没问题。但有些人会认为!cin更自然。

来自我之前的帖子https://stackoverflow.com/a/43421325/5890809

答案 3 :(得分:0)

您将输入声明为int但是当您输入字母数字字符进行输入时,它会尝试将输入隐式转换为整数。但是您的错误检查并未考虑到这一点。

通过更改while循环可以轻松解决Ur问题。而不是检查你如何检查

while(input!=1 || input!=2 || input!=3)