JavaScript：数组元素乘积返回值错误，如何排查修正？

让我帮你一步步找出这段代码里的问题，你遇到的结果不符合预期主要有几个关键错误：

• 循环起始索引错误：你初始化multi为1，但循环从i=1开始，直接跳过了数组的第一个元素A[0]（也就是1）。要计算整个数组的乘积，得从i=0开始遍历所有元素才行。
• 条件判断用错运算符：代码里的if(multi = 30)这类写法用的是赋值运算符=，而不是比较运算符===（或==）。这会导致不管multi原本的值是什么，都会被强行改成30，然后因为30是“真值”，第一个条件会直接成立，所以你才会得到返回值1，而不是正确的0。
• 变量i未声明：循环里的i没有用let或var声明，会变成全局变量，这是个不好的编程习惯，容易引发其他意外问题。
• 逻辑与需求不匹配：你的需求是计算数组所有元素的乘积，但当前代码的逻辑是判断乘积等于特定值时返回对应数字，否则输出日志。如果只是要返回乘积，那逻辑需要调整；如果是要保留这个判断逻辑，那得修正上面的错误。

修正后的代码（直接返回乘积） #

如果你的需求只是返回数组所有元素的乘积，用这段代码就可以：

function solution(A) {
  let multi = 1;
  // 从索引0开始遍历每一个元素
  for (let i = 0; i < A.length; i++) {
    multi *= A[i];
  }
  return multi;
}

console.log(solution([1,2,0,-5])); // 输出0，符合预期

修正后的代码（保留原判断逻辑） #

如果你需要保留原有的判断逻辑（乘积为30返回1，-30返回-1，0返回0），修正后的代码如下：

function solution(A) {
  let multi = 1;
  // 声明变量i，避免全局污染
  for (let i = 0; i < A.length; i++) {
    multi *= A[i];
  }
  // 用严格比较运算符===判断值和类型是否相等
  if (multi === 30) {
    return 1;
  } else if (multi === -30) {
    return -1;
  } else if (multi === 0) {
    return 0;
  } else {
    console.log("hey hey");
    // 建议这里也返回一个值，避免函数无返回值的情况
    return multi;
  }
}

console.log(solution([1,2,0,-5])); // 输出0，符合预期

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