1

好的，所以我一直在研究我的计算器。我目前正试图让它分辨有效整数和字符之间的区别。作为一个简单的解决方法，我做了：

int calc()
{
cout << "Number 1:"; cin >> fnum;
cout << "Number 2:"; cin >> snum;
    if (snum <= -1000 || fnum <= -1000)
    {
        cout << ("Error: Invalid Number!") << endl;
        calc();
    }
    else
    {
        ff();
    }
return 0;
}

每当我输入一个字符时，它就会进入一个无限循环，说： SymbolHere:Number 1:Number 2:

ff(); 正在调用计算函数。

我想知道如何解决这个问题并防止堆栈溢出/无限循环？Pastebin 链接：http://pastebin.com/GxN2uJAQ

4

1 回答 1

0

编辑：好的，这段代码有很多东西。

wait = 0;
while (wait <= 5)
{
   wait++;
}

绝对不会做任何事情，你的程序会增长得如此之快，这是人类无法察觉的。我建议完全删除它。

if (snum >= 0 || fnum >= 0)
                {
    cout << ("Error: Invalid Number!") << endl;
    wait = 0;
    while (wait <= 5)
    {
       wait++;
    }
    system("CLS");
    calc();
    }

如果这些数字有效，为什么要发送错误消息？除非您只添加负数，否则这应该有不同的范围。

您的函数调用也永远不会解析回 main，而是相互调用（ff和calc）无穷大，该程序确实有太多缺陷和糟糕的编程实践。放弃您拥有的任何教程/书籍，并尝试找到一个更新的列表（对不起，这很苛刻，但必须说）。

C++ 入门加

CPlusPlus.com

TheNewBoston（推荐）

您可能通过输入一个 char 表示 int 输入失败，您需要确保捕获 cin 抛出的任何内容并清除状态：

std::string err = "error!";
try {
    std::cin >> x;
    if(!cin)
        throw err;
    //....
}
catch(std::string& ee)
{
    std::cout << ee << std::endl;
    cin.clear();
    cin.ignore(numeric_limits<streamsize>::max(),'\n');
}

确保包含<limits>在您的文件中。

于 2013-05-21T02:53:27.807 回答