Question

我无法理解什么是错的。

     static int WinningColumn()
    {
        Random rnd = new Random(46);

        int[] winningnumbers = new int[6];
        int[] Check = new int[46];
        int i;
        for (i = 0; i < winningnumbers.Length; i++)
        {
            winningnumbers[i] = rnd.Next(46);
            Check[winningnumbers[i]]++;

            if (Check[winningnumbers[i]] > 1)
            {
                i--;
                continue;
            }

错误发生在这里：

        }
        return winningnumbers[i];
       }

Answer 1

退出for循环时，索引器变量 i 的值大于可能的最大索引（这是破坏循环的条件）。
在您的情况下，变量 i 的值为6，但是数组winnumbers可能的最大索引是5.（0到5是六个整数元素）。

目前尚不清楚你的意图是什么，但假设你的目的是产生六个中奖号码，范围从0到45，那么你的代码应该以这种方式重写和简化

static List<int> WinningColumn()
{
    // Do not initialize Random with a fixed value
    // You will get always the same 'random' sequence
    Random rnd = new Random();

    // Create a list to store the winners
    List<int> winningnumbers = new List<int>();
    int i = 0;
    while(i < 6)
    {
        int newNumber = rnd.Next(46);
        if(!winningnumbers.Contains(newNumber))
        {
            // If the list doesn't contain the number the add it and increment i
            // Otherwise run the loop again....
            winningnumbers.Add(newNumber);
            i++;
        }
    }

    // This returns the whole list to the caller, 
    // you can use it as an array 
    return winningnumbers;
}

请注意，您的实际代码包含随机数生成器声明中的错误。您传递初始种子，因此，每次调用此方法时，随机生成器将以相同的数字序列再次启动。结果将是相同的数字列表。 Not very random to me
如果你没有通过任何东西，那么生成器会用系统时间初始化，因此每次调用这个方法时都应该不同。

Answer 2

我不知道你想在这里实现什么。

但是当我变成6时，循环将终止。所以你基本上试图访问winnumbers [6]这是不正确的，因为winsnumbers数组的长度为6，所以你可以使用从0到5的索引。

你可以试试winnumbers [i-1]

如何解决“IndexOutOfRange异常”错误？

2 个答案: