索引越界错误c#

Question

使用我的数字返回因子函数，我得到了一个我不应该出现的错误。

这是我的代码：

static void Main(string[] args)
    {
        foreach (int element in Factors(16))
        {
            Console.Write(element.ToString() + ", ");
        }
    }
    static Array Factors(double value)
    {
        int[] factors = new int[] { };
        int counter = 0;

        for (int i = 1; i <= value; i++)
        {
            if (value % i == 0)
            {
                factors[counter] = i;
                counter++;
            }
        }
        return factors;
    }

这是我得到的错误：

TestSolver.exe 中出现“System.IndexOutOfRangeException”类型的未处理异常

附加信息：索引超出了数组的范围。

Answer 1

您正在初始化一个空数组，但随后尝试将值分配给不存在的索引。对于此应用程序，List<int> 可能会更好地为您服务。

    var factors = new List<int>();
    for (int i = 1; i <= (0.5 * value); i++)
        if (value % i == 0)
            factors.Add(i);
    return factors.ToArray(); // returns int[]

Answer 2

线

int[] factors = new int[] { };

创建一个包含 0 个元素的数组。 .NET 中的数组是固定大小的，所以稍后当您尝试使用 factors[counter] = i; 访问元素时，它会失败，因为没有分配任何元素。

您需要在分配factors（类似于int[] factors = new int[16];）时指定元素的数量，或者使用动态数组List<int>（您可以使用Add()在末尾添加新元素） .

Answer 3

如果您真的不想更改大部分代码，您可以采取的另一种方法是为数组分配一个初始大小（例如，它可以拥有的最大数量），然后再调整大小。

    static void Main(string[] args)
    {
        foreach (int element in Factors(16))
        {
            Console.Write(element.ToString() + ", ");
        }
    }
    static Array Factors(double value)
    {
        int[] factors = new int[Convert.ToInt32(Math.Round(value,0))];
        int counter = 0;

        for (int i = 1; i <= value; i++)
        {
            if (value % i == 0)
            {
                factors[counter] = i;
                counter++;
            }
        }
        Array.Resize(ref factors, counter);
        return factors;
    }

Answer 4

int[] factors = new int[] { };

正在创建一个没有元素的数组（零大小）。当你这样做时，它不会神奇地扩展：

factors[0] = 42;

要调整数组大小，您可以使用Array.Resize(ref arrName, newSize) 或选择一个可以调整大小的集合，例如List<int>。

Answer 5

您没有设置数组的大小，这会导致错误。作为一种解决方法，您可以将其放在列表中并迭代您传递的参数。你可以试试这样的。

 static Array Factors(double value)
    {
        List<int> factors = new List<int>();

        for (int i = 1; i <= value; i++)
        {
            if (value % i == 0)
                factors.Add(i);            
        }
        return factors.ToArray();
    }

Answer 6

问题是这一行：

int[] factors = new int[] { };

您正在创建一个名为factors 的新整数数组，但您没有指定该数组必须包含多少元素。以这种方式初始化数组时，必须使用长度声明数组。因此当你写...

factors[counter] = i;

...由于数组长度为 0 引发异常。它不能包含任何元素。 factors[0] = i 不是有效分配，因为 factors[0] 不存在。

您可以轻松地使用List<int> 而不是数组来完成此操作。可以根据需要将元素添加到列表中。完成列表后，您可以使用ToArray 将其转换为数组。因此你可以修改Factors如下：

static Array Factors(double value)
{
    List<int> factors = new List<int>();

    for (int i = 1; i <= (0.5 * value); i++)
    {
        if (value % 1 == 0)
        {
            factors.Add(i);
        }
    }
    return factors.ToArray();
}

请注意，不再需要 counter。