我正在研究合并排序并在JavaScript中实现它,但它返回了一个错误:
function mergeSort(input){
RangeError:超出最大调用堆栈大小
这是我的代码:
var array = [2,4,6,7,1,3,5,10,9,8];
// using merge sort: (best sort => O (n log n))
function mergeSort (array) {
var array1 = [];
var array2 = [];
for (let i = 0; i < array.length/2; i++) {
array1.push(array[i]);
}
for (let i = array.length/2; i < array.length; i++) {
array2.push(array[i]);
}
array1 = mergeSort(array1);
array2 = mergeSort(array2);
return merge(array1, array2);
}
function merge(a, b) {
let c = [];
while(a.length > 0 && b.length > 0) {
if (a[0] > b[0]) {
c.push(b[0]);
b.splice(0, 1);
} else {
c.push(a[0]);
a.splice(0, 1);
}
}
while (a.length > 0) {
c.push(a[0]);
a.splice(0, 1);
}
while (b.length > 0) {
c.push(b[0]);
b.splice(0, 1);
}
return c;
}
console.log(mergeSort(array));
我猜错误在mergeSort
函数中。我用递归来实现它。
答案 0 :(得分:3)
这个特殊原因与mergeSort
函数内的无限递归有关。你没有任何条件就可以越来越深地调用它。所以mergeSort
应该像这样重写:
function mergeSort(input) {
// Here is your recursion stop condition
if (input.length === 1) return input;
const median = Math.floor(input.length / 2);
// Limit arrays should get sliced with each iteration
const limitA = input.slice(0, median);
const limitB = input.slice(median);
return merge(
mergeSort(limitA), mergeSort(limitB)
);
};