Question

该代码适用于少数测试用例但大多数都失败了。我在哪里弄错了？

此外，如果可以采取任何措施来提高代码效率。

void quicksort(int *a, int a1, int b1)
{
if ((b1 - a1) > 0)
{
    int pivot = (a1 + b1) / 2;
    int i = a1;
    int j = b1;
    while (i <= j)
    {
        while (a[i] < a[pivot]) //left to right
            i++;
        while (a[j] > a[pivot]) //right to left
            j--;
        if (i < j) // swapping
        {
            int temp;
            temp = a[i];
            a[i] = a[j];
            a[j] = temp;
            i++;
            j--;
        }

    }
    quicksort(a, a1, j);
    quicksort(a, i, b1); //recursive calls
}

}

Answer 1

在i == j时你不会考虑这种情况，而且，正如@MK在评论中所说，数据透视应该是一个值，而不是一个索引（pivot = (a1+b1)/2;应该是pivot = a[(a1+b1)/2] {1}}）。

例如，如果您启动数组a = {2, 45, 56, 3, 125}（您在评论中指明的数组），您的实现可以转换为{2, 3, 45, 56, 125} i = 2，j = 2，以及pivot = 2（作为索引处理）。在这种情况下，程序将进入无限循环，因为它永远不会进入最后一个if块。

如果您将内部条件更改为i <= j，那么它应该是正确的。

最后，还应用了一些小的优化，您的代码应如下所示：

void quicksort(int *a, int a1, int b1)
{
    if (b1 > a1)    // one less operation (subtraction) with respect to the original
    {
        int pivot = a[a1 + ((b1 - a1) / 2)];    // it is now a value and not an index. the index is calculated in this way to avoid an arithmetic overflow
        int i = a1;
        int j = b1;
        while (i <= j)
        {
            while (a[i] < pivot) //left to right
                i++;
            while (a[j] > pivot) //right to left
                j--;
            if (i <= j) // swapping
            {
                int temp = a[i];
                a[i] = a[j];
                a[j] = temp;
                i++;
                j--;
            }
        }
        quicksort(a, a1, j);
        quicksort(a, i, b1); //recursive calls
    }
}

作为最后一次优化，我建议optimize the tail call。

如果有什么不清楚，请告诉我。

这个快速排序实施中的错误是什么？

1 个答案: