Java问题：未满足if条件时，ArrayList仍总是移除房间对象

嘿，我来帮你捋捋这个问题～你遇到的核心问题很明确：你的预订检查逻辑没有判断时间段是否重叠，而是只要房间存在任何历史预订记录，就直接把它从可用列表里移除了——这就导致哪怕你要订的16:00-17:00和已有的14:00-15:00完全不冲突，也会被误判为占用。

常见错误逻辑示例 #

很多人刚开始写这个功能时，容易犯这样的错：

// 错误示例：只要房间有过预订就直接标记为占用
for (Booking existingBooking : allBookings) {
    if (existingBooking.getRoomId().equals(currentRoomId)) {
        // 直接移除房间，完全不检查时间
        availableRooms.remove(currentRoom);
        System.out.println("该房间已被占用");
        break;
    }
}

这段代码的问题在于：它只验证了「这个房间有没有被订过」，而完全没验证「当前要订的时间段和已有预订是否重叠」——只要房间有过任何预订，不管时间对不对得上，都会被判定为占用。

正确的时间段冲突判断逻辑 #

你需要先明确：两个时间段冲突的条件是「新预订的开始时间早于已有预订的结束时间，且新预订的结束时间晚于已有预订的开始时间」（简单说就是两个时间段有重叠部分）。

给你一个修正后的逻辑示例：

boolean isRoomConflicted = false;
LocalTime newBookStart = yourBookingRequest.getStartTime();
LocalTime newBookEnd = yourBookingRequest.getEndTime();

// 遍历该房间的所有已有预订
for (Booking existingBooking : allBookings) {
    if (existingBooking.getRoomId().equals(currentRoomId)) {
        LocalTime existingStart = existingBooking.getStartTime();
        LocalTime existingEnd = existingBooking.getEndTime();
        
        // 关键：检查两个时间段是否重叠
        if (newBookStart.isBefore(existingEnd) && newBookEnd.isAfter(existingStart)) {
            isRoomConflicted = true;
            break;
        }
    }
}

// 根据判断结果处理
if (isRoomConflicted) {
    System.out.println("该房间在选定时间已被预订");
} else {
    // 房间可用，保留在列表或处理预订逻辑
    availableRooms.add(currentRoom);
    // ... 其他预订操作
}

额外需要排查的点 #

除了核心逻辑问题，你还可以检查这几个细节：

• 是不是把时间当成字符串直接比较了？比如用"16:00".compareTo("15:00")这种方式，虽然简单但容易出问题，建议用LocalTime这类时间对象来做先后判断。
• 遍历预订记录时，是不是找到该房间的第一条预订就直接跳出循环了？哪怕后面的预订时间段其实不冲突？
• 有没有可能你的「可用房间列表」初始化逻辑有问题？比如每次检查前没有重新加载最新的可用状态？

总结一下：你现在的if语句之所以总是为true，是因为它判断的是「房间存在预订记录」，而不是「当前时间段和已有预订冲突」——只要把判断条件改成时间段重叠检查，问题就能解决啦。

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