0

我必须编写一个运行随机猜谜游戏的程序。游戏是从 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;

}
4

5 回答 5

8

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

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

srand(number>0);

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

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

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

于 2013-10-29T02:22:33.330 回答
3

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

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

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

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

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

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

#include <ctime>

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

会好的。

于 2013-10-29T03:03:37.817 回答
2

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

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

number=rand()%100+1;

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

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

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

于 2013-10-29T03:10:45.873 回答
1

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

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

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

于 2013-10-29T02:21:28.483 回答
0

我可以建议使用 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;
}
于 2013-10-29T02:45:48.747 回答