如何在if-else中判断座位已占用？代码优化与逻辑修正咨询

一、正确实现座位占用判断逻辑 #

你当前代码的座位占用判断存在3个核心错误，修正后即可正常运行：

1. 数组索引计算错误 #

用户输入的座位行号（如1A中的1）是从1开始的，但C语言数组下标从0起始，因此行索引需用 p.seatrow - 1 转换；列号是字母（A-F），需用 p.seatcol - 'A' 转换成数组下标（比如'A'-'A'=0，'B'-'A'=1）。你原代码直接使用seat[p.seatrow][p.seatcol]，会导致索引越界或指向错误位置。

2. 判断逻辑颠倒 #

要判断座位是否已被占用，需检查对应位置是否等于'X'（你用'X'标记已预订），而非不等于'X'。原代码条件写反，导致逻辑完全错误。

3. 判断时机错误 #

正确的逻辑顺序应该是：先验证输入合法性→再检查座位是否已被占用→最后标记座位状态。原代码把占用判断放在大量if-else之后，逻辑顺序混乱。

修正后的chooseseat核心逻辑示例：

void chooseseat(){
    passenger p;
    printf("\n\t\t\tEnter your seat number: (EX: 1A)");
    scanf(" %d%c",&p.seatrow, &p.seatcol);

    // 1. 验证输入合法性：行号1-10，列号A-F
    if(p.seatrow < 1 || p.seatrow > 10 || p.seatcol < 'A' || p.seatcol > 'F'){
        printf("Invalid option!");
        chooseseat();
        return; // 递归调用后返回，避免执行后续代码
    }

    // 转换为数组索引
    int row_idx = p.seatrow - 1;
    int col_idx = p.seatcol - 'A';

    // 2. 检查座位是否已被占用
    if(seat[row_idx][col_idx] == 'X'){
        printf("\n\t\t Seat is already reserved. Choose another seat? (Y/N)");
        scanf(" %c", &answer);
        if(answer == 'Y' || answer == 'y'){ // 兼容大小写输入
            chooseseat();
        } else {
            printf("Your data will not be saved, returning to main menu...");
            return; // 禁止直接调用main()，避免栈溢出
        }
        return;
    }

    // 3. 标记座位为已预订
    seat[row_idx][col_idx] = 'X';
    seatavailable--;
    printf("Congratulations. Your seat number is %d%c", p.seatrow, p.seatcol);
}

二、用计算替换大量if-else分支 #

完全不需要for循环，直接通过行号和列号的数学转换就能替代所有冗余的if-else分支：

int row_idx = p.seatrow - 1;
int col_idx = p.seatcol - 'A';
seat[row_idx][col_idx] = 'X';

这一行代码就能替代你原代码中几十行的if-else判断，逻辑清晰且效率更高。

如果需要初始化座位数组（比如用'O'标记可用座位），可以用for循环完成：

void init_seats(){
    for(int i = 0; i < ROWS; i++){
        for(int j = 0; j < COLS; j++){
            seat[i][j] = 'O'; // 'O'表示可用，'X'表示已预订
        }
    }
}

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