Range Error: Maximum call stack size exceeded - JavaScript

Question

我正在创建一个接受函数和字符串的函数 saveOutput。 saveOutput 将 return 一个与传入函数完全相同的函数，除了密码字符串作为参数传入时。发生这种情况时，returned 函数将 return 一个对象，其中所有先前传入的参数作为键，相应的输出作为值。

我认为下面的代码是正确的，但是当我运行我的代码时，我运行进入了 Range Error: Maxiumum call stack size exceeded。

   function saveOutput(inputFunc, string) {
      let obj = {};
      //inputFunc() accepts one args
      //string like a pwd
     return function inputFunc(input) {
        if (input === string) {
            return obj;
             } else {
            obj[input] = inputFunc(input);
            return inputFunc(input);
        }
      }
      //returns a fxn
      return inputFunc;
   }

    //Test cases
    const multiplyBy2 = function(num) { return num * 2; };
    const multBy2AndLog = saveOutput(multiplyBy2, 'boo');
    console.log(multBy2AndLog(2)); // should log: 4
    console.log(multBy2AndLog(9)); // should log: 18
    console.log(multBy2AndLog('boo')); // should log: { 2: 4, 9: 18 }

Answer 1

您使用了名字 inputFunc 两次。返回的函数称为 inputFunc 因此它隐藏了作为参数传递的回调函数。返回的函数调用 inputFunc 本身并导致无限递归，最终抛出“超出最大调用堆栈大小”错误。

要解决此问题，请使用不同的名称或使其匿名，因为无论如何都不需要该名称，这是经过一些改进的工作代码：

function saveOutput(inputFunc, string) {
  let obj = {};

  return function (input) {                    // make it anonymous
    if (input === string) {
      return obj;
    }
                                               // improvement 1: the else block can be omitted here
    return obj[input] = inputFunc(input);      // improvement 2: no need to call inputFunc twice, just assign and return at the same time 
  }

  // the return statement here is never reached because there is a return right before it so just remove it
}

在此处阅读有关变量阴影的更多信息：An example of variable shadowing in javascript

演示：

function saveOutput(inputFunc, string) {
  let obj = {};

  return function(input) {
    if (input === string) {
      return obj;
    }

    return obj[input] = inputFunc(input);
  }
}

const multiplyBy2 = function(num) {
  return num * 2;
};
const multBy2AndLog = saveOutput(multiplyBy2, 'boo');
console.log(multBy2AndLog(2));
console.log(multBy2AndLog(9));
console.log(multBy2AndLog('boo'));

Range Error: Maximum call stack size exceeded - JavaScript

Range Error: Maximum call stack size exceeded - JavaScript

javascript

closures

higher-order-functions