超出合并排序调用堆栈

时间:2018-02-14 01:49:40

标签: javascript algorithm sorting recursion mergesort

我担心我在这里错过了一些非常明显的东西,但我无法看到它。在chrome控制台和节点8.4.0中运行此命令,mergeSort会导致a RangeError: Maximum call stack size exceeded用于大于长度1的输入数组。合并函数似乎工作正常。这是代码:

"use strict";

function merge(A, B) {
  if (!Array.isArray(A) || !Array.isArray(B)) {
    throw new Error("merge expects both of its arguments to be arrays");
  }

  let result = [];
  let [i, j] = [0, 0];

  while (A[i]) {
    if (B[j]) {
      if (A[i] <= B[j]) {
        result.push(A[i]);
        i++;
      } else {
        while (A[i] >= B[j]) {
          result.push(B[j]);
          j++;
        }
        result.push(A[i]);
        i++;
      }
    } else {
      result.push(A[i]);
      i++;
    }
  }

  while (B[j]) {
    result.push(B[j]);
    j++;
  }

  return result;
}

function mergeSort(A) {
  if (!Array.isArray(A)) {
    throw new Error("mergeSort expects its argument to be an array");
  }

  if (A.length === 1) return A;
  let i = A.slice(Math.floor(A.length / 2));
  let R = A.slice(i);
  let L = A.slice(0, i);
  return merge(mergeSort(R), mergeSort(L));
}

1 个答案:

答案 0 :(得分:3)

i不应该是一个数组。你只想要

let i = Math.floor(A.length / 2);

另外你的基本情况太严格了,你的函数也会在空数组上无限递归。