JavaScript脚本问题：输入5个数字后maxNum提示undefined求排查

你的代码存在几个关键问题，导致alert时maxNum显示undefined，同时还有逻辑错误会导致最大值计算不正确：

1. 内层循环重复声明局部变量maxNum #

你在内层循环里每次迭代都用let maxNum = arrNum[0]重新声明了一个局部变量，这个变量和外层的maxNum完全无关。外层的maxNum始终处于未赋值的状态，最终就是undefined。而且每次循环都重置这个局部变量，根本无法完成有效的最大值比较。

2. prompt输入未转为数字 #

prompt返回的是字符串类型，直接用字符串比较大小会按照字符的Unicode编码排序（比如"10"会比"2"小），导致最大值计算错误。

3. 循环变量i未声明 #

两个循环里的i都没有用let/const声明，会成为全局变量，可能引发意外的作用域问题。

4. 最大值比较逻辑写反 #

你的判断条件if (maxNum > arrNum[i])是把当前元素赋值给maxNum，这是找最小值的逻辑，而非最大值。

修正后的代码 #

function fiveNum() {
  const arrNum = [];
  // 用let声明循环变量i，避免全局污染
  for (let i = 0; i < 5; i++) {
    // 将输入转为数字，确保数值比较的正确性
    let inputNum = Number(prompt("Enter number!"));
    arrNum.push(inputNum);
  }
  // 初始化maxNum为数组第一个元素，放在循环外
  let maxNum = arrNum[0];
  // 从第二个元素开始比较，减少一次无效迭代
  for (let i = 1; i < arrNum.length; i++) {
    // 正确的最大值比较逻辑
    if (arrNum[i] > maxNum) {
      maxNum = arrNum[i];
    }
  }
  alert("Your highest number is " + maxNum);
}
fiveNum()

更简洁的实现（用内置API） #

可以直接使用Math.max结合展开运算符简化代码：

function fiveNum() {
  const arrNum = [];
  for (let i = 0; i < 5; i++) {
    let inputNum = Number(prompt("Enter number!"));
    arrNum.push(inputNum);
  }
  // 直接获取数组最大值
  const maxNum = Math.max(...arrNum);
  alert("Your highest number is " + maxNum);
}
fiveNum()

注：如果用户输入非数字内容，Number()会返回NaN，此时Math.max的结果也会是NaN。如果需要处理这种情况，可以在存入数组前增加输入有效性判断。

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