Question

我正在尝试在java中实现一个quicksort，它将数组的最后一个元素作为一个数据透视表。然而，它似乎并没有起作用，即使在纸面上也应该......

这是输出：

before sort: 5 2 3 1 4 

after sort: 3 2 4 5 1

预期产量为1,2,3,4,5但似乎并未发生。

public static void main(String[] args) {
    int A[] = {5,2,3,1,0};
    ArrayUtils.printArray("before sort", A);
    quick_sort(A, 0, A.length-1);
    ArrayUtils.printArray("after sort", A);
}

public static void quick_sort(int[] A, int p, int r) {
    if (p < r) {
        int q = partition(A, p, r);
        quick_sort(A, p, q-1);
        quick_sort(A, q+1, r);
    }
}

我已经使用调试器跟踪问题，但我似乎无法找到它的位置。似乎循环有时会产生错误的i，但有时它会正常工作..我不确定，这就是我要问的原因。

public static int partition(int A[], int p, int r) {
    int x = A[r];
    int i = p-1;
    for(int j = p; j < r-1; j++) {
        if (A[j] <= x) {
            i = i + 1;
            int temp = A[j];
            A[j] = A[i];
            A[i] = temp;
        }
    }
    int temp = A[r];
    A[r] = A[i + 1];
    A[i + 1] = temp;
    return i + 1;
}

其中printArray（）是

public static void printArray(String name, int[] array) {
    if (array.length >= 10) {
        System.out.print(name + ": \n");
    } else {
        System.out.print(name + ": ");
    }

    for(int i = 0; i < array.length; i++) {
        if (i % 10 == 0 && i > 0) {
            System.out.print("\n");
        }
        System.out.print(array[i] + " ");

    }
    System.out.println("\n");
}

Answer 1

看起来您的分区算法外观基于Wikipedia中的Lomuto分区方案。但是，在维基百科中，伪代码表示for j := lo to hi - 1，它基于Pascal语法。在Pascal类语言中，这意味着j的最后一个值是hi - 1。在像Java这样的类C语言中，我们通常在索引的最后一个值之后将值设置为，而不是最后一个值。这意味着，在您的代码中，j实际上永远不会具有值hi - 1。修复for循环以确保j确实取值[{1}}，这样就可以了解情况。

java中的递归QuickSort无法正常工作

1 个答案: