石头剪刀布游戏中score变量数值异常问题求助

首先，咱们来拆解你遇到的两个核心问题：

1. 连续获胜分数无法超过1 #

这个问题大概率是测试方式导致的——如果你每次测试都重新运行整个脚本（比如反复把所有代码粘贴到控制台），那么let score = 0;会被重新执行，直接把分数重置为0，所以每次获胜只能得到1分。

要让分数正常累加，你需要：

• 先一次性运行所有代码（完成computerPlay、score和playRound函数的定义）
• 然后在同一个会话中多次调用playRound，比如：

  playRound('rock', 'scissors'); // 分数变为1
  playRound('scissors', 'paper'); // 分数变为2，这时候就能看到正确的累加结果了
  

另外，你的胜负判断条件里，最后一个玩家获胜的分支虽然逻辑正确，但缺少括号会降低可读性，建议补上：

else if (
  (playerSelection === 'paper' && computerSelection === 'rock') ||
  (playerSelection === 'rock' && computerSelection === 'scissors') ||
  (playerSelection === 'scissors' && computerSelection === 'paper') // 补上括号，逻辑更清晰
)

2. 平局时分数变为0 #

这其实是个误解——你的平局分支根本没有修改score的值：

else {
  return "Tie and the score is " + score;
}

你看到分数变成0，大概率是平局前电脑赢了一次，触发了电脑获胜分支里的score = 0逻辑（当分数减到负数时强制设为0）。比如：

1. 初始分数为0
2. 电脑赢一次：score--变成-1，被重置为0
3. 平局时返回的就是0，让你误以为是平局导致分数归零

代码优化建议 #

为了避免类似的逻辑错误，咱们可以简化代码逻辑，用对象映射来判断胜负，这样更清晰也不容易出错：

function computerPlay() {
  // 用数组随机选择，比Math.ceil的写法更直观
  const choices = ["rock", "paper", "scissors"];
  return choices[Math.floor(Math.random() * choices.length)];
}

let score = 0;

function playRound(playerSelection, computerSelection) {
  // 统一转为小写，避免玩家输入大写字母导致判断失败
  const player = playerSelection.toLowerCase();
  const computer = computerSelection.toLowerCase();

  if (player === computer) {
    return `Tie and the score is ${score}`;
  }

  // 用对象存储玩家获胜的对应关系，避免冗长的条件判断
  const winMap = {
    rock: "scissors",
    paper: "rock",
    scissors: "paper"
  };

  if (winMap[player] === computer) {
    score++;
    return `Player wins and the score is ${score}`;
  } else {
    score--;
    // 用Math.max确保分数不会为负，替代手动判断
    score = Math.max(score, 0);
    return `Computer wins and the score is ${score}`;
  }
}

// 测试示例：连续调用两次
console.log(playRound('rock', computerPlay()));
console.log(playRound('scissors', computerPlay()));

这个优化后的代码做了这些改进：

• 简化了computerPlay的随机选择逻辑，更易读
• 增加了大小写转换，避免玩家输入格式错误导致判断失效
• 用对象映射胜负关系，减少了冗余的条件判断
• 用Math.max(score, 0)替代手动判断，代码更简洁

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