使用sstream失败进行验证

时间:2016-04-21 11:08:34

标签: c++ validation integer sstream

cout << "\nPlease enter x-ordinate: ";

cin >> test;
stringstream ss(test);
ss >> x;
while(ss.fail())
{
ss.str("");
cin.clear();
cin.ignore(256, '\n');  
cout << "Please enter integer: \n";
cin >> test;
stringstream ss(test);
ss >> x;
cout << x << ss;    
}       

嗨,我试图测试用户的输入是否是整数,通过使用sstream检查它是否可以转换为整数x,但是一旦我输入一个有效的整数,while循环仍然证明是真的,循环继续。可以帮忙吗?

3 个答案:

答案 0 :(得分:1)

你的代码存在轻微问题......你有两个完全不同的变量同名,一个用于循环条件,一个在循环体内。这两个变量没有任何关联或联系。

而是通过设置其字符串:

来重用第一个ss变量
...
cin >> test;
ss.str(test);
...

您也不能清除循环中ss流的状态,只有cin,这就是您的循环永远不会结束的原因。

我还建议您使用流可以用作布尔条件的事实,并且operator>>函数返回对流的引用,然后您可以执行类似

的操作
cout << "\nPlease enter x-ordinate: ";
getline(cin, test);

istringstream ss(test);
while(!(ss >> x))
{
    ss.clear();
    cout << "Please enter integer: \n";
    getline(cin, test);
    ss.str(test);
}

请参阅here for an example of the above

当然也可以直接使用cin

int x;
while (!(cin >> x))
{
    cin.clear();
    cin.ignore(numeric_limits<std::streamsize>::max(), '\n');
    cout << "Please enter integer: \n";
}

请参阅here for an example of the above

答案 1 :(得分:0)

测试变量是什么类型的?一个字符串? 您可以尝试将字符串作为输入的通用解析器:

template<class T>
bool tryparse(const string word, T& res)
{
   istringstream parser(word);
   bool status = (parser >> res);
   return status;
}

答案 2 :(得分:0)

cin流式传输到string,创建istringstream,然后从那里流式传输到您的号码是没有意义的:它只是为其他类型的错误留出空间,例如"12x"被视为12x被忽略,而不是为用户生成警告/错误消息。

您可以直接阅读以下内容:

const char* prompt = "\nPlease enter x-coordinate: ";
while (cout << prompt && !(cin >> x))
{
    cin.clear();
    cin.ignore(256, '\n');  
    prompt = "Please enter integer: \n";
}

如果您不需要更改提示,可以将其直接放入循环中:

while (cout << "\nPlease enter x-coordinate: " && !(cin >> x))
{
    cin.clear();
    cin.ignore(256, '\n');  
    std::out << "Please enter an integer.\n";
}

注意:您会看到类似于您的第一次努力的内容,string的阅读使用getline而不是>>,这可能是一个很好的做法,因为它让您计算行号(对于解析文件比std::cin更有用),并且检查值与其他预期数据一起显示在行上。看起来像这样:

std::string line;
while (std::cout << "Enter coordinate:" &&
       getline(std::cin, line))
{
    std::istringstream iss(line);
    if (iss >> x)
         break;
    std::cout << "Error - invalid coordinate, please enter an integer\n";
}
...use x...