在我的sort方法中,我的mergesort调用始终出现stackoverflow错误。它可能是一个快速解决方案,但目前我什么也没想到。请查看我的代码。
public class MergeSorter {
public static <T> void sort(Comparable<? extends T>[] items) {
if (items == null) {
throw new IllegalArgumentException("Item is null.");
}
mergeSort(items, 0, items.length - 1);
}
@SuppressWarnings("unchecked")
private static <T> void mergeSort(Comparable<? extends T>[] items, int begIndx, int endIndx) {
if(begIndx == endIndx) {
return;
}
if(items.length > 1) {
int midIndx = items.length / 2;
mergeSort(items, begIndx, midIndx);
mergeSort(items, midIndx + 1, endIndx);
merge(items, begIndx, midIndx, endIndx);
}
}
@SuppressWarnings("unchecked")
private static <T> void merge(Comparable<? extends T>[] array, int begIndx, int midIndx, int endIndx) {
int sizeOfLeft = midIndx - begIndx + 1;
int sizeOfRight = endIndx - midIndx;
/// change to generic later
@SuppressWarnings("unchecked")
T[] leftArr = (T[]) new Object[sizeOfLeft + 1];
@SuppressWarnings("unchecked")
T[] rightArr = (T[]) new Object[sizeOfRight + 1];
for (int i = 0; i <= sizeOfLeft; i++) {
leftArr[i] = (T) array[begIndx + i];
}
for (int j = 0; j <= sizeOfRight; j++) {
rightArr[j] = (T) array[midIndx + j + 1];
}
int i = 0;
int j = 0;
for (int k = begIndx; k <= endIndx; k++) {
if (i == sizeOfLeft) {
array[k] = (Comparable<? extends T>) rightArr[j++];
} else if(j == sizeOfRight) {
array[k] = (Comparable<? extends T>) leftArr[i++];
} else if(((Integer) leftArr[i]).compareTo((Integer)rightArr[j]) <= 0) {
array[k] = (Comparable<? extends T>) leftArr[i++];
} else {
array[k]=(Comparable<? extends T>) rightArr[j++];
}
}
}
}
如果我给它一个由5个东西组成的整数数组
5、24、79、8、3;
那我应该拿回3,5,8,24,79。
答案 0 :(得分:1)
您的问题是您始终将中索引基于items.length,每次调用都相同。
对于该调用,中间索引应介于begIndx和endIndx之间,这很重要。