JavaScript中round()函数无法返回值传递给play()函数问题

核心问题分析 #

你的代码有三个关键错误导致round函数返回undefined，且play函数逻辑混乱：

• 未保存函数调用结果：round函数里调用了getPlayerChoice()和getComputerChoice()，但没有把返回值存入变量，后续直接拿函数本身和字符串比较（比如getComputerChoice === "Rock"），这永远不成立，所以round函数没有进入任何分支，自然返回undefined。
• 重复调用round函数：play函数里每个if条件都调用一次round()，这会导致每次判断都重新弹出输入框和电脑选择提示，完全打乱游戏流程。
• 胜负判断逻辑对应错误：你定义的规则是-1=玩家输，0=平局，1=玩家赢，但play里的判断把0对应成了“lose”，和规则完全不符。

修复后的完整代码 #

// 石头剪刀布游戏
// 人机对战
// 电脑随机选择
// 玩家输入选择
// 单轮对战
// 结果判定

// 电脑选择函数
function getComputerChoice() {
  let value = Math.floor(Math.random() * 3);
  if (value === 0) {
    alert('我选石头');
    return "Rock";  
  } else if (value === 1) {
    alert('我选布');
    return "Paper";
  } else {
    alert('我选剪刀');
    return "Scissor";
  }
}

// 处理玩家输入，统一格式（首字母大写，其余小写）
function getPlayerChoice() {
  let rawinput = prompt("请输入你的选择：石头、布或剪刀。");
  // 处理用户取消输入的情况，避免报错
  if (!rawinput) {
    alert('请输入有效选项！');
    return getPlayerChoice();
  }
  let input = rawinput.charAt(0).toUpperCase() + rawinput.slice(1).toLowerCase();
  if (input === "Rock") {
    alert("玩家选择石头");
  } else if (input === "Paper") {
    alert("玩家选择布");
  } else if (input === "Scissor") {
    alert("玩家选择剪刀");
  } else {
    alert('输入无效，请重新输入！');
    return getPlayerChoice();
  }
  return input;
}

// 单轮对战函数
// 返回值规则：-1=玩家输，0=平局，1=玩家赢
function round() {
  // 保存玩家和电脑的选择结果
  const playerChoice = getPlayerChoice();
  const computerChoice = getComputerChoice();

  // 根据选择判定结果
  if (computerChoice === "Rock") {
    if (playerChoice === "Rock") {
      return 0;
    } else if (playerChoice === "Paper") {
      return 1;
    } else if (playerChoice === "Scissor") {
      return -1;
    }
  } else if (computerChoice === "Paper") {
    if (playerChoice === "Rock") {
      return -1;
    } else if (playerChoice === "Paper") {
      return 0;
    } else if (playerChoice === "Scissor") {
      return 1;
    }
  } else if (computerChoice === "Scissor") {
    if (playerChoice === "Rock") {
      return 1;
    } else if (playerChoice === "Paper") {
      return -1;
    } else if (playerChoice === "Scissor") {
      return 0;
    }
  }
}

// 游戏主函数
function play() {
  // 只调用一次round，保存结果
  const roundResult = round();
  if (roundResult === 0 ) {
    alert('平局！');
    return "没人获胜";
  } else if (roundResult === 1) {
    alert('你赢了！');
    return "大吉大利，今晚吃鸡！";
  } else if (roundResult === -1) {
    alert('你输了！');
    return "电脑赢了，你得加油了";
  }
}

play();
console.log(round());

关键修改说明 #

• 保存函数返回值：在round函数里，把getPlayerChoice()和getComputerChoice()的结果分别存入playerChoice和computerChoice变量，后续用变量做比较，确保逻辑能进入对应分支。
• 避免重复调用round：play函数里先调用一次round并把结果存入roundResult变量，后续所有判断都基于这个变量，不会重复触发输入和提示。
• 修正胜负对应逻辑：把play里的判断和你定义的返回值规则对应起来，0对应平局，1对应胜利，-1对应失败。
• 增加输入校验：在getPlayerChoice里处理用户取消输入或输入无效的情况，递归调用自身直到用户输入有效选项，避免程序出错。

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