JavaScript调用函数后reduce方法失效问题排查

嘿，我来帮你看看为啥代码没得到预期的120。先拆解下你的代码问题：

核心问题 #

1. 未调用生成数组的函数：你定义了factorialize函数，但没执行factorialize(5)来填充数组arr，这会导致后续reduce操作要么报错，要么得不到正确的计算对象。
2. reduce初始值错误：你给reduce传的初始值是0，但0乘以任何数都是0，这直接让乘积结果变成0（就算数组有值也不对）。乘法的单位元是1，所以初始值应该设为1。
3. 全局变量隐患：把arr定义成全局变量，容易被后续代码意外修改，最好把数组放在函数内部，保持作用域干净。

修正后的代码 #

function factorialize(num) {
  // 函数内部声明数组，避免全局污染
  var arr = [];
  for (var i = 1; i <= num; i++) {
    arr.push(i);
  }
  // 直接在函数内用reduce计算乘积并返回结果
  return arr.reduce(function(previousVal, currentVal) {
    return previousVal * currentVal;
  }, 1); // 初始值设为1，保证乘法逻辑正确
}

// 调用函数并获取阶乘结果
var factors = factorialize(5);
console.log(factors); // 输出120

分步解释 #

• 将数组arr移到factorialize函数内部，每次调用函数都会生成独立的新数组，避免全局变量带来的意外问题。
• 在函数内部直接调用reduce，初始值设为1，这样第一次计算是1*1，后续依次乘以后面的元素，最终得到正确的阶乘结果。
• 调用factorialize(5)就能直接拿到计算后的阶乘值，不用再单独处理数组。

如果想保留“先生成数组再计算”的分步逻辑，也可以这么写：

function getFactorialArray(num) {
  var arr = [];
  for (var i = 1; i <= num; i++) {
    arr.push(i);
  }
  return arr;
}

// 先生成目标数组
var arr = getFactorialArray(5);
// 再计算乘积，注意初始值为1
var factors = arr.reduce(function(previousVal, currentVal) {
  return previousVal * currentVal;
}, 1);

console.log(factors); // 输出120

这样就能正确得到你想要的120啦！

内容的提问来源于stack exchange，提问作者daggett