Question

我的代码用于猜谜游戏，用户猜测1-100之间的数字。他们有10次尝试，每次猜测后都告诉他们是否太低或太高等等。

我无法解决的一个问题是，在他们赢了之后，告诉他们他们赢了以及他们赢了多少猜测（使用guesses变量）。我的问题是，如果他们赢了，他们又玩了，猜测就不会重置。因此，如果在第一场比赛中需要5次尝试，第二场比赛需要8次尝试，在第二场比赛中，它会告诉他们他们在13次猜测中获胜（从第一场比赛中获胜5次，从第二场比赛中获胜8次）。

我知道它为什么会发生，但我尝试的一切似乎并没有解决它。我假设我只是在我guesses = 0;的同一地点执行attemptsLeft = 10;这样的事情，因为attemptsLeft工作正常，但它会起作用，但它总是说他们赢了并且花了它他们猜0。我还尝试在guesses = 0;语句中设置if，以确定它们输赢的时间，但是它做了同样的事情，并说它们在0猜测中获胜。

GuessingGame类的代码

namespace NumberGuessingGame
{
    public class GuessingGame
    {

        int myGuess = 0;
        int guessesLeft = 10;
        public int gamesPlayed = 0;
        public int gamesWon = 0;
        public int gamesLost = 0;
        public int attemptsLeft = 10;
        public int guesses = 0;

        Random rand;
        int number = 0;

        public GuessingGame()
        {           
            attemptsLeft = 10;
            rand = new Random();
            number = rand.Next(1, 100);            
        }
        public void ResetGame()
        {            
            attemptsLeft = 10;
            number = rand.Next(1, 100);
            guessesLeft = 10;            
        }                
        public int CheckGuess(int newGuess)
        {
            myGuess = newGuess;

            if (guessesLeft < 1)
            {               
                gamesLost++;
                gamesPlayed++;
                ResetGame();                
                return 2;                
            }
            else if (myGuess > number)
            {
                attemptsLeft--;
                guesses++;
                guessesLeft--;
                return 1;
            }
            else if (myGuess < number)
            {                               
                attemptsLeft--;
                guesses++;
                guessesLeft--;
                return -1;
            }
            else
            {               
                guesses++;
                gamesWon++;
                gamesPlayed++;
                ResetGame();                
                return 0;
            }
        }
    }
}

表单类的代码

namespace NumberGuessingGame
{
    public partial class frmMain : Form
    {
        public frmMain()
        {
            InitializeComponent();           
        }        

        GuessingGame myGuess = new GuessingGame();

        private void btnCheck_Click(object sender, EventArgs e)
        {
            int inputGuess = Convert.ToInt32(txtGuess.Text);
            int result = myGuess.CheckGuess(inputGuess);                                           

            if(result == 2)
            {
                lblLosses.Text = "Losses: " + myGuess.gamesLost;
                lblPlayed.Text = "Games Played: " + myGuess.gamesPlayed;
                lblWins.Text = "Wins: " + myGuess.gamesWon;                                       
                MessageBox.Show("You ran out of guesses and lost!");                
            }
            else if(result == 1)
            {               
                MessageBox.Show("Your guess was too high, try again!" + "\n You have " + myGuess.attemptsLeft + " guesses left");
            }
            else if(result == -1)
            {
                MessageBox.Show("Your guess was too low, try again!" + "\n You have " + myGuess.attemptsLeft + " guesses left");
            }
            else if(result == 0)
            {                
                MessageBox.Show("You won!" + "\n It took you " + myGuess.guesses + " guesses");                
                lblLosses.Text = "Losses: " + myGuess.gamesLost;
                lblPlayed.Text = "Games Played: " + myGuess.gamesPlayed;
                lblWins.Text = "Wins: " + myGuess.gamesWon;    
            }                       
        }

Answer 1

我想我有点困惑为什么你增加两个猜测和尝试的数量。难道你不能只使用任何一个逻辑来完成游戏的流程，而不是跟踪两个本质上意味着相同的东西的东西吗？当你重新玩游戏时，看看每个变量的等价，你是否正确地重置每个变量并调用该函数？祝你好运。

Answer 2

你永远不会重置guesses值，所以当然它会不断增加。您希望在任何一轮的最终猜测之后保留猜测计数，以便您可以显示它，然后在下一轮开始时重置它。

目前，您的代码都没有跟踪回合的状态，因此需要修复。您还使用两个不同的变量来跟踪所做的和剩余的猜测次数，这是过多的。我们可以将其切换回单个变量，以跟踪在一轮开始时重置的猜测。

所以，从游戏状态开始。我们可以将这些物品打包成这样的状态：

public enum EGameState
{
    None,
    Playing,
    Won,
    Lost
}

None状态将用于第一轮，之后不会用。在玩圆形状态时，状态为Playing，在该回合结束时Won或Loss表示结果。

现在可以修改您的GuessingGame类以使用状态来确定如何对猜测作出反应等。如果游戏状态未设置为Playing，那么您需要重新初始化圆形变量。

为什么我的价值在游戏结束后没有重置

GuessingGame类的代码

表单类的代码

2 个答案: