C++ 随机数猜谜游戏

Question

我必须编写一个运行随机猜谜游戏的程序。游戏是从 1 到 100 的数字，猜者有 20 次尝试，最后应该被问到是否想再玩一次。如果猜测器是高或低，还必须有多个打印输出选项。我已经完成了我的程序的一部分我知道我仍然需要为打印输出添加其他选项但是现在我的问题是当我尝试运行我所拥有的内容时它说成功但是然后有一个错误说变量“ number" 正在使用而没有被初始化。我不知道该怎么做才能让它初始化。 （我是 C++ 新手） 我已经更新了我的程序，但现在遇到了不同的问题。我的程序运行，但如果猜测低于它打印的数字太高再试一次，太低再试一次，但是当数字太高时，它只会打印太高再试一次。我还注意到，当用户选择再次玩游戏时，尝试次数计数器不会随着游戏重置。最后一件事我必须添加更多消息，说明他们何时赢、输并被要求玩另一场比赛，我必须使用随机数在其中进行选择。所以任何关于最佳路线的建议都会很棒

#include <iostream>
#include <iomanip> 
#include <ctime>
using namespace std;
char chr;

int main()
{
srand(time(NULL));                                           //the function that generates random numbers
int number=rand()%100+1;                                     //the range of the random numbers
int guess;                                                   //The guess is stored here
int tries=0;                                                 //The number of tries stored here
   char answer;                                                 //The answer to the question is stored here
answer='y';                  

while(answer=='y'||answer=='Y') 
{
    while (tries<=20 && answer=='y'|| answer=='Y')
    {
    cout<<"Enter a number between 1 and 100 "<<endl;          //The user is asked for a guess
    cin>>guess;                                               //The guess is stored here
    tries++;                                                 //Adding a number for every try
    if(guess==0||guess>100)                                  //If statement that produces an error message if user enters a number out of the peramiters
    {
     cout<<"This is not an option try again"<<endl;          //Error message
    }

    if(tries<20)                                            
    cout<<"Tries left: "<<(20-tries)<<endl;                   //Output to let the user know how many guess they have left

    if(number<guess);                                         //if the guess is to high
    cout<<"Too high try again"<<endl;                         //This message prints if guess it to high

    if(number>guess)                                          //if the guess is to low
    cout<<"Too low try again"<<endl;                          //This message prints if the guess is to low

    if(number==guess)                                          //If the user guesses the number
    {
     cout<<"Congratualtions!! "<<endl;                          //Message printed out if the user guesses correctly
     cout<<"You got the right number in "<<tries<<" tries"<<endl;  //Lets the user know how many guess they used
     answer = 'n';
    }
    if(tries >= 20)                                               //If the user uses all their guesses
    {
    cout << "You've run out of tries!"<<endl;                      //The message that prints when a user is out of guesses
    answer='n';
    }
    if(answer=='n')
    {
     cout<<"Would you like to play again?  Enter Y/N"<<endl;       //asking if play would like to play again
     cin>>answer;                                                  //Store users answer
     if (answer=='N'|| answer=='n')                                //if the user says no
     cout<<"Thanks for playing!"<<endl;                            //This message prints out if they choose not to play again

    else
        number=rand()%100+1;                                        //This starts the game over if they choose to play again
    }

    }
    }

cin>>chr;
    return 0;

}

Answer 1

[编辑：添加演员表以消除编译器警告。]

正如 Jim Rhodes 在评论中所说，问题出在这条线

srand(number>0);

srand() 用于初始化随机数生成器，因此用number>0 调用它甚至用number 调用它都没有意义。它需要一个“种子”值，每次运行程序时都应该不同。获取此类种子的常用方法是使用系统时间：

srand(static_cast<unsigned int>(time(NULL)));

您可能需要#include 另一个标头才能访问time()。

Answer 2

编译器警告您的问题在于这两行：

int number;
//...
srand(number>0);

这里，你没有给变量number 一个初始值，所以它是未初始化的——你完全无法知道此时的值可能是什么。事实上，每次运行程序时它都可能发生变化。接下来你要问神秘值是否大于零——可能是，也可能不是，但你只是不知道。这是一种神秘的行为，这是编译器警告你的。

当然，现在您正在尝试初始化随机种子，因此您可能会追求这种神秘的行为！但不幸的是，即使这样也行不通。

在 C++ 中，表达式number>0 是布尔值，即表达式的结果是true 或false。但是srand() 函数将unsigned int 作为它的参数，因此编译器必须将bool 转换为unsigned，它通过将false 更改为0 和true 来实现1（可能：我相信这在技术上取决于实现）。所以无论number 的初始值是多少，你的随机种子只会有两个可能的值之一——根本不是很随机！

最好使用基于（相对）不可预测的随机种子。非加密需要根据当前时间初始化种子是很常见的。比如：

#include <ctime>

std::time_t now = std::time(0);
srand(static_cast<unsigned>(now));

会好的。

Answer 3

我看到的一个问题是，您要为每个用户猜测选择一个新的随机数。这是错误的。

要解决这个问题，你应该把

number=rand()%100+1;

在循环询问猜测的 do while 循环之前的行。

第二个问题是该循环的条件不正确。您应该循环直到 number == guess not number > guess 并将两个 cout 告诉用户猜测是高还是低在循环内，并在循环内增加您的尝试。

此外，您可能希望为要求您再次播放的问题设置一个外部 do while() 循环，而不是在等待用户获得正确数字的循环之后。

Answer 4

在使用它们之前初始化你的变量（总是！！），否则你会在使用这些的任何操作中调用未定义的行为（甚至是编译器错误）！

int number = 0;
int guess = 0;
int tries = 0;
char answer = '\0';

注意（针对反对者、质疑者）：
当然这个答案并没有说明，如何让 srand(number>0); 语句正确获得所需的结果（即使用比 0 更合适的值初始化 number），但它解决了首先要求的编译时错误位置，对于 任何 其他情况是很好的建议，导致类似的编译器错误（警告）。使用适当的值初始化 number 以传递给 srand() 种子方法以在运行时获得预期的正确结果，这是一个完全不同的问题，应该这样问。

Answer 5

我可以建议使用 gettimeofday 函数来为您的随机函数提供种子吗？

#include <stdio.h>
#include <stdlib.h>
#include <sys/time.h>
//clock_t times(struct tms *buffer);
int
main(int argc, char **argv)
{
    int a;
    struct timeval _wall;
    gettimeofday( &_wall, NULL ); //(_wall.tv_sec,_wall.tv_usec);
    srand(_wall.tv_usec);
    a = rand() % 100 + 1;
    printf("%d\n",a);
    return 0;
}