Sql Server 触发器触发 Square 次数

Question

我正在尝试向记录更改的某些行的值的表添加触发器。一切都按预期工作，除了触发器执行行数的平方。例如，如果我要在批量查询中更改表中 5 个项目的名称，触发器将在 TriggerTable 中记录 25 个条目。每个项目都会记录更改的总项目数，因此第一个名称更改将被记录 5 次，因为总共有 5 次名称更改。在下面粘贴了触发器的一部分。

我尝试添加IF TRIGGER_NESTLEVEL() <= 1 没有成功。除了这部分之外，其余功能都很好用。任何建议表示赞赏！

CREATE OR ALTER TRIGGER [dbo].[Trigger]
ON [dbo].[Table]
AFTER INSERT, UPDATE, DELETE
AS
BEGIN
    IF UPDATE(Name)
        IF EXISTS(SELECT Name FROM inserted) AND EXISTS(SELECT Name FROM deleted)
            INSERT INTO TriggerTable (Id, ColumnName, OldValue, NewValue)
                SELECT i.Id, 'Name', d.Name, i.Name
                FROM deleted d, inserted i
                WHERE d.Name <> i.Name OR (d.Name IS NULL AND i.Name IS NOT NULL) OR (d.Name IS NOT NULL AND i.Name IS NULL)
        ELSE IF EXISTS(SELECT Name FROM inserted)
            INSERT INTO  TriggerTable (Id, ColumnName, NewValue)
                SELECT i.Id, 'Name', i.Name
                FROM inserted i
        ELSE IF EXISTS(SELECT Name FROM deleted)
            INSERT INTO  TriggerTable (Id, ColumnName, OldValue)
                SELECT d.Id, 'Name', d.Name
                FROM deleted d

Answer 1

@Larnu 是正确的，您的连接语法不是最佳的，但它向您隐藏了潜在的问题（问题不在于触发器执行了多次）。

您当前的逻辑与名称不同的任何行匹配。如果您更新两行，除非所有四个名称前后都相同，否则它们都会匹配。您需要匹配名称不同的任何行，但仅限于相同的Id。作为使用#temp 表和每个可能场景的示例（名称更改，名称不变，名称变为 NULL 或变为 NOT NULL，或名称保持 NULL）：

CREATE TABLE #deleted (Id int identity(1,1), name nvarchar(32));
CREATE TABLE #inserted(Id int identity(1,1), name nvarchar(32));

INSERT #deleted (name) VALUES('bob'),('frank'), (NULL),  ('bla'),(NULL);
INSERT #inserted(name) VALUES('bob'),('franky'),('boop'),(NULL), (NULL);

-- not correct:
SELECT i.Id, 'Name', d.Name, i.Name
  FROM #deleted d, #inserted i
  WHERE d.Name <> i.Name 
    OR (d.Name IS NULL AND i.Name IS NOT NULL) 
    OR (d.Name IS NOT NULL AND i.Name IS NULL)

-- correct:
SELECT i.Id, 'Name', d.Name, i.Name
  FROM #deleted AS d
  INNER JOIN #inserted AS i
  ON d.Id = i.Id
  WHERE d.Name <> i.Name 
    OR (d.Name IS NULL AND i.Name IS NOT NULL) 
    OR (d.Name IS NOT NULL AND i.Name IS NULL);

您需要加入触发器才能看起来更像后者。

此外，我建议始终在逻辑块周围使用BEGIN/END。

IF something
BEGIN
    do a bunch of stuff
END

...可能会打字多一些，但对于读者来说，它比...更具可预测性和直观性。

IF something
    do a bunch of stuff