Question

我正在开发Quicksort的修改版本，该版本使用((lowIndex + highIndex) / 2)公式确定其分区。

我的代码如下。但是，每当我运行它时，都会出现以下错误：

start ARRAY
1
2
10
9
3
4
8
7
5
6
Exception in thread "main" java.lang.StackOverflowError
        at newQuickSort.quicksort(newQuickSort.java:37)
        at newQuickSort.quicksort(newQuickSort.java:39)

重复at newQuickSort.quicksort(newQuickSort.java:39)行的地方。

本质上，我只是试图将分区函数更改为使用三位数分割，而根本不涉及Quicksort算法，而只是触摸分区函数。

我不确定为什么会发生此错误，尽管我已经引用了以下内容：

任何建议都会有所帮助，谢谢！

public class newQuickSort{ 

    static int count;

    static void move(int myArray[], int a, int b){
        int temp;
        temp = myArray[a];
        myArray[a] = myArray[b];
        myArray[b] = temp;
    } //end move

    //Create the partition function
    static int partition(int myArray[], int p, int r){
        int medIndex = (p+r) / 2;
        int pivot;

            if (myArray[p] > myArray[medIndex]){
                move(myArray, p, medIndex); 
            }else if (myArray[p] > myArray[r]){
                move(myArray, p, r);
            }else if (myArray[medIndex] > myArray[r]){
                move(myArray, medIndex, r);
            }
            //end if checks

            //now set proper movements
            move(myArray, medIndex, r-1);
            //set pivot
            pivot = myArray[r-1];
            //return pivot for which to partition around
            return pivot;
    } //end partition 

    static void quicksort(int myArray[], int p, int r){ 
        if (p < r){
        checkCount += 1; 
            int partition = partition(myArray, p, r); 
            quicksort(myArray, p, partition-1); 
            quicksort(myArray, partition+1, r); 
        } //end if 
    } //end quicksort

    public static void main (String[] args){
        int testarray[] = {1,2,10,9,3,4,8,7,5,6};
       // int testarray[] = {1,2,3,4,5,6,7,8,9,10,11,12,13,14,15};
        System.out.println("start ARRAY");
        for (int i = 0; i < testarray.length; i++){
            System.out.println(testarray[i]);
        } //end for

        int p = 0;
        int r = testarray.length-1;
        newQuickSort mySort = new newQuickSort();
        mySort.quicksort(testarray, p, r);

        System.out.println("end ARRAY");
        for (int j = 0; j < testarray.length; j++){
            System.out.println(testarray[j]);
         } //end for

    }
}

Answer 1

您的partition函数返回枢轴的值。但是其调用者（quicksort）期望数据透视表的 index 。

递归的终止取决于分区点在索引范围内。

解决此问题后，请考虑优化quicksort以最大程度地减少堆栈使用量：

首先，递归对较小的范围进行排序（从具有较少元素的意义上说）。
然后循环以继续更大的范围。

这保证了递归深度不超过log ₂ N。

Quicksort算法因StackOverflow错误而失败

1 个答案: