Question

我正在尝试编写一个计算快速排序程序中比较次数的程序。

这是我的代码

package algo_quicksort;

public class Algo_quicksort {

    public static int partition(int[]A,int p,int r){
        int x=A[p];
        int i=p+1;
        int temp;
        for(int j=p+1;j<r;j++){
            if(A[j]<x){//if A[j] is bigger than the pivot do nothing 
                temp=A[j];
                A[j]=A[i];
                A[i]=temp;
                i++;
            }
        }
        temp=A[p];
        A[p]=A[i-1];
        A[i-1]=temp;
        return i-1;
    }
    public static long quickSort(int[]A,int startPos,int length){
        if(length==1){
            return 0;
        }
        else{
            if(startPos<length){
            int pivot= partition(A,0,length);
          quickSort(A,startPos,pivot+1);
          quickSort(A, pivot+2,length); 
            return length-startPos-1;
        }
            else{
                return 0;
            }
    }
    }


    public static void main(String[] args) {
        int a[]={3,2,4};
        System.out.println("# of comparisons is: " +quickSort(a,0,a.length));

        System.out.println("A[] after quicksort is: ");

        for(int i=0;i<a.length;i++){
            System.out.print(a[i]+"  ");
        }

    }
}

它适用于任何大小为3或更小的数组，但如果它比任何大于它在递归调用时给我一个stackoverflow异常我尝试调试我的代码但无法弄清楚它出错的地方？

Answer 1

你有一个递归函数， quickSort（）。

通常当您使用递归方法获得stackoverflow条件时，这是因为您要么没有正确的“结束”条件（或何时停止），要么输入参数不正确。

我通过调整输入参数来试验你的代码并得到以下结果。

int a[]={3,2,4,5};    
System.out.println("# of comparisons is: " +quickSort(a,0,a.length -1));
//changed from a.length to a.length - 1

<强>结果

比较数是：2 快速排序后的[]是： 2 3 4 5

但是我不相信这是修复，因为如果我将数组更改为“int a [] = {5,3,2,4};”，则会再次发生stackoverflow错误：（

这让我相信你的最终条件有些不对劲......在 quickSort（）的某个地方。检查维基百科或stackoverflow并使用正确的实现验证您的代码。

因此，在为此编写了一些测试后，似乎 quicksort 的实现不正确。如果长度为1，它返回零。但是如果我传递一个值为5的长度为1的数组，则返回零，而我期望为5.

这意味着您的停止条件不正确。经过一些谷歌搜索后，我发现了以下内容：

第一个=最后一个;数组中只有一个元素表示已排序。
首先＆gt;持续;数组中没有值表示已排序。

然后你需要看一下 quicksort 的论据。我不相信你需要开始位置或数组长度。它应该只需要数组本身。

Answer 2

如果数组中有四个元素，则第一遍是3次比较+第二次传递中的2次+第三次传递中的1次，或6次深度。你必须在堆栈上推送一些东西进行每次比较。这表明在给定其他代码调用子例程的情况下，您的堆栈深度为8级，但是这些天的堆栈通常使用堆栈指针完成，并且可能长达数千个条目。 JVM Organization解释了整体结构。

计算快速排序中的比较次数给出stackoverflow

2 个答案: