Java预约改期功能for循环错误触发else语句问题排查

问题根因 #

你当前的逻辑错误在于：

• 把「无匹配预约」的提示放在了循环内部的else分支里，每遍历一条不匹配的记录就会触发一次提示，所以第一条ID为32456的记录不匹配时，就先输出了无预约的错误提示
• 没有标记是否找到匹配的记录，无法在遍历完全部数据后再统一判断是否存在对应预约
• 未处理同一个用户存在多条预约的场景，当前逻辑会把该用户所有匹配的预约都走一遍改期流程

修复方案 #

调整逻辑为：先遍历全量预约记录，用标记位记录是否找到匹配项，遍历结束后再判断是否需要输出无预约提示。修复后的代码如下：

static void rescheduleAppointment()
{
    System.out.print("Enter Patient ID: ");
    int id = input.nextInt();
    input.nextLine();
    // 新增标记位，记录是否找到匹配的预约
    boolean hasAppointment = false;

    for (int i = 0; i < schedule.size(); i++)
    {
        if (id == schedule.get(i).getPatientID())
        {
            hasAppointment = true;
            System.out.println("The coming scheduled appointment for " + schedule.get(i).getPatientName() + "(ID#: " + schedule.get(i).getPatientID() + ") on " + schedule.get(i).getAppointmentTime());

            System.out.print("Enter the Appointment new Date and Time as dd-mm-yyyy hh:mm : ");
            String dateAndTime = input.nextLine();
            schedule.get(i).setAppointmentTime(dateAndTime);

            System.out.println("Appointment has been updated\n");
            // 如果只需要修改最新的一条预约，可以在这里加break跳出循环
            // break;
        }
    }
    
    // 遍历完所有记录后再判断是否无匹配预约
    if (!hasAppointment) {
        System.out.println("No coming appointment found for " + id +".");
        System.out.println("You might need to schedule a new appointment.\n");
    }
}

修改点说明 #

• 新增布尔类型的hasAppointment标记位，初始值为false，匹配到对应ID的预约时设置为true
• 把原来循环内的else分支删除，移到循环结束后统一判断，只有遍历完全部记录都没有匹配项时才输出无预约提示
• 额外增加了可选的break注释，如果业务场景只需要修改用户的某一条预约，匹配到之后直接跳出循环即可，避免遍历剩余的记录

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