JS技术问题：脚本无法终止、分数不递增的排查求助

核心问题分析 #

你的代码主要有3个关键问题导致分数无法正确更新：

1. 值传递导致分数无法同步到全局变量
   JavaScript的原始类型（如数字）采用值传递，checkWinner函数里的cscore和pscore是全局分数的副本，修改它们只会改变函数内部的局部变量，不会影响全局的computerScore和playerScore。

2. 循环中未更新玩家与电脑的选择结果
   循环内调用getPlayerChoice和getComputerChoice后，没有将返回值重新赋值给playerChoice和computerChoice，导致每次循环都使用初始化时的那一组选择，分数自然不会根据新的对局变化。

3. 无效输入未处理，可能导致对局逻辑异常
   getPlayerChoice在用户输入无效时仅弹出提示，但没有返回有效选择，会导致playerChoice变为undefined，触发checkWinner的else分支，直接跳过分数计算。

修复后的完整代码 #

// 生成电脑选择，无需冗余参数
function getComputerChoice() {
    const n = Math.random() * 3;
    if (n <= 1) return "rock";
    else if (n <= 2) return "paper";
    else return "scissors";
};

// 获取玩家选择，循环处理无效输入直到用户输入正确
function getPlayerChoice() {
    while (true) {
        const input = prompt("Rock, Paper, or Scissors?").toLowerCase(); 
        if (["rock", "paper", "scissors"].includes(input)) {
            return input;
        }
        alert("输入无效，请重新输入！");
    }
};

let computerScore = 0; // 初始分数设为0更符合常规计分逻辑，若要从1开始可自行调整
let playerScore = 0;

// 直接操作全局分数，简化逻辑
function checkWinner(computer, player) {
    if (computer === player) {
        alert(`平局！双方都选了${computer}`);
    } else if (
        (computer === "rock" && player === "paper") ||
        (computer === "paper" && player === "scissors") ||
        (computer === "scissors" && player === "rock")
    ) {
        playerScore++;
        alert(`你赢了！${player}击败了${computer}`);
    } else {
        computerScore++;
        alert(`你输了！${computer}击败了${player}`);
    }
    // 实时显示当前分数
    alert(`当前分数：你 ${playerScore} - ${computerScore} 电脑`);
};

function checkWin() {
    if (playerScore === 6) {
        alert("恭喜你，最终获胜！");
        return true; // 返回true标记游戏结束
    } else if (computerScore === 6) {
        alert("很遗憾，你最终失败了！");
        return true;
    }
    return false;
};

// 游戏主循环
do {
    const playerChoice = getPlayerChoice();
    const computerChoice = getComputerChoice();
    checkWinner(computerChoice, playerChoice);
} while (!checkWin());

额外优化说明 #

• 移除了未使用的冗余参数，简化函数定义
• 重构checkWinner的判断逻辑，用条件组合替代大量重复的else if
• 增加无效输入的循环处理，确保玩家必须输入有效选择才会进入对局
• 实时显示当前分数，方便玩家查看进度
• 调整循环终止逻辑，当一方达到6分时直接结束游戏，避免多余的循环执行

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