如何修复石头剪刀布游戏中始终显示“not a tie”的问题

碧海醫心 2026-01-06 00:00:00 次阅读

该问题源于 `getcomputerchoice()` 函数未返回字符串值（如 "rock"），导致比较时 `userchoice === computerchoice` 恒为 `false`，所有结果均被判定为非平局。修复关键在于确保两个选择函数都**显式返回标准化的字符串**。

在实现石头剪刀布（Rock Paper Scissors）游戏逻辑时，一个常见却容易被忽略的错误是：函数声明了返回意图，却遗漏了实际的 return 表达式。你提供的代码中，getComputerChoice() 函数虽有 console.log() 输出，但每个 case 分支均以 return;（无值返回）结束，这意味着该函数实际返回 undefined。而 getUserChoice() 正确返回了小写化的用户输入（如 "rock"）。因此，每次执行 userChoice === computerChoice 时，实质是在比较类似 "rock" === undefined，结果永远为 false，从而始终进入 else 分支，输出 "not a tie"。

✅ 正确做法：统一返回可比较的字符串

必须让 getComputerChoice() 显式返回 "rock"、"paper" 或 "scissors" 字符串，且与 getUserChoice() 的返回值类型和格式一致（均为小写、无空格、纯字符串）：

function getComputerChoice() {
  const randomNumber = Math.floor(Math.random() * 3);
  switch (randomNumber) {
    case 0: return "rock";
    case 1: return "paper";
    case 2: return "scissors";
    default: throw new Error("Unexpected random number"); // 健壮性补充
  }
}

function getUserChoice() {
  const input = prompt("Rock, paper, or scissors?");
  if (!input) throw new Error("User cancelled or entered empty input");
  return input.trim().toLowerCase();
}

function playRound() {
  const userChoice = getUserChoice();
  const computerChoice = getComputerChoice();

  console.log(`You chose: ${userChoice}, Computer chose: ${computerChoice}`);

  if (userChoice === computerChoice) {
    console.log("Tie game!");
  } else {
    console.log("Not a tie");
  }
}

⚠️ 注意事项与最佳实践

避免副作用混淆逻辑：console.log() 仅用于调试或反馈，不能替代 return。函数的核心职责是产出值，而非打印。
输入校验不可少：prompt() 可能返回 null（用户取消）或空字符串，建议添加 trim() 和存在性检查，防止 " " === "rock" 类误判。
变量作用域合理：如你所疑，将 userChoice 或 computerChoice 提升至全局会引发重复调用（如两次 prompt）。保持它们在 playRound() 内声明是正确设计——既保证单次交互，又维持函数纯净性。

后续扩展提示：当准备添加胜负逻辑时，可基于如下规则封装判断：

const winConditions = { rock: "scissors", paper: "rock", scissors: "paper" };
if (winConditions[userChoice] === computerChoice) {
  console.log("You win!");
} else {
  console.log("You lose!");
}