而不是 SQL Server 中的触发器丢失 SCOPE_IDENTITY？

Question

我有一个表，我在其中创建了一个 INSTEAD OF 触发器来执行一些业务规则。

问题是当我将数据插入此表时，SCOPE_IDENTITY() 返回一个 NULL 值，而不是实际插入的标识。

插入 + 范围代码

INSERT INTO [dbo].[Payment]([DateFrom], [DateTo], [CustomerId], [AdminId])
VALUES ('2009-01-20', '2009-01-31', 6, 1)

SELECT SCOPE_IDENTITY()

触发器：

CREATE TRIGGER [dbo].[TR_Payments_Insert]
   ON  [dbo].[Payment]
   INSTEAD OF INSERT
AS 
BEGIN
-- SET NOCOUNT ON added to prevent extra result sets from
    -- interfering with SELECT statements.
    SET NOCOUNT ON;

    IF NOT EXISTS(SELECT 1 FROM dbo.Payment p
              INNER JOIN Inserted i ON p.CustomerId = i.CustomerId
              WHERE (i.DateFrom >= p.DateFrom AND i.DateFrom <= p.DateTo) OR (i.DateTo >= p.DateFrom AND i.DateTo <= p.DateTo)
              ) AND NOT EXISTS (SELECT 1 FROM Inserted p
              INNER JOIN Inserted i ON p.CustomerId = i.CustomerId
              WHERE  (i.DateFrom <> p.DateFrom AND i.DateTo <> p.DateTo) AND 
              ((i.DateFrom >= p.DateFrom AND i.DateFrom <= p.DateTo) OR (i.DateTo >= p.DateFrom AND i.DateTo <= p.DateTo))
              )

    BEGIN
        INSERT INTO dbo.Payment (DateFrom, DateTo, CustomerId, AdminId)
        SELECT DateFrom, DateTo, CustomerId, AdminId
        FROM Inserted
    END
    ELSE
    BEGIN
            ROLLBACK TRANSACTION
    END


END

代码在创建此触发器之前工作。我在 C# 中使用 LINQ to SQL。我看不到将SCOPE_IDENTITY 更改为@@IDENTITY 的方法。我该如何完成这项工作？

Answer 1

使用@@identity 代替scope_identity()。

scope_identity() 返回当前作用域中最后创建的 id，@@identity 返回当前会话中最后创建的 id。

通常建议在 @@identity 字段上使用 scope_identity() 函数，因为您通常不希望触发器干扰 id，但在这种情况下您会这样做。

Answer 2

由于您使用的是 SQL 2008，因此我强烈建议您使用 OUTPUT 子句而不是自定义标识函数之一。 SCOPE_IDENTITY 目前在并行查询方面存在一些问题，导致我完全反对它。 @@Identity 没有，但它仍然不像 OUTPUT 那样明确和灵活。加上 OUTPUT 处理多行插入。看看the BOL article 有一些很好的例子。

Answer 3

我对使用@@identity 有很大的保留意见，因为它可能会返回错误的答案。

但是有一种解决方法可以强制 @@identity 具有 scope_identity() 值。

为了完整起见，首先我将列出我在网上看到的这个问题的其他一些解决方法：

1. 使触发器返回一个行集。然后，在执行插入的包装器 SP 中，对另一个表执行 INSERT Table1 EXEC sp_ExecuteSQL ...。然后 scope_identity() 将起作用。这很麻烦，因为它需要动态 SQL，这很痛苦。另外，请注意，动态 SQL 在调用 SP 的用户的权限下运行，而不是在 SP 所有者的权限下运行。如果原始客户端可以插入到表中，他应该仍然具有该权限，只要知道如果您拒绝直接插入到表中的权限，您可能会遇到问题。

2. 如果有另一个候选键，则使用这些键获取插入行的标识。例如，如果 Name 上有一个唯一索引，那么您可以插入，然后使用 Name 从刚刚插入的表中选择（最多为多行）ID。虽然如果另一个会话删除了您刚刚插入的行，这可能会出现并发问题，但如果有人在应用程序可以使用它之前删除了您的行，这并不比原来的情况更糟。

现在，以下是如何明确地使您的触发器安全地让@@Identity 返回正确的值，即使您的 SP 或其他触发器在主插入之后插入到承载身份的表中。

另外，请在您的代码中添加 cmets，说明您在做什么以及为什么这样做，以便触发器的未来访问者不会破坏事情或浪费时间试图弄清楚。

CREATE TRIGGER TR_MyTable_I ON MyTable INSTEAD OF INSERT
AS
SET NOCOUNT ON

DECLARE @MyTableID int
INSERT MyTable (Name, SystemUser)
SELECT I.Name, System_User
FROM Inserted

SET @MyTableID = Scope_Identity()

INSERT AuditTable (SystemUser, Notes)
SELECT SystemUser, 'Added Name ' + I.Name
FROM Inserted

-- The following statement MUST be last in this trigger. It resets @@Identity
-- to be the same as the earlier Scope_Identity() value.
SELECT MyTableID INTO #Trash FROM MyTable WHERE MyTableID = @MyTableID

通常，对审计表的额外插入会破坏所有内容，因为因为它有一个标识列，所以@@Identity 将返回该值，而不是从插入到 MyTable 中的值。但是，最终选择会根据我们之前保存的 Scope_Identity() 创建一个正确的新 @@Identity 值。这也证明了它不受 MyTable 表上任何可能的附加 AFTER 触发器的影响。

更新：

我刚刚注意到这里不需要 INSTEAD OF 触发器。这可以满足您的所有需求：

CREATE TRIGGER dbo.TR_Payments_Insert ON dbo.Payment FOR INSERT
AS 
SET NOCOUNT ON;
IF EXISTS (
   SELECT *
   FROM
      Inserted I
      INNER JOIN dbo.Payment P ON I.CustomerID = P.CustomerID
   WHERE
      I.DateFrom < P.DateTo
      AND P.DateFrom < I.DateTo
) ROLLBACK TRAN;

这当然允许 scope_identity() 继续工作。唯一的缺点是身份表上的回滚插入确实会消耗所使用的身份值（身份值仍会根据插入尝试中的行数递增）。

我已经盯着这个看了几分钟，现在还不能完全确定，但我认为这保留了包含开始时间和排除结束时间的含义。如果结束时间包含在内（这对我来说很奇怪），那么比较需要使用

Answer 4

主要问题：触发器和实体框架都在不同的范围内工作。 问题是，如果您在触发器中生成新的 PK 值，则范围不同。因此该命令返回零行，EF 将抛出异常。

解决方案是在触发器末尾添加以下 SELECT 语句：

SELECT * FROM deleted UNION ALL
SELECT * FROM inserted;

在 * 的位置，您可以提及所有列名，包括

SELECT IDENT_CURRENT(‘tablename’) AS <IdentityColumnname>

Answer 5

就像 araqnid 评论的那样，触发器似乎在满足条件时回滚事务。您可以使用 AFTER INSTERT 触发器更轻松地做到这一点：

CREATE TRIGGER [dbo].[TR_Payments_Insert]
   ON  [dbo].[Payment]
   AFTER INSERT
AS 
BEGIN
    SET NOCOUNT ON;

    IF <Condition>
    BEGIN
        ROLLBACK TRANSACTION
    END
END

然后您可以再次使用 SCOPE_IDENTITY()，因为 INSERT 不再在触发器中完成。

条件本身似乎让两个相同的行过去，如果它们在同一个插入中。使用 AFTER INSERT 触发器，您可以像这样重写条件：

IF EXISTS(
    SELECT *
    FROM dbo.Payment a
    LEFT JOIN dbo.Payment b
        ON a.Id <> b.Id
        AND a.CustomerId = b.CustomerId
        AND (a.DateFrom BETWEEN b.DateFrom AND b.DateTo
        OR a.DateTo BETWEEN b.DateFrom AND b.DateTo)
    WHERE b.Id is NOT NULL)

它会捕获重复的行，因为现在它可以根据 Id 区分它们。如果您删除一行并将其替换为同一语句中的另一行，它也可以工作。

无论如何，如果您需要我的建议，请完全远离触发器。正如你所看到的，即使在这个例子中，它们也非常复杂。通过存储过程执行插入。它们比触发器更简单、更快：

create procedure dbo.InsertPayment
    @DateFrom datetime, @DateTo datetime, @CustomerId int, @AdminId int
as
BEGIN TRANSACTION

IF NOT EXISTS (
    SELECT *
    FROM dbo.Payment
    WHERE CustomerId = @CustomerId
    AND (@DateFrom BETWEEN DateFrom AND DateTo
    OR @DateTo BETWEEN DateFrom AND DateTo))
    BEGIN

    INSERT into dbo.Payment 
    (DateFrom, DateTo, CustomerId, AdminId)
    VALUES (@DateFrom, @DateTo, @CustomerId, @AdminId)

    END
COMMIT TRANSACTION

Answer 6

聚会有点晚了，但我自己正在研究这个问题。一种解决方法是在执行插入的调用过程中创建一个临时表，将范围标识从而不是触发器内部插入到该临时表中，然后在插入完成后从临时表中读取标识值.

在程序中：

CREATE table #temp ( id int )

... insert statement ...

select id from #temp
-- (you can add sorting and top 1 selection for extra safety)

drop table #temp

在而不是触发器：

-- this check covers you for any inserts that don't want an identity value returned (and therefore don't provide a temp table)
IF OBJECT_ID('tempdb..#temp') is not null
begin
    insert into #temp(id)
    values
    (SCOPE_IDENTITY())
end

为了安全起见，您可能希望将其称为 #temp 以外的其他名称（足够长且随机的名称，以至于其他人都不会使用它：#temp1234235234563785635）。