JavaScript游戏函数执行后返回undefined问题求助

核心问题：返回undefined的原因 #

你的game函数最后返回undefined，是因为循环结束后gameRound的值是5，但你判断的是gameRound == 4，这个条件永远不成立，函数没有进入这个分支，也就没有返回任何值，所以最终输出undefined。

修复步骤 #

1. 修正循环结束后的判断条件：把if(gameRound == 4)改成if(gameRound == 5)，因为循环执行5次后，gameRound会从0递增到5，此时才是循环结束的状态。
2. 优化变量作用域（可选但推荐）：把gameRound、playerScore、computerScore这三个变量移到game函数内部，避免全局变量污染，每次调用game都会重置这些值，更符合函数的独立性。
3. 处理平局回合（可选）：当前代码中平局时会消耗回合次数，如果你希望平局不算有效回合，可以调整循环逻辑，只有非平局时才增加回合数。

修复后的完整代码 #

const getComputerChoice = () => {
    let randNum = Math.floor(Math.random()*3);
    return randNum === 0 ? "rock"
         : randNum === 1 ? "scissor"
         : "paper";
}

const playRound = (playerSelection, computerSelection) => {
    if (playerSelection === "rock" && computerSelection === "paper") {
        computerScore++;
        return "Paper beats rock, computer win";
    } else if (playerSelection === "rock" && computerSelection === "scissor") {
        playerScore++;
        return "Rock beats scissor, you win!";
    } else if (playerSelection === "paper" && computerSelection === "rock") {
        playerScore++;
        return "Paper beats rock, you win";
    } else if (playerSelection === "paper" && computerSelection === "scissor") {
        computerScore++;
        return "Scissor beat paper, computer win";
    } else if (playerSelection === "scissor" && computerSelection === "rock") {
        computerScore++;
        return "Rock beats scissor, computer win";
    } else if (playerSelection === "scissor" && computerSelection === "paper") {
        playerScore++;
        return "Scissor beats paper, you win!";
    } else {
        return "Same choice. Try again!";
    }
}

const game = () => {
    let gameRound = 0;
    let playerScore = 0;
    let computerScore = 0;
    
    while (gameRound < 5) {
        const computerChoice = getComputerChoice();
        const playerInput = prompt("Pick paper, scissor, or rock : ");
        const playerChoice = playerInput.toLowerCase();
        const roundResult = playRound(playerChoice, computerChoice);
        console.log(roundResult);
        // 非平局时才增加回合数（可选，根据需求调整）
        if (!roundResult.includes("Try again")) {
            gameRound++;
        }
    }
    
    if (playerScore > computerScore) {
        return `You win! Your score : ${playerScore} Computer score : ${computerScore}`;
    } else if (computerScore > playerScore) {
        return `You lose! Your score : ${playerScore} Computer score : ${computerScore}`;
    } else {
        return "It's a tie!";
    }
}

console.log(game());

关于输出逻辑是否需要移到函数外 #

不需要将结果输出逻辑移到函数外。保持函数返回结果、外部用console.log打印的方式是合理的，这样函数职责更清晰：game函数负责计算游戏结果并返回，外部负责输出。只要修复判断条件的问题，就能正常返回结果并打印。

内容的提问来源于stack exchange，提问作者Raymond Sam Chia