餐厅星级提升时厨师涨薪触发器可行性及优化咨询

首先直接给出结论：你的这段触发器代码不可行，存在几个关键问题，我来逐一拆解并给出修正和优化方案：

原代码的核心问题 #

• 字段引用错误：
  1. 判断星级提升时，你用了NEW.stars > stars，这里的stars没有指定是更新前的记录值，应该用OLD.stars（OLD代表更新前的Restaurant记录，NEW是更新后的记录）。
  2. 你试图更新Chef表的salary字段，但根据你的表结构，salary实际存储在CooksFor关联表里，Chef表并没有这个字段，这会直接导致语法错误。
• 关联逻辑错误：
  在EXISTS子查询里，Restaurant.rname的写法无效，你需要引用当前触发更新的餐厅名称，应该用OLD.rname（如果餐厅名称不会被修改，用NEW.rname也可以，毕竟本次更新的是stars字段）。
• 语法不完整：
  触发器里的IF块结尾、UPDATE语句结尾都缺少分号，这会导致数据库解析失败。

修正后的可行代码 #

修复上述问题后，得到能正常运行的触发器：

CREATE TRIGGER salary_increase 
BEFORE UPDATE ON Restaurant 
FOR EACH ROW
BEGIN
  -- 仅当星级确实提升时执行薪资调整
  IF NEW.stars > OLD.stars THEN
    -- 更新CooksFor表中对应餐厅的厨师薪资
    UPDATE CooksFor CF
    SET CF.salary = CF.salary * 1.1
    WHERE CF.rname = OLD.rname; -- 关联当前更新的餐厅
  END IF;
END;

更优的实现方式 #

如果想要逻辑更清晰，或者需要关联Chef表做额外判断（比如只给特定评级的厨师涨薪），可以用JOIN替代EXISTS，可读性更强：

CREATE TRIGGER salary_increase 
BEFORE UPDATE ON Restaurant 
FOR EACH ROW
BEGIN
  IF NEW.stars > OLD.stars THEN
    UPDATE CooksFor CF
    JOIN Chef C ON CF.cname = C.cname
    SET CF.salary = CF.salary * 1.1
    WHERE CF.rname = OLD.rname
    -- 这里可以加额外条件，比如只给评级>=4的厨师涨薪
    -- AND C.rating >= 4;
  END IF;
END;

另外补充一点：如果你的数据库支持AFTER UPDATE触发器，用它也完全没问题，因为这里的操作不依赖于修改Restaurant的过程，两种触发时机都能达到效果，可根据个人习惯选择。

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