QuickSort代码不起作用

时间:2013-10-13 21:10:07

标签: java quicksort

我一直在努力学习如何实现快速排序。我读了基本算法并尝试首先实现我自己的代码(根据我对算法的理解)。

为了简单起见,我将枢轴作为第一要素。如果可以,请忽略它。我的代码运行但没有返回正确的结果,我无法弄清楚原因。非常感谢任何帮助。

public class QuickSortMain 
{
public static void displayArray(int[] arr)
{
    for(int i = 0; i < arr.length; i++)
        System.out.print(arr[i] + "  ");
}

public static void quickSort(int[] arr)
{
    quickSort(arr, 0, arr.length - 1);
}

private static void quickSort(int[] arr, int lo, int hi)
{
    if(lo >= hi) return;

    int j = lo;
    int left = lo;
    int right = hi;

    while(arr[left] <= arr[j])
        left++;

    while(arr[right] >= arr[j])
        right--;

    if (left >= right) return;

    swap(arr, left, right);
    swap(arr, j, left);

    quickSort(arr, lo, j-1);
    quickSort(arr, j+1, hi);
}

private static void swap(int[] arr, int i1, int i2)
{
    int temp = arr[i1];
    arr[i1] = arr[i2];
    arr[i2] = temp;
}

public static void main(String[] args)
{

    int[] myArray = {14, 79, 11, 42, 3, 27, 192, 44, 26, 742, 129};
    System.out.println(myArray.length);
    System.out.println("Original Array:");
    displayArray(myArray);

    quickSort(myArray);
    System.out.println("");
    System.out.println("Sorted Array:");
    displayArray(myArray);

}
}

以下是我得到的输出: 11 原始阵列: 14 79 11 42 3 27 192 44 26 742 129
排序数组: 3 14 11 42 79 27 192 44 26 742 129

谢谢!

1 个答案:

答案 0 :(得分:0)

您需要确保所有小于枢轴的值都在一侧 并且所有更大的值都在另一个上。你没有在分区方法中做足够的交换。

要解决此类情况,我建议您编写测试用例以突出显示算法无效的区域。另外使用调试器有帮助。

private static void quickSort(int[] arr, int lo, int hi)
{
    if(lo >= hi) return;

    int j = lo;
    int left = lo;
    int right = hi;

    while (left < right){
        left++;
        while(arr[left] < arr[j] && (left < right))
            left++;

        while(arr[right] > arr[j] && (left < right))
            right--;

        if (left < right){
            swap(arr, left, right);
        }
    }

    if(arr[left] < arr[j]){
        swap(arr, j, left);
    }

    quickSort(arr, lo, left-1);
    quickSort(arr, left, hi);
}