触发优化

Question

如何优化此触发器？它工作得很好，但我认为它可以改进：

CREATE TRIGGER `update_request_missions_after_insert` BEFORE INSERT ON `request_missions`
 FOR EACH ROW
IF ((SELECT id FROM request_missions ORDER BY id DESC LIMIT 1)+1) >= 1  AND 
   ((SELECT id FROM request_missions ORDER BY id DESC LIMIT 1)+1) <= 9 THEN
        SET New.rm_number = CONCAT(DATE_FORMAT(CURDATE(),'%y'), '-',
            CONCAT('00000',(SELECT id FROM request_missions ORDER BY id DESC LIMIT 1)+1));
    END IF

结果将在rm_number 列中，例如：

对于 id >=1 和 id

19-000001
19-000002
...
19-000009

Answer 1

感谢 M. Bill Karwin，代码现在更干净了 对于今年的问题，我将其更改为 '%Y' 。 现在的代码是：

CREATE TRIGGER `update_request_missions_after_insert` BEFORE INSERT ON `request_missions`
FOR EACH ROW
BEGIN
  DECLARE next_id INT;
  SELECT COALESCE(MAX(id),0) + 1 INTO next_id FROM request_missions;
  IF next_id BETWEEN 1 AND 9 THEN
    SET NEW.rm_number = CONCAT(DATE_FORMAT(CURDATE(),'%Y'), '-', '00000', next_id);
  ELSEIF next_id BETWEEN 10 AND 99 THEN
    SET NEW.rm_number = CONCAT(DATE_FORMAT(CURDATE(),'%Y'), '-', '0000', next_id);
  ELSEIF next_id BETWEEN 100 AND 999 THEN
    SET NEW.rm_number = CONCAT(DATE_FORMAT(CURDATE(),'%Y'), '-', '000', next_id);
  ELSEIF next_id BETWEEN 1000 AND 9999 THEN
    SET NEW.rm_number = CONCAT(DATE_FORMAT(CURDATE(),'%Y'), '-', '00', next_id);
  ELSEIF next_id BETWEEN 10000 AND 99999 THEN
    SET NEW.rm_number = CONCAT(DATE_FORMAT(CURDATE(),'%Y'), '-', '0', next_id);
  ELSEIF next_id BETWEEN 100000 AND 999999 THEN
    SET NEW.rm_number = CONCAT(DATE_FORMAT(CURDATE(),'%Y'), '-', next_id);
  END IF;
END

警告：这会受到竞争条件的影响。例如，如果两个 会话同时插入到表中，它们可能都 读取相同的 next_id。

知道此触发器的目的是为添加的每一行添加一个格式为“YYYY-ID”的数字，是否有解决此问题的方法。 YYYY：是当前年份 ID : 是当前行的id，格式如触发器所示

Answer 2

这是使用局部变量而不是重复子查询三次的建议。你说那是你关心的问题。

CREATE TRIGGER `update_request_missions_after_insert` BEFORE INSERT ON `request_missions`
FOR EACH ROW
BEGIN
  DECLARE next_id INT;
  SELECT COALESCE(MAX(id),0) + 1 INTO next_id FROM request_missions;
  IF next_id BETWEEN 1 AND 9 THEN
    SET NEW.rm_number = CONCAT(DATE_FORMAT(CURDATE(),'%y'), '-', '00000', next_id);
  END IF;
END

警告：这会受到竞争条件的影响。例如，如果两个会话同时插入到表中，它们可能都读取相同的next_id。

正如@P.Salmon 上面评论的那样，如果next_id > 9，则结果未定义。我不确定您要做什么。

您可能希望允许更长的数字，并用零将它们填充为固定长度：

LPAD(next_id, 6, '0')

请参阅LPAD() 函数的手册。

我使用COALESCE(MAX(id),0) 确保至少返回一个非 NULL 值，以防表有零行。

这也涉及到：

DATE_FORMAT(CURDATE(),'%y')

两位数的年份将在 81 年后滚动到 00。这正是千年虫问题的产生方式！我知道到那时你不会处理代码，所以你可能不在乎。但是，您应该始终像最终维护您的代码的人是一个知道您住在哪里的暴力精神病患者一样编写代码。

---

在另一个答案中发布您的新代码：

这是处理高达 999,999 的 next_id 数字的更好解决方案。

CREATE TRIGGER `update_request_missions_after_insert` BEFORE INSERT ON `request_missions`
FOR EACH ROW
BEGIN
  DECLARE next_id INT;
  SELECT COALESCE(MAX(id),0) + 1 INTO next_id FROM request_missions;
  IF next_id < 1000000 THEN
    SET NEW.rm_number = CONCAT(DATE_FORMAT(CURDATE(),'%y'), '-', LPAD(next_id, 6, '0'));
  ELSE
    SIGNAL SQLSTATE '45000' SET MESSAGE_TEXT = 'RM Number has reached capacity'
  END IF;
END

竞态问题不同，当您使用MAX(id) 查找最后一个id 时没有解决方案。问题是您可能有多个客户端同时插入到表中。每个客户端的会话都会为MAX(id) 读取相同的值并使用它。

你应该阅读Race Conditions。

您真正需要的是某种使用AUTO_INCREMENT 值的方法，该值以线程安全的方式生成新的唯一值，并将该值复制到您的rm_number 中。

很遗憾，无法在触发器中执行此操作。在 BEFORE INSERT 触发期间，尚未生成新的 AUTO_INCREMENT 值。在 AFTER INSERT 触发期间，您无法修改 NEW.rm_number，因为插入已经完成。我过去写过这个：

• Concatenating a string and primary key Id while inserting
• Two autoincrements columns or autoincrement and same value in other column