[英]How to implement mergeSort?
我正在尝试实现递归合并排序,但是该方法仅在排序向量中写入零。 这是我的代码。 在主要功能上,我只是将值读取到向量并将其作为参数传递给mergesort。 我做错任何想法吗? 谢谢。
public static void mergesort(int [] v){
int mid=v.length/2;
if( v.length < 2)
return;
int l[]=new int [mid];
int r[] = new int [v.length-mid];
for(int i=0; i<mid-1; i++)
l[i]=v[i];
for(int i=mid; i<v.length-1; i++)
r[i-mid]=v[i];
mergesort(l);
mergesort(r);
mergesort(l,r,v);
}
public static void mergesort(int [] l, int [] r, int [] v){
int i=0, j=0, k=0;
while(i< l.length && j< r.length){
if(l[i] <= r[j]){
v[k]=l[i];
i++;
}
else{
v[k]=r[j];
j++;
}
k++;
}
while(i < l.length){
v[k]=l[i];
i++;
k++;
}
while(j< r.length){
v[k]=r[j];
j++;
k++;
}
}
该算法有两个错误。
错误1:
while (i < r.length && j < r.length) {
if (l[i] <= r[j]) {
v[k] = l[i];
i++;
} else {
v[k] = r[j];
j++;
}
k++;
}
在这里,我必须小于l.length。 如果l.length!= r.length,这就是ArrayIndexOutOfBoundsException的原因。
错误2:
for (int i = 0; i < mid - 1; i++)
l[i] = v[i];
for (int i = mid; i < v.length - 1; i++)
r[i - mid] = v[i];
i的范围必须为0到(中-1)。 所以for循环应该在两个地方都使用<=运算符:
for (int i = 0; i <= mid - 1; i++)
l[i] = v[i];
for (int i = mid; i <= v.length - 1; i++)
r[i - mid] = v[i];
这就是得到零的原因。
修改后的代码如下:
public static void mergesort(int [] v) {
int mid = v.length / 2;
if ( v.length < 2)
return;
int l[] = new int [mid];
int r[] = new int [v.length - mid];
for (int i = 0; i <= mid - 1; i++)
l[i] = v[i];
for (int i = mid; i <= v.length - 1; i++)
r[i - mid] = v[i];
mergesort(l);
mergesort(r);
mergesort(l, r, v);
}
public static void mergesort(int [] l, int [] r, int [] v) {
int i = 0, j = 0, k = 0;
while (i < l.length && j < r.length) {
if (l[i] <= r[j]) {
v[k] = l[i];
i++;
} else {
v[k] = r[j];
j++;
}
k++;
}
while (i < l.length) {
v[k] = l[i];
i++;
k++;
}
while (j < r.length) {
v[k] = r[j];
j++;
k++;
}
}
假设您输入了一个大小为2的向量; 然后根据你的代码
mid=1;
for(int i=0;i<mid-1;i++)
// here i < (1-1) which will never execute
l[i]=v[i];
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.