本文详解如何用 while 循环替代嵌套 for 循环，安全、健壮地实现用户从动态变化的神祇列表中重复选取 6 个不重复 id 的对象，并实时移除已选项，避免索引越界与逻辑错判。
在 Java 游戏模拟器类项目中，常见的需求是让用户从一个初始加载的神祇（God）列表中手动挑选固定数量（如 6 个）且互不重复的对象，每选中一个后需立即将其从候选池中移除，防止重复选择。原始代码中使用 for (int i = 0; i <= 5; i++) 外层循环 + 内层遍历匹配的结构存在根本性缺陷：
- ❌ 内层 else 分支逻辑错误：只要当前遍历位置的 id 不匹配，就立即触发“重输”提示——这导致即使目标 ID 存在于列表靠后位置，也会在检查第一个元素时误判为无效；
- ❌ 边遍历边删除引发索引偏移：在 for (j = 0; j < list.size(); j++) 中调用 list.remove(j) 会导致后续元素前移，但循环变量 j 仍自增，造成跳过下一个元素或 IndexOutOfBoundsException；
- ❌ 缺乏重复选择校验：未检查用户是否已选过该 ID（即是否已在 selectedGods 中存在），违背“不可重复选取”需求。
✅ 正确解法是放弃固定次数的 for 循环，改用条件驱动的 while 循环，以 selectedGods.size() < 6 为唯一终止条件，并将“查找+验证+移动”逻辑封装为独立方法，提升可读性与健壮性。
✅ 推荐实现方案
首先，添加一个通用查找工具方法（建议置于 Player 类中）：
/**
* 在指定神祇列表中查找 id 匹配的 God 对象索引
* @param godList 待搜索的 God 列表
* @param targetId 目标 id 值
* @return 匹配项的索引（>=0），未找到返回 -1
*/
private int findGodById(List<God> godList, int targetId) {
for (int i = 0; i < godList.size(); i++) {
if (godList.get(i).getId() == targetId) {
return i;
}
}
return -1;
}然后，在 selectGodsForTeam() 方法中重构核心选择逻辑：
void selectGodsForTeam() {
Scanner scanner = new Scanner(System.in);
System.out.println("请依次输入您要选择的 6 位神祇的 ID（每次输入后按回车）：");
while (selectedGods.size() < 6) {
System.out.print("您已选择 " + selectedGods.size() + "/6 位神祇。请输入下一位神祇的 ID > ");
int chooseGodId;
try {
chooseGodId = scanner.nextInt();
} catch (InputMismatchException e) {
System.out.println("❌ 输入错误：请输入有效的整数 ID！");
scanner.nextLine(); // 清空非法输入
continue;
}
// 检查是否已选（防重复）
if (findGodById(selectedGods, chooseGodId) >= 0) {
System.out.println("⚠️ 提示：ID " + chooseGodId + " 的神祇已被选中，请选择其他神祇。");
continue;
}
// 检查是否在候选池中存在
int indexInPool = findGodById(listOfAllGods, chooseGodId);
if (indexInPool == -1) {
System.out.println("❌ 错误：ID " + chooseGodId + " 的神祇不存在或已被选走，请重新输入。");
continue;
}
// 安全添加并移除
God selectedGod = listOfAllGods.get(indexInPool);
selectedGods.add(selectedGod);
listOfAllGods.remove(indexInPool); // 此时 indexInPool 是有效索引，无偏移风险
System.out.println("✅ 成功选择：" + selectedGod.getName() + "（ID：" + chooseGodId + "）");
}
System.out.println("\n? 神祇队伍组建完成！共选择 " + selectedGods.size() + " 位神祇。");
}? 关键设计说明
- 单一层级查找：findGodById() 一次性遍历并返回索引，避免嵌套循环中“一不匹配就报错”的误判；
- 双重校验机制：先查 selectedGods（防重复），再查 listOfAllGods（防无效 ID），语义清晰；
- 安全移除：使用 remove(int index) 而非 remove(Object)，确保移除的是当前匹配项，且因查找与删除分离，无并发修改风险；
- 异常防护：捕获 InputMismatchException 处理非数字输入，提升用户体验；
- 即时反馈：每次成功/失败均给出明确提示，符合交互式 CLI 应用最佳实践。
⚠️ 注意事项
- 若 God 类未重写 equals() 和 hashCode()，请勿使用 list.contains(new God(...)) 进行重复判断，应始终基于 id 字段比对；
- Scanner 实例建议在类级别复用或显式关闭（本例中因生命周期短可忽略，生产环境建议 try-with-resources）；
- 后续扩展时，可将 findGodById() 抽离为静态工具方法，或改用 Java 8+ 的 Stream API（如 list.stream().filter(g -> g.getId() == id).findFirst()），但需注意性能与可读性权衡。
此方案彻底规避了原始逻辑中的索引错乱、提前中断和重复漏洞，是处理“用户交互式动态集合筛选”场景的标准实践。
