BMI计算器bug修复咨询：为何BMI≥25仍返回正常体重？

核心问题 #

第二个else if条件误用了逻辑或运算符||，导致只要result >= 18.5成立（哪怕result >=25），就会触发「正常体重」的返回分支，永远走不到超重的判断逻辑。正常体重的判断需要同时满足result >=18.5和result <=24.9，必须将||替换为逻辑与运算符&&。

额外优化点 #

除了核心逻辑错误，代码还有几个细节问题：

• result变量未声明，会污染全局作用域
• prompt返回的是字符串类型，直接参与计算会导致错误结果
• 函数参数weight和height被内部的prompt覆盖，完全冗余
• 返回结果的数值未做格式化，可读性差

修复后的完整代码 #

function bmiCalculator() {
  // 将输入转为数字类型，确保计算准确
  const weight = Number(prompt("你的体重是多少？"));
  const height = Number(prompt("你的身高是多少？"));

  // 声明局部变量存储计算结果
  const result = weight / Math.pow(height, 2);

  if (result < 18.5) {
    return `你的BMI值是 ${result.toFixed(1)}，你体重过轻`;
  } else if (result >= 18.5 && result <= 24.9) { // 修正逻辑运算符为&&
    return `你的BMI值是 ${result.toFixed(1)}，你的体重正常`;
  } else { // 简化分支，剩余情况必然是>=25
    return `你的BMI值是 ${result.toFixed(1)}，你体重超重`;
  }
}

console.log(bmiCalculator());

修复说明 #

1. 替换逻辑运算符：把||改成&&，确保只有处于正常体重区间时才返回对应结果
2. 类型转换：将prompt的返回值转为数字，避免字符串运算导致的错误
3. 变量声明：给result添加const声明，避免全局变量污染
4. 简化分支：最后一个判断直接用else，因为前面的条件已经覆盖了所有非超重的情况
5. 结果格式化：用toFixed(1)保留一位小数，让输出结果更直观

内容的提问来源于stack exchange，提问作者Moroni Bamvakiades Ramos