c#数组中的随机数== winNumber [关闭]

Question

所以我是编码初学者（1 个月）。 我已经更改代码 4 天了，没有运气。 我坚持让我的 randomNumber == winNum 中断。 如果用户写入，用户将获得 10 个数字（假设 winNum 为 9） 1,2,5,9,10,11.... 等等。代码不会选择数组中的数字，并打印“你幸运”。 该代码用于工作。 9.2.3.4.5 ......等等，它得到“你很幸运”，但不再是。 我也得到“你输了”“你输了”“你输了”和“游戏结束”。

不确定我做错了什么。

    static void Main(string[] args)
    {
        // -int size = Convert.ToInt32(Console.In.ReadLine());   // this to change amount of variables the user wants instead of [x]
        int[] numbers = new int[10];  //  task 1, modul 3. The 10 in the array is a statment for how many varibles this array can hold.
                                     // you can have as many varibles as you want.
        Random randomNumbers = new Random();
        int winNum = randomNumbers.Next(1, 26);
        Console.WriteLine(winNum);
        Console.WriteLine("Pick your 10 numbers! "); // här får vi våra 10 tal av användaren 
            for (int hallon = 0; hallon < 10; hallon++)  //this loop will print the users numbers
            {
                
                Console.WriteLine($"Your have used: {hallon}" ); // här får vi våra 10 tal av användaren 
            try
            {
                numbers[hallon] = Convert.ToInt32(Console.In.ReadLine());  // numbers lagrar våra 10 tal som kommer spelas nu mot winNum
            }
            catch (SystemException e) // going to catch if the user writes in a letter instead of a number        
            {
                --hallon;
                Console.WriteLine(e.Message);
            }
            if (numbers[hallon] < 1 || numbers[hallon] > 25)
                {
                    Console.WriteLine("Write a number within 1-25. ");
                    --hallon;
                }
            } 
        for (int hallon = 0; hallon < 10; hallon++)
        {
            if (numbers[hallon] == winNum)  // vad gör jag om jag vill ha två vinnande resultat? 
            {
                Console.WriteLine("Lucky number:" + numbers[hallon]);
                break;
            } ////task 2 modul 3 this loops my array untill i have all varbiles printed 
        }

        for (int hallon = 0; hallon < 10; hallon++)
        {
            if (numbers[hallon] != winNum)
            Console.WriteLine("You lost");

        }
        Console.WriteLine("End of the game");
        Console.ReadLine();
    }
    }

Answer 1

我认为您在这里遇到的问题是“break”关键字只超出了它自己的 for 循环。你有一个 for 循环来查看中奖号码是否在猜测中，如果是，则打印“幸运号码” - 但随后它会再次遍历所有猜测并打印“你输了”。因此，即使我猜对了 10 条中的一条，我也希望看到“幸运数字”，然后是 9 条“你输了”。

解决此问题的一种方法是设置一个“赢得”布尔变量，如果您找到中奖号码，则该变量设置为 true，并且仅在“幸运号码”for 循环后为 false 时打印“您输了”。

一些额外的想法：

• 您的“Catch”语句在 if (numbers[hallon] < 1 || numbers[hallon] > 25) 行上的hallon 变量可能为-1，我认为这会导致错误。建议您在 catch 块中使用continue，这意味着它会再次进入循环顶部。 （要尝试此操作，请输入一个字母作为您的第一个回复）
• 而不是numbers，guesses 可能是变量的更清晰的名称。

Answer 2

我认为与其遍历每个数字来找到结果，不如执行以下操作：

    static void Main(string[] args)
    {
        // -int size = Convert.ToInt32(Console.In.ReadLine());   // this to change amount of variables the user wants instead of [x]
        int[] numbers = new int[10];  //  task 1, modul 3. The 10 in the array is a statment for how many varibles this array can hold.
                                        // you can have as many varibles as you want.
        Random randomNumbers = new Random();
        int winNum = randomNumbers.Next(1, 26);
        Console.WriteLine(winNum);
        Console.WriteLine("Pick your 10 numbers! "); // här får vi våra 10 tal av användaren 
        for (int hallon = 0; hallon < 10; hallon++)  //this loop will print the users numbers
        {

            Console.WriteLine($"Your have used: {hallon}"); // här får vi våra 10 tal av användaren 
            try
            {
                numbers[hallon] = Convert.ToInt32(Console.In.ReadLine());  // numbers lagrar våra 10 tal som kommer spelas nu mot winNum
            }
            catch (SystemException e) // going to catch if the user writes in a letter instead of a number        
            {
                --hallon;
                Console.WriteLine(e.Message);
            }
            if (numbers[hallon] < 1 || numbers[hallon] > 25)
            {
                Console.WriteLine("Write a number within 1-25. ");
                --hallon;
            }
        }

        if (numbers.Any(n => n == winNum))
        {
            Console.WriteLine("You won! The lucky number was:" + winNum);
        } else
        {
            Console.WriteLine("You lost");
        }

        Console.WriteLine("End of the game");
        Console.ReadLine();
    }