C#酒店房间管理程序退房后无法重新入住空房问题求助

嘿，这个问题我之前帮不少开发者排查过！你遇到的酒店退房后没法再次入住空房的问题，核心根源确实出在checkIn()调用hasRoomsAvailable()时的逻辑漏洞上，大概率是房间状态的同步没做好。咱一步步来捋：

最可能的错误原因 #

这种问题通常逃不开这几种情况：

• hasRoomsAvailable()只盯着“已预订房间数”的计数判断，可checkOut()执行后没把这个计数减回去；
• 或者hasRoomsAvailable()根本没去检查房间的实际占用状态，而是依赖了过时的统计值；
• 还有可能checkIn()找空房的时候，没遍历实时的房间状态，直接被hasRoomsAvailable()的错误判断拦住了。

给你看个典型的错误代码示例，你对照下是不是和你的情况类似：

public class Hotel {
    private int totalRooms;
    private int bookedRooms;
    private boolean[] roomStatus; // true=已占用，false=空闲

    public Hotel(int totalRooms) {
        this.totalRooms = totalRooms;
        this.bookedRooms = 0;
        this.roomStatus = new boolean[totalRooms];
    }

    public boolean hasRoomsAvailable() {
        // 坑就在这：只看bookedRooms计数，没同步实际房间状态
        return bookedRooms < totalRooms;
    }

    public boolean checkIn() {
        if (!hasRoomsAvailable()) {
            return false;
        }
        // 找第一个空闲房间
        for (int i = 0; i < totalRooms; i++) {
            if (!roomStatus[i]) {
                roomStatus[i] = true;
                bookedRooms++;
                return true;
            }
        }
        return false;
    }

    public boolean checkOut(int roomNumber) {
        int index = roomNumber - 1;
        if (index < 0 || index >= totalRooms || !roomStatus[index]) {
            return false;
        }
        // 又一个坑：只改了房间状态，没更新bookedRooms计数
        roomStatus[index] = false;
        return true;
    }
}

你看，当所有房间都订满后，bookedRooms等于totalRooms，这时候退房，虽然房间状态改成了空闲，但bookedRooms没减，hasRoomsAvailable()还是返回false，直接把后续的checkIn()给拦了，哪怕有房间空着也没法住。

修复方案 #

有两种靠谱的修复方式，选哪种看你的需求：

方案1：让hasRoomsAvailable()实时检查房间状态 #

这种方式最稳妥，直接遍历房间状态数组，确认有没有空闲房间，完全不依赖计数：

public boolean hasRoomsAvailable() {
    for (boolean isOccupied : roomStatus) {
        if (!isOccupied) {
            return true;
        }
    }
    return false;
}

同时别忘了把checkOut()里的计数同步补上：

public boolean checkOut(int roomNumber) {
    int index = roomNumber - 1;
    if (index < 0 || index >= totalRooms || !roomStatus[index]) {
        return false;
    }
    roomStatus[index] = false;
    bookedRooms--; // 同步减少预订计数，避免其他依赖计数的逻辑出问题
    return true;
}

方案2：严格保证计数与房间状态同步 #

如果你不想改hasRoomsAvailable()的逻辑，那必须确保所有修改房间状态的操作（checkIn和checkOut）都同步更新bookedRooms，绝对不能遗漏任何场景。比如刚才的checkOut()里一定要加bookedRooms--这行。

验证修复效果 #

修复后你可以这么测试：

1. 创建一个只有1间房的酒店实例
2. 调用checkIn()，成功入住（这时候bookedRooms=1，roomStatus[0]=true）
3. 调用checkOut(1)，成功退房（bookedRooms=0，roomStatus[0]=false）
4. 再次调用checkIn()，这时候hasRoomsAvailable()会正确返回true，就能顺利入住空房了

这样问题就解决啦！

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