我正在尝试实施快速排序,输出不正确。我知道ArrayList
是由值引用的,但我不明白为什么它没有给我正确的输出。谁能告诉我我做错了什么?
import java.util.ArrayList;
public class Quicksort {
static ArrayList<Integer> quicksort(ArrayList<Integer> place){
if(place.size()<=1)
return place;
int median=place.size()/2;
int pivot=place.get(median);
place.remove(median);
ArrayList<Integer> place2=new ArrayList<>();
ArrayList<Integer> place3=new ArrayList<>();
for(int i=0;i<place.size();i++){
if(place.get(i)<pivot){
place2.add(place.get(i));
}else if(place.get(i)>pivot){
place3.add(place.get(i));
}else if(place.get(i)==pivot){
place3.add(place.get(i));
}
}
return concatenate(quicksort(place2),pivot,quicksort(place3));
}
static ArrayList<Integer> concatenate( ArrayList<Integer> first, int pivot, ArrayList<Integer> second){
ArrayList<Integer> third=new ArrayList<>();
for(int i=0;i<first.size();i++){
third.add(first.get(i));
}
third.add(pivot);
for(int i=0;i<second.size();i++){
third.add(second.get(i));
}
return third;
}
public static void main(String[] args) {
ArrayList<Integer> arraylist=new ArrayList<>();
arraylist.add(20);
arraylist.add(3);
arraylist.add(8);
arraylist.add(4);
arraylist.add(7);
arraylist.add(1);
arraylist.add(9);
arraylist.add(13);
quicksort(arraylist);
for(int n:arraylist){
System.out.println(n);
}
}
}
答案 0 :(得分:0)
您的quicksort
方法返回已排序的数组,但您没有指定它。将其更改为
arraylist = quicksort(arraylist);
它运作得很好。
您可以在quicksort方法中修改arraylist,因此不需要分配它,但这不是您正在做的事情。您所要做的就是删除枢轴,然后返回由concatenate
创建的新列表。这就是为什么在你的算法中你会得到原始列表,7
丢失。
如果您不想再次分配列表,则需要执行以下操作:
private static void quicksort(List<Integer> place) {
if (place.size() <= 1)
return;
int median=place.size()/2;
int pivot=place.get(median);
place.remove(median);
List<Integer> place2 = new ArrayList<>();
List<Integer> place3 = new ArrayList<>();
Iterator<Integer> it = place.iterator();
while (it.hasNext()) {
int i = it.next();
if (i < pivot) {
place2.add(i);
} else {
place3.add(i);
}
it.remove();
}
quicksort(place2);
quicksort(place3);
for (Integer i : place2) {
place.add(i);
}
place.add(pivot);
for (Integer i : place3) {
place.add(i);
}
}