修复Java JOptionPane选择第二支球队时方法被跳过的问题

需求背景 #

开发一个输入对话框，用于提示用户选择与第一支球队对阵的第二支球队，要求：

• 已选的第一支球队不显示在可选列表中
• 对用户输入非球队选项的情况进行错误校验
• 已完成第一支球队的选择方法，使用Team类从数据文件获取球队名称，League为包含球队数据的数组，当前方法接收league数组和已选的team1，需返回第二支球队的选择结果。
• 预期输出：提示用户选择对阵球队的对话框，其中已选的第一支球队不会出现在可选列表中。

原代码运行时该选择方法被完全跳过，无法实现预期功能。

原代码核心问题 #

1. 循环逻辑错误：外层for循环无意义，且while循环条件!team2.equalsIgnoreCase(league[i].getName()) && team2.equals(team1)存在逻辑矛盾——既要求输入不是当前遍历的球队，又要求输入等于已选的team1，同时team2初始值可能未定义，直接导致循环根本不会执行。
2. 硬编码扩展性差：通过多个else if硬编码处理4支球队，后续球队数量变化时需要大量修改代码。
3. 输入校验缺失：没有将对话框输入赋值给team2，也未对用户输入的合法性做完整校验。

修复后的代码 #

public Team selectSecondTeam(Team[] league, Team team1) {
    // 构建可选球队的提示文本，自动排除已选的team1
    StringBuilder prompt = new StringBuilder("Select a team to play the " + team1.getName() + "\n\n");
    for (Team team : league) {
        if (!team.getName().equalsIgnoreCase(team1.getName())) {
            prompt.append("- ").append(team.getName()).append("\n");
        }
    }

    Team selectedTeam = null;
    while (selectedTeam == null) {
        String userInput = JOptionPane.showInputDialog(prompt.toString());
        
        // 处理用户取消输入的场景
        if (userInput == null) {
            JOptionPane.showMessageDialog(null, "Input cancelled.");
            return null;
        }

        // 校验输入是否为合法球队
        boolean inputValid = false;
        for (Team team : league) {
            if (userInput.equalsIgnoreCase(team.getName())) {
                inputValid = true;
                // 校验是否选择了同一支球队
                if (team.getName().equalsIgnoreCase(team1.getName())) {
                    JOptionPane.showMessageDialog(null, "Cannot select the same team as the first one. Please try again.");
                } else {
                    selectedTeam = team;
                }
                break;
            }
        }

        // 输入不是合法球队的情况
        if (!inputValid) {
            JOptionPane.showMessageDialog(null, "Sorry, I couldn't find '" + userInput + "', please try again...");
        }
    }
    return selectedTeam;
}

关键改进点 #

• 动态生成可选列表：通过遍历league数组自动生成排除team1的提示文本，适配任意数量的球队，无需硬编码。
• 正确的循环校验逻辑：用while循环持续弹出对话框，直到用户输入有效选项（合法球队且非team1）。
• 完善的边界处理：覆盖用户取消输入的场景，避免空指针异常。
• 清晰的错误提示：针对“重复选同一队”和“输入非法球队”分别给出明确提示，提升用户体验。

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