我正在尝试实现一个Quicksort算法,我已经阅读了如何使用伪代码来实现它,因为我正在学习Java(因为我几个月前已经在C ++中完成了快速排序)我想把它推到这样的语言,但每当我尝试运行它时,我都会遇到堆栈溢出或堆空间问题,请检查我的代码? :d
public static int[] quicksort(int arreglo[]){
int size=arreglo.length;
int pivote=arreglo[size/2];
int menor[] = new int[size+2]; //If I leave the +2 I get stack overflow
int mayor[] = new int[size+2]; //If I delete them I get heap space problems
int j=0,k=0;
if(size>2){
for(int i=0;i<size;i++){
if(arreglo[i]<=pivote){
menor[j]=arreglo[i];
j++;
}
else{
mayor[k]=arreglo[i];
k++;
}
}
if(menor.length>=1&&mayor.length>=1)
return concatena(Ordena.quicksort(menor),Ordena.quicksort(mayor),j,k);
else
if(menor.length>mayor.length)
return menor;
else
return mayor;
}
else
return arreglo;
}
public static int[] concatena(int menor[],int mayor[],int limite1,int limite2){
int completo[] = new int[limite1+limite2];
System.arraycopy(menor,0,completo,0,limite1);
System.arraycopy(mayor,0,completo,limite1,limite2);
return completo;
}
感谢您的所有评论和回答,我已做出建议的更改,我将粘贴确切的例外:
线程“main”中的异常java.lang.OutOfMemoryError:Java堆空间 在Ordena.quicksort(Ordena.java:6) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21) 在Ordena.quicksort(Ordena.java:21)
这是修改后的代码(我已经翻译了我的变量,对不起,我没有注意到):
public static int[] quicksort(int array[]){
int size=array.length;
int pivot=array[size/2];
int less[] = new int[size+2];
int greater[] = new int[size+2];
int j=0,k=0;
if(size>2){
for(int i=0;i<size;i++){
if(array[i]<=pivot){
less[j]=array[i];
j++;
}
else{
greater[k]=array[i];
k++;
}
}
less[j]=pivot;
if(j>=1&&j>=1)
return concatenate(Ordena.quicksort(less),Ordena.quicksort(greater),j,k);
else
if(j>k)
return less;
else
return greater;
}
else
return array;
}
public static int[] concatenate(int less[],int greater[],int limit1,int limit2){
int complete[] = new int[limit1+limit2];
System.arraycopy(less,0,complete,0,limit1);
System.arraycopy(greater,0,complete,limit1,limit2);
return complete;
}
答案 0 :(得分:3)
主要问题来自这条线:
if(menor.length>=1&&mayor.length>=1)
应该是
if(j>=1&&k>=1)
为什么呢?好吧,第一个语句总是正确的,并且当所有被分区的元素等于或小于数据透视表时,它们都将以与它们进入的完全相同的顺序位于市长中。在函数上再次调用quicksort它完全相同的分区,你得到一个无限循环。根据你对menor或mayor数组的大小,程序首先会出现堆栈溢出或内存错误。
即使您更改了以上行,您的排序也无法正常工作。为什么?嗯,有几个原因。第一,行
if(menor.length>mayor.length)
应该是
if(j>k)
但是,这只是问题的一部分。当它们包含输入到函数的所有元素时,您将不会继续对市长或庄园进行排序。但是,如果您将它们发送到quicksort,那么您仍然可以进行无限循环。我建议将pivot与输入到quicksort的数组的其余部分分开(例如,将它与第一个元素交换)并对数组的其余部分进行分区。然后在这些数组自己快速排序后,将分段放在分区的mayor和menor数组之间。
祝你好运。