为何调用AddValidBooking存储过程始终显示餐桌已预订？

核心问题：参数名与列名冲突 #

你的存储过程输入参数booking_date、table_number和Bookings表的列名Booking_Date、Table_Number完全一致（MySQL默认不区分标识符大小写）。执行查询时，MySQL会将WHERE子句里的booking_date和table_number解析为表的列名，而非输入参数。

这导致判断条件变成：

DATE(Booking_Date) = DATE(Booking_Date) AND Table_Number = Table_Number

该条件永远为真，booked变量始终被赋值为TRUE，无论目标餐桌是否被预订，都会进入回滚分支，返回"table X is booked - booking canceled"。

解决方法：修改参数名避免冲突 #

给输入参数添加前缀（比如p_），明确区分参数和表列。修改后的存储过程代码如下：

DROP PROCEDURE IF EXISTS AddValidBooking;
DELIMITER //
CREATE PROCEDURE AddValidBooking(IN p_booking_date DATE, IN p_table_number INT)
BEGIN
    DECLARE booked BOOLEAN;

    START TRANSACTION;
    
    SELECT CASE WHEN EXISTS (
        SELECT 1
        FROM Bookings
        WHERE DATE(Booking_Date) = DATE(p_booking_date) AND Table_Number = p_table_number
    ) THEN TRUE ELSE FALSE END INTO booked;


    IF booked THEN
        ROLLBACK;
        SELECT CONCAT("table ", p_table_number, " is booked - booking canceled") AS booking_status;
    ELSE
        SELECT CONCAT("table ", p_table_number, " is free - booked successfully") AS booking_status;
        INSERT INTO Bookings (Booking_Date, Table_Number) VALUES (p_booking_date, p_table_number);
        COMMIT;
    END IF;
END //
DELIMITER ;

-- 调用示例
CALL AddValidBooking('2023-11-7', 2);

额外优化建议 #

• 给Bookings表添加唯一复合索引：UNIQUE KEY idx_booking_date_table (Booking_Date, Table_Number)，从数据库层面避免同一日期同一餐桌被重复预订，作为业务逻辑的双重保障。
• 存储过程中可添加异常处理，避免事务因意外情况长时间未提交/回滚。

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