陷入无限 do while 循环

Question

我正在尝试用 C++ 进行简单的猜测我的数字游戏，但计算机需要猜测我的数字。但问题是我陷入了这个无限循环。我只是一个初学者，所以这是一个非常基本的程序。

这是我的代码：

int secretNumber = rand() %100 + 1; // random number between 1-100
int tries=0;
int input;

cout <<"typ your number\n";
cin >> input;

do
{
    cout <<secretNumber <<endl;
    ++tries;

    if (secretNumber > input)
    {
        cout <<"To high i guess?\n";
    }
    else if (secretNumber < input)
    {
        cout <<"To low I guess?\n";
    }
    else 
    {
        cout <<"Yes, i got it in " <<tries <<" tries!";
    }
}while (input != secretNumber);

return 0;

}

Answer 1

地点cin>>输入到循环体中

Answer 2

我认为你应该将随机数生成移到循环内。

Answer 3

变量 input 的值在循环中永远不会改变，因此终止条件 input != SecretNumber 永远不会满足。

您应该在循环内获取输入。所以写 cin >>>在循环开始处输入。

编辑：

如果计算机应该猜测，那么仍然需要在循环中更改input的值，这在您的代码中不存在。循环每次都使用 input 中的相同值运行。

为了让你的计算机做出猜测，你应该遵循一些方案。计算机可能会随机抽取数字 - 您可以通过在循环内移动 secretNumber = rand()%100 + 1 来获得这些数字。但这种方法可能性能不佳，循环可能仍会运行很长时间。这在 @Kaii 的回答中有所体现。

更有效的方法是二分搜索。在这种情况下，您应该记录计算机所做的猜测。保留两个变量 high 和 low，它们应分别存储高于和低于 input 的猜测值。每当猜测值高于该数字时，请将其存储在 high 中，并将任何低于 input 的猜测值存储在 low 中。然后计算机应该尝试在 high 和 low 之间进行新的猜测。随机猜测应该是 secretNumber = low + rand() % (high - low)。在最坏的情况下，将需要多达 100 次迭代。为了获得最佳结果，每次猜测应该是(high + low) / 2。根据条件，每次迭代都会更新high和low之一。这种方法将确保计算机在 7 次猜测之内猜出正确的数字。

在你的代码中应该是这样的：

int secretNumber = rand() % 100 + 1; // random number between 1-100
int tries=0;
int input;
int low = 1, high = 100;

cout <<"typ your number\n";
cin >> input;

do
{
    secretNumber = (high + low) / 2;
    cout << secretNumber <<endl;
    ++tries;

    if (secretNumber > input)
    {
        cout << "Too high I guess?\n";
        high = secretNumber;
    }

    else if (secretNumber < input)
    {
        cout << "Too low I guess?\n";
        low = secretNumber;
    }

    else 
    {
        cout << "Yes, i got it in " << tries << " tries!";
    }        
} while (input != secretNumber);

return 0;

Answer 4

计算机仅在程序启动时（随机）猜测一次，但每次迭代循环时都应该猜测。您应该在循环内移动随机数生成：

int secretNumber = 0;
int tries=0;
int input;

cout <<"typ your number\n";
cin >> input;

do
{
    /* the fix is here */
    secretNumber = rand() %100 + 1; // random number between 1-100
    cout <<secretNumber <<endl;
    ++tries;

    if (secretNumber > input)
    {
        cout <<"To high i guess?\n";
    }
    else if (secretNumber < input)
    {
        cout <<"To low I guess?\n";
    }
    else 
    {
        cout <<"Yes, i got it in " <<tries <<" tries!";
    }
}while (input != secretNumber);

return 0;

Answer 5

您必须在循环内进行输入。现在，您在开始循环之前提示输入一个值，然后永远不要向用户询问另一个数字。如果无法改变猜测，则循环永远不会退出。

更改为

do {
    cin >> input;

将解决无限循环。

Answer 6

代码中的无限循环，因为 while 的参数始终为 true，直到获得真实数字为止。

你需要添加另一个参数。例如，您希望将最大尝试次数设置为 5 次。那么就会这样，

do{
    //do your stuff here
}while((input != secretNumber)&&(tries<=5))

当你输入错误的号码5次，申请就结束了