我正在使用Median of Medians pivot方法实现select-kth算法。具体来说,我跟随pseudocode listed here.。但是,我的代码崩溃了(下面讨论的错误),我知道它崩溃的原因,但我不明白我能做些什么。
错误
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 32017219
at Selection.partition(Selection.java:173)
原因
在Wiki链接中,如果select
,方法left == right
将返回值。但是,当从pivot
的return语句调用相互递归时,这意味着它将返回一个值(而不是索引)回到父级,父级将该值设置为pivotIndex
。
守则
public static int alg5(int[] a, int left, int right, int k) {
if (left == right) {
return a[left];
}
while (true) {
int pivotIndex = pivot(a, left, right);
pivotIndex = partition(a, left, right, pivotIndex);
if (k == pivotIndex) {
return a[k];
} else if (k < pivotIndex) {
right = pivotIndex - 1;
} else {
left = pivotIndex + 1;
}
}
}
public static int pivot(int[] a, int left, int right) {
for (int i = left; i <= right; i = i + 5) {
int subRight = i + 4;
if (subRight > right) {
subRight = right;
}
int median5 = partition5(a, i, subRight);
swap(a, median5, (int)(Math.floor(i / 5)));
}
return alg5(a, left, (int)(left + Math.ceil((right - left) / 5) - 1), ((right - left) / 10));
}
public static int partition5(int[] a, int left, int right) {
Arrays.sort(a);
return ((a.length - 1) / 2);
}
public static int partition(int[] a, int left, int right, int pivotIndex) {
int pivotValue = a[pivotIndex];
a = swap(a, pivotIndex, right);
int storeIndex = left;
for (int i = left; i <= right; i++) {
int aOfi = a[i];
if (a[i] < pivotValue) {
swap(a, storeIndex, i);
storeIndex++;
}
}
swap(a, right, storeIndex);
return storeIndex;
}
我完全理解为什么我的代码不起作用,我只是不理解如何修复它,因为它看起来正是算法指定的内容。指针非常感谢!
答案 0 :(得分:1)
有很多错误:
方法pivot
不应修改数组a
。它应该只为未来找到一个支点partition
。一个肮脏的解决方法是调用pivot(a.clone(), left, right);
。 (你不应该这样做,只是为了给你一个想法。)
Math.ceil((right - left) / 5)
是整数除法。您应该将它们转换为浮点数:Math.ceil(((float)(right - left)) / 5f)
。
在partition5
中,您对整个数组a
进行排序!您应该进行比较,找到a[left]
和a[right]
之间的中位数的索引。
在某些时候,您可能会right < left
,因此alg5
的第一行应该写if (left >= right)