桌游数据库触发器失效求助：Players表更新触发逻辑修复

原代码的核心问题 #

• NULL判断逻辑错误：SQL中= NULL永远不会返回真，因为NULL是未知值，必须用IS NULL来判断字段是否为空。
• 冗余且不安全的条件校验：原条件NEW.on_location = (SELECT location_id FROM Real_estate WHERE location_id = NEW.on_location)完全多余，且当location不存在时，子查询返回NULL，比较结果为NULL，无法正确触发条件；应使用EXISTS来校验location_id的存在性。
• 不必要的子查询引发风险：更新Real_estate的owned_by时，无需通过子查询获取player_id——触发器针对当前更新的玩家记录，NEW.player_id就是当前玩家的ID，若多个玩家处于同一位置，子查询会返回多行，直接报错。
• Players表更新逻辑冗余且危险：通过on_location查询玩家ID可能导致逻辑混乱（多玩家同位置时无法定位目标），且AFTER UPDATE触发器二次更新Players表可能触发递归触发器。

修正后的触发器代码 #

DROP TRIGGER IF EXISTS R1;
DELIMITER ;; 
CREATE TRIGGER R1 BEFORE UPDATE ON Players FOR EACH ROW
BEGIN
    -- 一次性校验：location存在于Real_estate且无所有者
    IF EXISTS (SELECT 1 FROM Real_estate WHERE location_id = NEW.on_location AND owned_by IS NULL) THEN
        -- 将对应地产的所有者设置为当前玩家
        UPDATE Real_estate
        SET owned_by = NEW.player_id
        WHERE location_id = NEW.on_location;
        
        -- 直接修改当前玩家的余额（BEFORE触发器可直接修改NEW字段值，无需额外UPDATE）
        SET NEW.player_balance = NEW.player_balance - (SELECT estate_cost FROM Real_estate WHERE location_id = NEW.on_location);
    END IF;
END;;
DELIMITER ;

关键修改说明 #

1. 改用BEFORE UPDATE触发器：直接修改NEW.player_balance，避免对Players表的二次更新，彻底杜绝递归触发的可能。
2. 用EXISTS合并校验逻辑：一次性完成location存在性和无所有者的校验，逻辑更简洁，性能更优。
3. 直接使用NEW.player_id：无需子查询，确保操作的是当前触发触发器的玩家，避免多玩家同位置时的错误。
4. 修正NULL判断为IS NULL：符合SQL语法规范，确保条件能正确触发。

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