Question

我已经实现了使用列表的第一个元素作为数据透视的quicksort算法，它运行正常。现在我refactored选择一个随机索引作为pivot元素，与第一个元素交换并执行quicksort子例程。不知何故，它不起作用，我没有得到排序的数组。这是我的代码，这是不言自明的，但我很乐意解释是否需要澄清。

public class Qsort {
       public static void quickSort2(int[] arr, int i, int j){
        if (i<j){
            int part =randPartition(arr, i, j);
            quickSort2(arr, i, part-1);
            quickSort2(arr, part+1, j); 
        }
    }
    public static int randPartition(int[] arr, int start, int end){
        //int pivId = (int)(Math.random()*(end-start)+1)+start; -- *****
        int pivId = start;
        //System.out.println("pivId is "+ pivId + "; start is " + start + "; end is " + end);
        System.out.println("arr is "+ Arrays.toString(arr));
        int pivot = arr[pivId];
        swap(arr, start, pivId);
        int i = start;

        for (int j = start+1;j<=end;j++){
            if (arr[j]<pivot){
                    i++;
                    swap(arr,i,j);
            }
        }
        swap(arr, start,i);
        return i;
    }

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

   public static void main(String[] args) {
        int[] data2 = new int[]{10,11,9,7,5};
        System.out.println("Unsorted array data " + Arrays.toString(data2));
        quickSort2(data2, 0, data.length-1);
        System.out.println("Sorted array data " + Arrays.toString(data2));
      }
}

我已评论random pivot calculation in the code with *****。我没有看到任何问题，但随机枢轴计算破坏了代码

Answer 1

其余代码没问题，但

int pivId = (int)(Math.random()*(end-start)+1)+start; -- *****

有一些问题。考虑结束==开始，然后你得到了pivID == 1 + start。

Java中的随机快速排序需要修复吗？

1 个答案: