使用插入触发器后防止重复数据

Question

我可以检测到重复记录，但是当我插入新数据时，即使不存在，它也会将其检测为重复记录。

这是我的代码：

ALTER TRIGGER [dbo].[SDPRawInventory_Dup_Trigger] 
ON [dbo].[SDPRawInventory]
AFTER INSERT  
AS 
    DECLARE @Year float, 
            @Month float, 
            @SDPGroup nvarchar(255);

    SELECT 
        @Year = i.InvYear, @Month = i.InvMonth, @SDPGroup = i.SDPGroup 
    FROM inserted i;

    IF (SELECT COUNT(*) FROM SDPRawInventory A
        WHERE A.InvYear = @Year
          AND A.InvMonth = @Month
          AND A.SDPGroup = @SDPGroup) >= 1
    BEGIN
        RAISERROR ('Duplicate data', 16, 1)
        ROLLBACK;
    END
    ELSE
    BEGIN
        INSERT INTO SDPRawInventory
            SELECT * FROM inserted;
    END

这是桌子

为了澄清没有主键也没有唯一标识符。

Answer 1

如果您无法设置约束，那么您需要处理Inserted 可能有多个记录的事实。并且因为它是一个after insert 触发器，所以如果没有发现重复项，您无需执行任何操作，因为记录已经插入。

ALTER TRIGGER [dbo].[SDPRawInventory_Dup_Trigger] 
ON [dbo].[SDPRawInventory]
AFTER INSERT  
AS 
BEGIN
    SET NOCOUNT ON;

    IF EXISTS (
        SELECT 1
        FROM dbo.SDPRawInventory S
        INNER JOIN Inserted I ON
            -- Test for a duplicate
            S.InvYear = I.InvYear
            AND S.InvMonth = I.InvMonth
            AND S.SDPGroup = I.SDPGroup
            -- But ensure the duplicate is a *different* record - assumes a unique ID
            AND S.ID <> I.ID
    )
    BEGIN
        THROW 51000, 'Duplicate data.', 1; 
    END;
END;

注意简化和现代的错误处理。

编辑：如果您没有唯一键，也无权添加一个，那么您需要一个 instead of 触发器来仅插入非重复项，例如

ALTER TRIGGER [dbo].[SDPRawInventory_Dup_Trigger] 
ON [dbo].[SDPRawInventory]
INSTEAD OF INSERT  
AS 
BEGIN
    SET NOCOUNT ON;

    -- Reject the entire insert if a single duplicate exists
    -- Note if multiple records are inserted, some of which are duplicates and some of which aren't, they all get rejected
    IF EXISTS (
        SELECT 1
        FROM dbo.SDPRawInventory S
        INNER JOIN Inserted I ON
            -- Test for a duplicate
            A.InvYear = I.InvYear
            AND A.InvMonth = I.InvMonth
            AND A.SDPGroup = I.@SDPGroup
    )
    -- Test that Inserted itself doesn't contain duplicates
    OR EXISTS (SELECT 1 FROM Inserted GROUP BY InvYear, InvMonth, SDPGroup HAVING COUNT(*) > 1)
    BEGIN
        THROW 51000, 'Duplicate data.', 1; 
    END;

    INSERT INTO dbo.SDPRawInventory (SDP_SKU_DESC, WholeQty, InvYear, InvMonth, SDPGroup, invUOM, LooseQty)
      SELECT SDP_SKU_DESC, WholeQty, InvYear, InvMonth, SDPGroup, invUOM, LooseQty
      FROM Inserted I 
      WHERE NOT EXISTS (
          SELECT 1
          FROM dbo.SDPRawInventory S
          -- Test for a duplicate
          WHERE S.InvYear = I.InvYear
          AND S.InvMonth = I.InvMonth
          AND S.SDPGroup = I.SDPGroup
      );
END;

注意：这对处理现有的重复项没有任何作用。

Answer 2

此触发器在插入新记录之后执行，因此它至少会在SELECT COUNT 语句中找到原始记录。将>= 1 更改为>= 2 只能在保证一次插入一条记录时部分解决此问题。而且，如果新插入的记录在插入之前已经有多个重复记录在数据库中，它仍然会失败。

您需要从COUNT 中排除最新插入的记录。但更好的主意可能是添加一个UNIQUE 约束来防止重复，因此不需要触发器。

如果尚无法添加约束，则应启动清理过程以预先消除现有重复项。对我来说，其他一切看起来都存在很大缺陷，因为目前的方法不太可能使表格进入没有重复的状态。

Answer 3

您正在创建无限循环。您只需从触发器中移除插入部分即可。

ELSE
    BEGIN
         INSERT INTO SDPRawInventory
         SELECT * FROM inserted;
    END

这不应该在触发器中，因为触发器是作为插入的一部分调用的。您不应该在触发器中将实际插入写入表中。