合并排序中超出了最大调用堆栈大小

时间:2017-10-27 10:46:31

标签: javascript algorithm mergesort

我正在研究合并排序并在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函数中。我用递归来实现它。

1 个答案:

答案 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)
  );
};