JavaScript生成不重复随机数求助：函数连续返回相同值

安全阿岳

2026-5-25

问题排查与修复方案

嗨，我来帮你捋清楚这个问题的根源～

你的函数之所以会连续返回相同的随机数，核心问题出在变量作用域上：你在random()函数内部用var声明了prevnumb，这意味着每次调用这个函数时，prevnumb都会被重新创建并初始化为undefined。

这样一来，每次执行do-while循环时，判断条件prevnumb == randNum其实是在比较undefined和生成的随机数，这个条件几乎永远不成立（除非randNum是undefined，但显然不可能），循环根本起不到过滤重复值的作用。而且函数执行结束后，prevnumb就会被销毁，完全没法记住上一次生成的数值，自然无法避免连续重复。

修复方案：用闭包保存状态

最优雅的解决方式是用闭包把prevnumb放在私有作用域里，既不会污染全局命名空间，又能持续保存上一次的数值：

const random = (function() {
  let prevnumb; // 这个变量会在闭包中持续存在，不会每次调用都重置
  return function() {
    // 处理边界情况：如果数组只有1个元素，直接返回0避免无限循环
    if (quotes.length <= 1) {
      return 0;
    }
    let randNum;
    do {
      randNum = Math.floor(Math.random() * quotes.length);
    } while (prevnumb === randNum); // 推荐用严格相等===，避免隐式类型转换坑
    prevnumb = randNum;
    return randNum;
  };
})();

另一种简单方案（不推荐，会污染全局）

如果你不想用闭包，也可以把prevnumb移到函数外部作为全局变量，但这种方式会污染全局命名空间，容易引发冲突，仅适合小型项目快速测试：

let prevnumb;
function random() {
  if (quotes.length <= 1) {
    return 0;
  }
  let randNum;
  do {
    randNum = Math.floor(Math.random() * quotes.length);
  } while (prevnumb === randNum);
  prevnumb = randNum;
  return randNum;
}