为什么JavaScript函数返回值时出现undefined？求排查原因

1. 为什么会出现undefined？ #

你看到的undefined并不是函数返回的内容，而是控制台执行console.log(megaball())这个语句本身的返回值。

在Node.js或浏览器控制台中，如果你直接输入console.log(megaball())来调用函数：

• 首先执行megaball()，函数内部的console.log会依次输出55、67、17、break；
• 接着console.log会打印函数返回的17；
• 最后，因为console.log方法本身的返回值是undefined，控制台会自动输出这个返回值，这就是你看到的最后一行undefined。

如果不想看到这个undefined，直接在控制台输入megaball()调用函数即可——函数内部的日志会正常输出，同时控制台只会打印函数的返回值17，不会额外出现undefined。

2. 代码里的致命错误 #

你的代码里有一个语法级错误：JavaScript的全局数学对象是大写的Math，不是小写的math。当前代码里的math.floor和math.random会抛出ReferenceError: math is not defined错误，必须修正为：

random_number = Math.floor(Math.random() * 100)

3. 优化随机数生成逻辑 #

你当前通过循环筛选1-25随机数的方式效率很低，其实可以直接生成目标范围内的随机数，完全不需要循环：

const megaball = () => {
  // Math.random()生成0-1的随机数，乘25得到0-24.999...，取整后是0-24，加1变成1-25
  const random_number = Math.floor(Math.random() * 25) + 1;
  console.log(random_number);
  return random_number;
}

这样代码更简洁，也避免了无意义的循环消耗。

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