递归sum函数返回值为undefined的原因排查及修复方案求助

嘿，看你的描述和代码片段，我几乎能确定问题出在两个地方之一（或者两者都有）：递归调用时没有返回函数执行结果，或者是处理单个元素的分支漏掉了return语句——毕竟你贴的代码里if (numbers.length === 1)后面是...，明显没写完对吧？

为什么会返回undefined？ #

JavaScript里，如果函数执行到最后没有遇到return语句，或者某个分支里没有return，那这个函数调用的返回值就是undefined。递归函数尤其依赖每一层调用都返回正确的值，否则上层调用拿到undefined进行累加，最终结果自然就是undefined了。

比如你可能写出了这样的错误代码：

function sum() {
  var numbers = [...arguments];
  if (numbers.length === 0) {
    return 0;
  }
  if (numbers.length === 1) {
    // 这里漏掉了return numbers[0]
  }
  // 这里只执行了递归，但没return结果
  numbers[0] + sum(...numbers.slice(1));
}

修复后的代码 #

下面是修正后的版本，我帮你补全了缺失的部分，并且优化了逻辑：

function sum() {
  var numbers = [...arguments];
  // 基准情况：空数组直接返回0
  if (numbers.length === 0) {
    return 0;
  }
  // 递归分支：必须return递归调用的结果，加上当前第一个元素
  return numbers[0] + sum(...numbers.slice(1));
}

关键修复点 #

1. 确保所有分支都有返回值：我把单个元素的判断分支去掉了——因为当数组只剩1个元素时，numbers.slice(1)会返回空数组，递归调用sum()会返回0，加上当前元素正好是正确值，这样代码更简洁。
2. 递归调用必须return：一定要把sum(...numbers.slice(1))的结果作为当前函数的返回值返回，否则上层调用拿不到递归的计算结果，最终就会得到undefined。

测试一下：

console.log(sum(1,2,3)); // 输出6，正常工作
console.log(sum()); // 输出0，符合预期
console.log(sum(5)); // 输出5，没问题

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