2

我对编程和java很陌生,所以请不要笑这个:)。现在..我在网上查看了快速排序算法的正确实现,我知道这一点。但我试图以我能想到的非常天真、基本的方式自己实现它。实际上创建两个单独的数组(左,右)并继续对它们进行排序等等......

这是代码:

package excercise;

public class MyQuickSort {


    public static int list[] = {6,5,3,1,8,7,2,4};


    public static int[] addNumberToArray(int array[],int num){
        int newArray[] = new int[array.length+1];
        for(int i = 0;i<array.length;i++){
            newArray[i] = array[i];
        }
        newArray[newArray.length-1] = num;
        array = newArray;
        return array;
    }


    public static int[] partition(int arr[]){

        while(arr.length>1){
            int pivotIndex = (int)(arr.length/2);
            int left[] = new int[0];
            int right[] = new int[0];

            for(int i = 0;i<arr.length;i++){
                if(i==pivotIndex){
                    continue;
                }

                else if(arr[i]<=arr[pivotIndex]){
                    left = addNumberToArray(left,arr[i]);
                }
                else{
                    right = addNumberToArray(right,arr[i]);
                }
            }
            int origPivot = arr[pivotIndex];
            int k = 0;
            while(k<left.length){
                arr[k] = left[k];
                k++;
            }
            arr[k] = origPivot;
            k++;
            while(k<arr.length){
                arr[k] = right[k-(left.length+1)];
            }


            if(left.length>1){
            partition(left);
            }

            if(right.length>1){
            partition(right);
            }

        }

            return arr;
    }




    public static void main(String[]args){
            list = partition(list);
            for(int i = 0;i<list.length;i++){
                System.out.print(list[i]+" ");
            }

        }
    }

但是为什么这会陷入循环并且不起作用?我不明白这段代码有什么问题..除了这不是很有效(至少可以说)!但我很固执,无论如何都想尝试让它发挥作用……如果你有任何建议,那就太好了!提前谢谢

好的,这是新代码,调试后一切似乎都正常,但是当我想打印新的排序 arr 时,它仍然会打印我原来的未排序数组。递归使整个事情回到它开始的地方。我怎样才能让它“保存步骤”?我应该在哪里打一个“返回”电话,我应该返回什么?

public class MyQuickSort {

    public static int list[] = {6,5,3,1,8,7,2,4};
    public static boolean sorted;
    public static int[] addNumberToArray(int arr[],int num){
        int newArr[] = new int[arr.length+1];
        for(int i = 0;i<arr.length;i++){
            newArr[i] = arr[i];
        }
        newArr[newArr.length-1] = num;
        arr = newArr;
        return arr;

    }


    public static void partition(int arr[]){

        while(!sorted){
            int pivotIndex = (int)(arr.length/2);
            int left[] = new int[0];
            int right[] = new int[0];

            for(int i = 0;i<arr.length;i++){
                if(i==pivotIndex){
                    continue;
                }
                else if(arr[i]<=arr[pivotIndex]){
                    left = addNumberToArray(left,arr[i]);
                }
                else{
                    right = addNumberToArray(right,arr[i]);
                }
            }
            int origPivot = arr[pivotIndex];
            int k = 0;
            while(k<left.length){
                arr[k] = left[k];
                k++;
            }
            arr[k] = origPivot;
            k++;
            while(k<arr.length){
                arr[k] = right[arr.length-arr.length-(left.length+1)+k];
                k++;
            }


            if(left.length>1){
                partition(left);
            }


            if(right.length>1){
                partition(right);
            }

            if(left.length<=1&&right.length<=1){
                sorted = true;

            }

        }



    }








    public static void main(String[] args) {
        partition(list);
        for(int i = 0;i<list.length;i++){
            System.out.print(list[i]+" ");
        }


    }

}
4

2 回答 2

1

你的算法卡在这个循环中

while(k<arr.length){
    arr[k] = right[k-(left.length+1)];
}

原因是你没有增加你的 k。

尝试

while(k<arr.length){
    arr[k] = right[k-(left.length+1)];
    k++;
}
于 2013-02-11T10:19:47.210 回答
1

凉爽的!您的实现似乎在逻辑上描述了快速排序。但是,请注意快速排序不能使用额外的缓冲区来实现。它应该在自己的主数组中排序,递归调用仅传递开始和结束边界。您的实现更像是一种合并排序风格,使用快速排序逻辑。是的,你错过了上面评论者所说的 k++。

于 2013-02-11T10:37:10.323 回答