C# 范围问题，我确定可以简单修复

Question

所以我有以下代码，它的工作几乎完美无缺，除了以下事实：无论我做什么我都无法得到类似的部分： for (int.parse(txtGuess.Text) == numbGen)< /code> 无论我将“numbGen”放在代码中的哪个位置，它都不会识别它。我无法将其放置在按钮单击功能中，因为我不希望数字发生更改，除非他们得到正确的结果或重新打开表单。

    using System;
    using System.Collections.Generic;
    using System.ComponentModel;
    using System.Data;
    using System.Drawing;
    using System.Text;
    using System.Windows.Forms;


    namespace WindowsApplication1
    {
        public partial class rndNum : Form
        {
            public rndNum()
            {
            }

            private void rndNum_Load(object sender, EventArgs e)
            {
                int numbGen = RandMake(0, 100);
            }

            private void txtGuess_TextChanged(object sender, EventArgs e)
            {

            }

            private void btnEval_Click(object sender, EventArgs e)
            {
                int guesses = 0;
                while (txtGuess.Text != "")
                {
                    if (int.Parse(txtGuess.Text) == numbGen)
                    {
                        MessageBox.Show("You got it!", "Congratulations!");
                        break;
                    }
                    else if (int.Parse(txtGuess.Text) > numbGen)
                    {
                        MessageBox.Show("Sorry, but you're too high. The number was " + numbGen + "!", "Please try again.");
                        txtGuess.Clear();
                        txtGuess.Focus();
                        guesses++;
                        break;
                    }
                    else if (int.Parse(txtGuess.Text) < numbGen)
                    {
                        MessageBox.Show("Sorry, but you're too low. The number was " + numbGen + "!", "Please try again.");
                        txtGuess.Clear();
                        txtGuess.Focus();
                        guesses++;
                        break;
                    }
                }
            }



            private static int RandMake(int min, int max)
            {
                Random mkRnd = new Random();
                return mkRnd.Next(min, max);
            }
}

}

Answer 1

numbGen 必须是类成员。

改为

        private void rndNum_Load(object sender, EventArgs e)
        {
            int numbGen = RandMake(0, 100);
        }

实际上

        private int numbGen;
        private void rndNum_Load(object sender, EventArgs e)
        {
            numbGen = RandMake(0, 100);
        }

不需要将初始化放在Form.Load中，可以直接初始化类成员。

    public partial class rndNum : Form
    {
        private int numbGen = RandMake(0, 100);
        public rndNum()
        {
        }

并进一步细化：如果您想确保值没有更改，您可以将其设置为只读

    public partial class rndNum : Form
    {
        private readonly int numbGen = RandMake(0, 100);
        public rndNum()
        {
        }

Answer 2

int numbGen;
private void rndNum_Load(object sender, EventArgs e)
{
    numbGen = RandMake(0, 100);
}

试试这个！

Answer 3

其他人已经对您注意到的范围问题发表了评论。但你的 RandMake 方法也有缺陷。您不应为每个数字创建一个 Random 实例，而应重用该实例。

这里的问题是 new Random() 使用时间作为种子，并且时间仅每隔几毫秒改变一次。这意味着，如果您在该时间间隔内多次调用 RandMake，您将获得相同的“随机”数字。

这似乎不是一个迫在眉睫的问题，因为您只调用它一次，但您将来应该意识到这一点。

Answer 4

当您将 numbGen 的声明放在方法内部时，该数字仅存在于该函数范围内。你必须像这样把它放在外面：
<代码>

        int numbGen;
        private void rndNum_Load(object sender, EventArgs e)
        {
            numbGen = RandMake(0, 100);
        }

<代码>