本文详解如何用 java 构建一个结构清晰、逻辑正确的回合制骨牌游戏，解决“cpu 方法未执行”等常见控制流错误，通过单轮交替调用、状态统一管理及资源复用，确保玩家与计算机严格轮流行动直至分出胜负。
本文详解如何用 java 构建一个结构清晰、逻辑正确的回合制骨牌游戏，解决“cpu 方法未执行”等常见控制流错误，通过单轮交替调用、状态统一管理及资源复用，确保玩家与计算机严格轮流行动直至分出胜负。
在原始代码中，computer() 方法看似被调用（如 int computerScore = computer(10);），但实际并未实现“与玩家交替进行一局游戏”的核心逻辑——它被当作一次性独立计算执行，且其内部仍错误调用了 bonegame(10)，导致流程混乱、变量作用域错乱、循环条件失效（如 while (ron >= 0 || cpu <= 20) 永真），最终使 CPU 行为不可控、不响应、不参与真实对战。
要构建真正公平、可预测的回合制游戏，关键在于抽象“单轮行动”为可复用单元，并在主循环中严格序贯调度。以下是重构后的专业实践方案：
✅ 核心设计原则
- 单一职责：turn() 方法仅负责掷骰（生成随机图案并返回本轮得分），不处理输入、输出或胜负判断；
- 角色分离：turnPlayer() 封装用户交互（含重掷逻辑），turnComputer() 仅调用 turn() 并输出提示，体现 AI 的“无决策”本质；
- 资源复用：全局静态 Random 和 Scanner 实例避免重复创建，提升性能与随机性稳定性；
- 状态外置：分数（scorePlayer/scoreComputer）、重掷次数（rerolls）等状态由 main 循环统一维护，杜绝方法内局部变量遮蔽。
✅ 重构后完整可运行代码
import java.util.Random;
import java.util.Scanner;
public class BoneGame {
// 游戏常量 —— 语义化命名提升可维护性
private static final int POINTS_BLANK = 0;
private static final int POINTS_CHECKERED = 1;
private static final int POINTS_CIRCLE = 3;
private static final int POINTS_DIAMOND = 4;
private static final int POINTS_LINE = 1;
private static final int POINTS_TRIANGLE = 2;
private static final int MAX_REROLLS = 3;
private static final int TOTAL_ROUNDS = 10;
private static final int WINNING_SCORE = 20;
// 全局共享资源 —— 避免重复初始化
private static final Random RANDOM = new Random();
private static final Scanner STDIN = new Scanner(System.in);
// 游戏文本常量
private static final String BLANK = "blank";
private static final String CHECKERED = "checkered";
private static final String CIRCLE = "circle";
private static final String DIAMOND = "diamond";
private static final String LINE = "line";
private static final String TRIANGLE = "triangle";
// 状态变量（静态字段便于跨方法访问）
private static int rerolls = 0;
/**
* 执行一次掷骰行动：随机生成5种图案，累加对应分数
* @return 本轮获得的总分（0～11）
*/
private static int turn() {
int checkered = RANDOM.nextInt(2); // 0 or 1
int circle = RANDOM.nextInt(2);
int diamond = RANDOM.nextInt(2);
int line = RANDOM.nextInt(2);
int triangle = RANDOM.nextInt(2);
int score = 0;
if (checkered == 1) { System.out.println(CHECKERED); score += POINTS_CHECKERED; }
if (circle == 1) { System.out.println(CIRCLE); score += POINTS_CIRCLE; }
if (diamond == 1) { System.out.println(DIAMOND); score += POINTS_DIAMOND; }
if (line == 1) { System.out.println(LINE); score += POINTS_LINE; }
if (triangle == 1) { System.out.println(TRIANGLE); score += POINTS_TRIANGLE; }
if (score == POINTS_BLANK) {
System.out.println(BLANK);
}
return score;
}
/**
* 计算机回合：仅执行一次掷骰，无交互
*/
private static int turnComputer() {
System.out.println("====================================================================");
System.out.println("Computer turn...");
return turn();
}
/**
* 玩家回合：支持一次重掷机会（最多3次）
*/
private static int turnPlayer() {
System.out.println("====================================================================");
System.out.println("Player turn...");
int score = turn();
System.out.println("You scored: " + score);
if (rerolls < MAX_REROLLS) {
System.out.print("Roll again? [Y/N]: ");
String choice = STDIN.nextLine().trim();
if ("Y".equalsIgnoreCase(choice)) {
rerolls++;
System.out.println("(Rerolling...)");
score = turn();
} else {
System.out.println("Continuing to next round.");
}
} else {
System.out.println("You have no more rerolls.");
}
return score;
}
public static void main(String[] args) {
System.out.println("Welcome to the Bone Game — Created by the Indigenous Blackfoot Tribe");
System.out.println("====================================================================");
int scorePlayer = 0;
int scoreComputer = 0;
int round = 1;
while (round <= TOTAL_ROUNDS) {
System.out.printf("\nRound %2d:%n", round);
// 玩家行动
scorePlayer += turnPlayer();
System.out.println("Your current score: " + scorePlayer);
if (scorePlayer >= WINNING_SCORE) {
System.out.println("? Player wins!");
break;
}
// 计算机行动
scoreComputer += turnComputer();
System.out.println("Computer current score: " + scoreComputer);
if (scoreComputer >= WINNING_SCORE) {
System.out.println("? Computer wins!");
break;
}
round++;
}
// 轮次耗尽仍未分胜负 → 平局
if (round > TOTAL_ROUNDS && scorePlayer < WINNING_SCORE && scoreComputer < WINNING_SCORE) {
System.out.println("? It's a tie!");
}
STDIN.close(); // 善始善终：关闭 Scanner
}
}⚠️ 关键注意事项与最佳实践
- 勿在循环内创建 Scanner：原代码中 bonegame() 内多次 new Scanner(System.in) 会导致输入流竞争与 NoSuchElementException，必须全局唯一实例；
- 避免魔数（Magic Numbers）：使用 private static final int 常量替代硬编码数字（如 20, 3, 10），大幅提升可读性与后期调整效率；
- 重掷逻辑需显式反馈：玩家选择重掷后应有明确提示（如 (Rerolling...)），增强用户体验；
- 边界条件全覆盖：main 中 if (round > TOTAL_ROUNDS) 判断确保平局逻辑不被胜利分支跳过；
- 资源释放：程序结束前调用 STDIN.close() 是良好习惯，尤其在生产环境防止资源泄漏。
该实现不仅修复了“CPU 方法不运行”的根本缺陷，更以模块化、低耦合、高内聚的设计范式，为后续扩展（如难度分级、AI 策略、图形界面）奠定了坚实基础。
