为何Math.random函数返回值固定？石头剪刀布游戏代码异常

让我们逐个解决你的两个技术问题：

问题1：为何Math.random()始终返回相同值？ #

这种情况几乎都是因为你把Math.random()的结果赋值给了固定变量，然后重复复用这个变量，而不是每次需要随机数时重新调用函数。

举个常见的错误示例：

// 错误：只生成一次随机数，后续一直复用这个固定值
const fixedRandom = Math.random();
console.log(fixedRandom); // 第一次输出随机数
console.log(fixedRandom); // 和上次结果完全相同

正确的做法是每次需要随机数时直接调用Math.random()，它每次执行都会生成一个介于0（包含）到1（不包含）之间的新伪随机值：

// 正确：每次调用都生成新的随机数
console.log(Math.random()); // 输出一个随机数
console.log(Math.random()); // 输出另一个不同的随机数

Math.random()本身的伪随机算法是可靠的，只要你避免复用单次生成的结果，就不会出现重复值的问题。

问题2：getComputerChoice函数始终返回'paper' #

你的函数存在两个核心逻辑错误：

1. 随机数没有被保存：你执行了Math.floor(Math.random() * 3)，但没有把这个结果赋值给任何变量，导致生成的随机数直接被丢弃，后面的判断完全没用到它。
2. 判断条件逻辑错误：if (0)、else if (1)这种写法是直接判断字面量的真假。在JavaScript中，0是假值，所以if (0)永远不会触发；而1是真值，所以else if (1)会每次都执行，直接返回'paper'。

修正后的代码 #

先保存随机数，再用变量做条件判断：

function getComputerChoice() { 
  // 把随机生成的0、1、2保存到变量中
  const choice = Math.floor(Math.random() * 3); 
  // 根据变量值返回对应结果
  if (choice === 0) { 
    return 'rock'; 
  } else if (choice === 1) { 
    return 'paper'; 
  } else { 
    // 剩余情况只能是2，直接返回scissors
    return 'scissors'; 
  } 
}

更简洁的写法（推荐） #

你还可以用数组简化逻辑，代码更易读：

function getComputerChoice() {
  const choices = ['rock', 'paper', 'scissors'];
  // 生成对应数组长度的随机索引
  const randomIndex = Math.floor(Math.random() * choices.length);
  return choices[randomIndex];
}

这样修改后，每次调用函数都会随机返回三个选项中的一个，不会再固定返回'paper'了。

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