如何防止更新表，但有一种情况除外

Question

我有一个表，其中包含可以成为账单一部分的记录。我可以分辨出哪些已经是账单的一部分，因为该表有一个 BillId 列，当发生这种情况时，应用程序代码会更新该列。我想阻止更新具有非空 BillId 的任何记录。我认为以下应该解决这个问题：

CREATE TRIGGER [Item_Update_AnyBilled]
ON [dbo].[Item]
FOR UPDATE
AS 
BEGIN
SET NOCOUNT ON;
DECLARE @AnyBilled BIT;

SELECT  TOP(1) @AnyBilled = 1
  FROM  inserted i
        JOIN deleted d ON i.ItemId = d.ItemId
 WHERE  d.BillId IS NOT NULL; 

IF COALESCE(@AnyBilled, 0) = 1 BEGIN
    RAISERROR(2870486, 16, 1);  -- Cannot update a record that is part of a bill.
    ROLLBACK TRANSACTION;
END;
END;

但是，还有一个问题。 Item 表还有一个 DATETIME Modified 列，以及一个更新它的触发器。

CREATE TRIGGER [dbo].Item_Update_Modified 
ON  [dbo].[Item] 
AFTER UPDATE
AS 
BEGIN
SET NOCOUNT ON;

UPDATE a
   SET Modified = getdate()
  FROM Item a JOIN inserted i ON i.ItemId = a.ItemId
END

使用这些触发器，将项目添加到账单总是会导致 RAISERROR 触发。大概是因为当 BillId 被填充时，Item_Update_AnyBilled 允许它通过，因为 deleted.BillId 为 NULL，但 Item_Update_Modified 然后被执行，并且该次要更改导致 Item_Update_AnyBilled 再次被执行，而这一次 deleted.BillId 不再为 NULL。

我如何防止更新 Item 表，除非是在 BillId 被填充或唯一更改是 Modified 列的情况下？

我更喜欢不需要我比较每列的插入和删除值（或使用 COLUMNS_UPDATED()）的解决方案，因为这会产生维护问题（有人必须记住更新触发器向表中添加或删除新列的时间）。我使用的是 SQL Server 2005。

Answer 1

为什么不使用INSTEAD OF 触发器？它需要更多的工作（即重复的UPDATE 声明），但任何时候你都可以阻止工作，而不是让它发生然后回滚，你会变得更好。

CREATE TRIGGER [dbo].[Item_BeforeUpdate_AnyBilled]
ON [dbo].[Item]
INSTEAD OF UPDATE
AS 
BEGIN
  SET NOCOUNT ON;

  IF EXISTS 
  (
     SELECT 1 FROM inserted i
       JOIN deleted AS d ON i.ItemId = d.ItemId
       WHERE d.BillId IS NULL -- it was NULL before, may not be NULL now
  )
  BEGIN
     UPDATE src 
       SET col1 = i.col1 --, ... other columns
          ModifiedDate = CURRENT_TIMESTAMP -- this eliminates need for other trigger
       FROM dbo.Item AS src
       INNER JOIN inserted AS i
       ON i.ItemId = src.ItemId
       AND (criteria to determine if at least one column has changed);
  END
  ELSE
  BEGIN
     RAISERROR(...);
  END
END
GO

这并不完美。我遗漏的标准是有原因的：确定列值是否已更改可能很复杂，因为它取决于数据类型、列是否可以为 NULL 等。AFAIK 内置触发器函数只能判断是否指定了某个列，而不能判断该值是否实际上比以前发生了变化。

EDIT 考虑到您只关心由于 after 触发器而更新的其他列，我认为以下 INSTEAD OF 触发器可以替换您现有的两个触发器并处理一次更新多行（有些不符合您的条件）：

CREATE TRIGGER [dbo].[Item_BeforeUpdate_AnyBilled]
ON [dbo].[Item]
INSTEAD OF UPDATE
AS 
BEGIN
  SET NOCOUNT ON;

  UPDATE src SET col1 = i.col1 --, ... other columns,
     ModifiedDate = CURRENT_TIMESTAMP
     FROM dbo.Item AS src
     INNER JOIN inserted AS i
     ON src.ItemID = i.ItemID
     INNER JOIN deleted AS d
     ON i.ItemID = d.ItemID 
     WHERE d.BillID IS NULL; 

  IF @@ROWCOUNT = 0
  BEGIN
    RAISERROR(...);
  END
END
GO