-2

我正在尝试使用合并排序对文本文件进行排序。但是，当我尝试运行它时，我不断收到索引超出范围的异常。我将指定发生异常的区域。

class Program
{
    static public void DoMerge(int[] numbers, int left, int mid, int right)
    {

        int[] temp = new int[25];

        int i, left_end, num_elements, tmp_pos;



        left_end = (mid - 1);

        tmp_pos = left;

        num_elements = (right - left + 1);



        while ((left <= left_end) && (mid <= right))
        {

            if (numbers[left] <= numbers[mid])

                temp[tmp_pos++] = numbers[left++];

            else

                temp[tmp_pos++] = numbers[mid++];

        }



        while (left <= left_end)

            temp[tmp_pos++] = numbers[left++];

        while (mid <= right)

上面的部分很好，但下面我有一个例外。

            temp[tmp_pos++] = numbers[mid++];

接下来的几行很好。

        for (i = 0; i < num_elements; i++)
        {

            numbers[right] = temp[right];

            right--;

        }

    }



    static public void MergeSort_Recursive(int[] numbers, int left, int right)
    {

        int mid;



        if (right > left)
        {

            mid = (right + left) / 2;

接下来的几行也给了我例外。

            MergeSort_Recursive(numbers, left, mid);

            MergeSort_Recursive(numbers, (mid + 1), right);

            DoMerge(numbers, left, (mid + 1), right);

据我所见，其余的都很好，但我会添加它，以便你们可以看到。

        }

    }

    static void Main(string[] args)
    {
        string fileNumbers = File.ReadAllText("sort(5).txt");
        string[] intNumbers = fileNumbers.Split(new char[] { ' ', '\t', '\r', '\n' }, StringSplitOptions.RemoveEmptyEntries);
        int[] numbers = new int[intNumbers.Length];
        int len = 999;
        //for (int n = 0; n < intNumbers.Length; n++)
        //{
        //    numbers[n] = int.Parse(intNumbers[n]);
        //}

        Console.WriteLine("MergeSort: ");
        MergeSort_Recursive(numbers, 0, len - 1);
        for (int i = 0; i < 1; i++)
            Console.WriteLine(numbers[i]);
        Console.Read();
    }
}

我要读取的文本文件有 500 个数字，最大数字是 999，最小数字是 6，数字非常随机，我必须对它们进行排序。PS我知道什么是超出范围的异常，我知道是什么原因造成的，因此我问我的代码为什么抛出异常。

4

1 回答 1

1

您的排序代码看起来大部分都很好。我在一个小数组上运行它并正确排序。

您的代码中有两个问题导致您的超出范围异常。

第一个问题是您设置len为 999。

您提到该文件包含 500 个数字，但最好设置len为numbers.Length.

static void Main(string[] args)
{
    string fileNumbers = File.ReadAllText("sort(5).txt");
    string[] intNumbers = fileNumbers.Split(new char[] { ' ', '\t', '\r', '\n' }, StringSplitOptions.RemoveEmptyEntries);
    int[] numbers = new int[intNumbers.Length];
    int len = numbers.Length;
    for (int n = 0; n < intNumbers.Length; n++)
    {
        numbers[n] = int.Parse(intNumbers[n]);
    }

    Console.WriteLine("MergeSort: ");
    MergeSort_Recursive(numbers, 0, len - 1);
    for (int i = 0; i < numbers.Length; i++)
        Console.WriteLine(numbers[i]);

    //Console.Read(); //Not sure what this is for....
}

第二个问题是当您创建temp数组时，您创建的数组长度为 25。这将修复该部分：

int[] temp = new int[numbers.Length];

请注意，更好的方法是temp在 main 函数中创建数组，并将其作为参数传递给MergeSort_Recursive()and DoMerge()，这样您就不必在每次合并操作时创建新数组。

于 2015-03-23T03:02:20.507 回答