本文详解如何用 while 循环替代嵌套 for 循环，安全、可靠地实现用户从动态变化的 god 对象列表中重复选择 6 个唯一神祇，并实时移除已选项，避免索引越界与重复校验失效问题。
在开发类似《Battle Simulator》这类基于角色/神祇选择的策略游戏时，一个常见但易出错的需求是：让用户从初始神祇池中交互式选取固定数量（如 6 个）唯一神祇，每选一个即从可用池中移除，且需持续提示直至输入有效 ID。原始代码中使用 for (int i = 0; i
-
✅ 问题根源：
- 内层循环对每个 j 执行 else 分支中的 while (!isGodAvailable)，导致只要当前遍历位置不匹配，就立即进入死循环询问新 ID，完全忽略后续位置可能存在的匹配项；
- 在 listOfAllGods.remove(j) 后未 break，继续遍历会导致 ConcurrentModification 风险或跳过元素（因列表长度动态缩短而索引错位）；
- getSelectedGods() 方法错误返回 listOfAllGods 而非 selectedGods，暴露逻辑混乱。
-
✅ 正确设计原则：
- 外层用 while (selectedGods.size() ：以目标状态（选够 6 个）为循环条件，而非固定次数，自然支持重试；
- 内层用独立查找方法 findGod()：封装线性搜索逻辑，返回索引或 -1，解耦验证与操作；
- 两次校验缺一不可：既要检查 ID 是否存在于 listOfAllGods（可用性），也要检查是否已在 selectedGods 中（防重复提交）；
- 成功后立即 add() + remove() 并 break 或 continue：确保单次输入只触发一次状态变更。
以下是重构后的 selectGodsForTeam() 方法完整实现（含辅助方法）：
立即学习“Java免费学习笔记（深入）”；
import java.util.*;
public class Player {
private List<God> listOfAllGods;
private List<God> selectedGods;
Player(List<God> listOfAllGods) {
this.listOfAllGods = new ArrayList<>(listOfAllGods); // 防止外部修改影响
this.selectedGods = new ArrayList<>();
}
public List<God> getSelectedGods() { // ✅ 修正 getter
return new ArrayList<>(selectedGods); // 返回不可变副本更安全
}
void selectGodsForTeam() {
Scanner scanner = new Scanner(System.in);
System.out.println("Please choose the 6 id's of the gods you wish to pick:");
while (selectedGods.size() < 6) {
System.out.print("You have selected " + selectedGods.size()
+ "/6 gods. Enter God ID > ");
int chooseGodId;
try {
chooseGodId = scanner.nextInt();
} catch (InputMismatchException e) {
System.out.println("Invalid input. Please enter a number.");
scanner.nextLine(); // 清空错误输入
continue;
}
// 检查是否已选（防重复）
if (findGod(selectedGods, chooseGodId) >= 0) {
System.out.println("❌ God ID " + chooseGodId + " is already selected. Try another.");
continue;
}
// 检查是否可用（存在于剩余池中）
int idx = findGod(listOfAllGods, chooseGodId);
if (idx == -1) {
System.out.println("❌ God ID " + chooseGodId + " is not available. Try another.");
continue;
}
// ✅ 安全添加并移除
selectedGods.add(listOfAllGods.get(idx));
listOfAllGods.remove(idx); // ArrayList.remove(int) 时间复杂度 O(n)，可接受
System.out.println("✅ Added: " + selectedGods.get(selectedGods.size() - 1).getName());
}
System.out.println("\n? Selection complete! Your team: ");
selectedGods.forEach(god ->
System.out.println("- " + god.getName() + " (ID: " + god.getId() + ")")
);
}
// ? 核心查找工具方法：返回匹配 God 的索引，未找到返回 -1
private int findGod(List<God> godList, int targetId) {
for (int i = 0; i < godList.size(); i++) {
if (godList.get(i).getId() == targetId) {
return i;
}
}
return -1;
}
}- ⚠️ 关键注意事项：
- 不要在遍历 ArrayList 时调用 remove() —— 原始嵌套循环正是因此崩溃。本方案先查索引，再统一操作，彻底规避风险；
- 使用 new ArrayList(original) 初始化副本，防止外部意外修改影响游戏逻辑；
- 增加 InputMismatchException 处理，提升健壮性（用户输字母时不会崩溃）；
- 若神祇数量极大（>10⁴），建议将 listOfAllGods 替换为 Map
以实现 O(1) 查找，但当前 12 个神祇无需过度优化。
通过此重构，用户交互流程变得清晰可控：每次输入仅触发一次查找 → 一次判断 → 一次状态更新。既满足“选满 6 个为止”的业务目标，又保证了数据一致性与用户体验的友好性。
