首先,我要说明这是一个家庭作业问题,我已经做了大量尝试。
我被要求修改Java中的quicksort,使用公式i * (n-1) /8
我编写了一个computeMedian
方法,它接受3个整数,确定最高值,然后返回该值。
代码:
public static int computeMedian(int x, int y, int z)
{
if((x >= y && x <= z) || (x >= z && x <= y)) {return x;}
else if((y >= x && y <= z) || (y >= z && y <= x)) {return y;}
else if((z >= x && z <= y) || (z >= y && z <= x)) {return z;}
else { return 0; }
}
然后我在我的findPivot
方法中使用它,该方法获取当前array, from, to
值并使用它们构建数据透视
以下是代码:
public static int findPivot(int[] a, int from, int to)
{
if(a.length <= 7)
{
return a[(to)/2];
}
else if(a.length > 7 && a.length <= 40)
{
return computeMedian(a[from], a[(to)/2] , a[to]);
}
else
{
int x = computeMedian(a[0 * (to) / 8], a[1 * (to) / 8], a[2 * (to) / 8]);
int y = computeMedian(a[3 * (to) / 8], a[4 * (to) / 8], a[5 * (to) / 8]);
int z = computeMedian(a[6 * (to) / 8], a[7 * (to) / 8], a[8 * (to) / 8]);
return computeMedian(x,y,z);
}
}
此方法适用于排序小于或等于40的任何数组,但是一旦大于40,我就会收到堆栈溢出错误,导致回{{1}中的computeMedian
方法部分。我会注意到else {}
适用于&gt;如果我把它放在那里40分,但这只是3个值的中位数,而不是3个中位数的中位数。
目前,这就是return computeMedian(a[from], a[(to)/2] , a[to]);
插入快速排序分区方法的方法:
findPivot
我非常难以理解为什么我的private static int modPartition(int[] a, int from, int to)
{
int pivot = findPivot(a, from, to);
int i = from - 1;
int j = to + 1;
while(i < j)
{
i++; while (a[i] < pivot) { i++; }
j--; while (a[j] > pivot) { j--; }
if (i < j) { swap(a, i, j); }
}
return j;
}
方法无法处理更大的数据集。我尝试通过for循环将computeMedian
值放在数组中,对它们进行排序并在中间返回值,并将值放在数组i * (n-1) / 8
中并调用{{1我得到了相同的堆栈溢出问题,但它往往会移动到我的代码的不同部分并引导我圈。
如果有人需要我可以发布任何更多的片段,但我认为我的问题可能就在这里。
感谢您的帮助。我还在学习,我认为抓住这个能够完全帮助我解决未来的问题。
以下是堆栈跟踪中的问题行:
第16行:p
第18行computeMedian(computeMedian(p[0], p[1], p[2]), computeMedian(p[3],p[4],p[5]),...etc
第23行int p = modPartition(a, from, to);
继承我的整个modSort方法:
modSort(a, p+1, to);
答案 0 :(得分:1)
转载并更正
添加代码以重现错误...
private static void swap(int[] a, int i, int j) {
int tmp = a[i];
a[i] = a[j];
a[j] = tmp;
}
public static void main(String[] args) {
// Generate a sample
// ArrayList<Integer> list = new ArrayList<>(64);
// for (int i = 0; i < 64; i++) list.add(i);
// Collections.shuffle(list);
// System.out.println(list);
int[] arr = {40, 9, 2, 62, 8, 42, 46, 23, 61, 45, 63, 48, 43, 36, 33, 32, 1, 55, 7, 17, 16, 25, 5, 26, 22, 11, 56, 38, 60, 31, 58, 29, 51, 34, 24, 54, 4, 3, 30, 20, 57, 18, 50, 44, 41, 12, 59, 6, 53, 39, 37, 35, 28, 13, 14, 15, 0, 19, 49, 52, 21, 27, 47, 10};
modSort(arr, 0, arr.length-1);
System.out.println(Arrays.toString(arr));
}
调试。设置StackOverFlowError
的断点(如评论中所示)不起作用。所以我去了一个常规断点(modSort
的开头)。
对于此示例数据,使用modSort
开始对from=3;to=5
进行无限递归。对于该范围,枢轴p = 2,这似乎是异常的。
我责备findPivot(a,from,to)
方法。看起来很适合找到整个a
的支点,但不适用于范围。尝试这种更正:
public static int findPivot(int[] a, int from, int to) {
final int rangeLength = to - from + 1;
if(rangeLength <= 7) {
return a[(from + to)/2];
} else if(rangeLength <= 40) { // why test "a.length > 7" ?
return computeMedian(a[from], a[(from + to)/2] , a[to]);
} else {
final int rangeLength_8 = (to - from) / 8;
int x = computeMedian(a[from], a[from + rangeLength_8], a[from + 2 * rangeLength_8]);
int y = computeMedian(a[from + 3 * rangeLength_8], a[from + 4 * rangeLength_8], a[from + 5 * rangeLength_8]);
int z = computeMedian(a[from + 6 * rangeLength_8], a[from + 7 * rangeLength_8], a[to]);
return computeMedian(x,y,z);
}
}
然后它适用于我的例子。我此时就停止了(必须要睡一觉)。
我认为您应该尝试熟悉调试器。我认为应该更容易弄明白。
答案 1 :(得分:0)
既然您已经为堆栈溢出问题实际包含了代码和错误消息,我们可以帮助您。
从堆栈跟踪中,我们可以看到无限递归可能第二次调用modSort
,因为第18行重复。
由于该调用与传入参数之间的唯一区别是第二个参数,我认为p
上的钱少于from
。
确认这是插入一个好的老式print
声明的最佳方法。
public static void modSort(int[]a, int from, int to)
{
if(from >= to) { return; }
int p = modPartition(a, from, to);
System.out.println("from=" + from + ", to=" + to + ", p=" + p);
modSort(a, from, p);
modSort(a, p+1, to);
}
结果输出应该显示出错误的非常明确的模式。