2

对于学校作业,我必须实施归并排序。

我用这段代码来做到这一点:

static int[] MergeSort(int[] C)
    {
        int left = 0;
        int right = C.Length;
        int middle = (left + right) / 2;
        int[] A, B;
        A = new int[middle];
        B = new int[middle];

        if (C.Length == 0 || C.Length == 1)
        {
            return C;
        }
        else
        {
            for (int i = left; i < middle; i++)
            {
                A[i] = C[i];
                B[i] = C[middle + i];
            }

            MergeSort(A);
            MergeSort(B);
            return Merge(A, B, C);
        }
    }

    static int[] Merge(int[] A, int[] B, int[] C)
    {
        int i, j, k;
        i = j = k = 0;
        int n = A.Length;
        int m = B.Length;
        int c = C.Length;
        int middle = C.Length / 2;

        while (i < n && j < m)
        {
            if (A[i] < B[j])
            {
                C[k] = A[i];
                i++;
            }
            else
            {
                C[k] = B[j];
                j++;
            }
            k++;

            if (i == n)
            {
                for (int b = i; b < B.Length; b++)
                {
                    C[middle + b] = B[b];
                }
            }
            else
            {
                for (int a = i; a < A.Length; a++)
                {
                    C[middle + a] = A[a];
                }
            }
        }

        return C;
    }

但它不适用于许多不同的行。我已经调试并检查了约束是否有问题,但我似乎找不到问题。

提前致谢!

4

2 回答 2

2

我发现的第一件事是您如何将数组分成两部分:

A = new int[middle];
B = new int[middle];

如果长度不均匀,您将省略最后一项。你应该有:

A = new int[middle];
B = new int[right - middle];

然后你会为它们使用单​​独的循环,因为它们的长度可能不同:

for (int i = left; i < middle; i++) {
  A[i - left] = C[i];
}
for (int i = middle; i < right; i++) {
  B[i - middle] = C[i];
}
于 2012-10-07T19:35:30.540 回答
1

除了 Guffa 答案之外,您还应该像这样编辑 Merge 方法的代码:

    while (i < n && j < m)
    {
        if (A[i] < B[j])
        {
            C[k] = A[i];
            i++;
        }
        else
        {
            C[k] = B[j];
            j++;
        }
        k++;
    }
    if (i == n)
    {
        for (int b = j; b < B.Length; b++)
        {
            C[k++] = B[b];
        }
    }
    else
    {
        for (int a = i; a < A.Length; a++)
        {
            C[k++] = A[a];
        }
    }

While 循环块应该在 k++ 之后立即结束,并且在第一个 for 循环中,您应该用 j 而不是 i 初始化 b。另外,注意 C 数组的下一个元素的索引,它是 k,不一定是中间 + a 或 b。

于 2012-10-07T23:02:57.067 回答