【问题标题】:SQL Server Table Triggers update an aggregate table-update/insert multiple recordsSQL Server 表触发器更新聚合表-更新/插入多条记录
【发布时间】:2018-04-20 08:37:52
【问题描述】:

(注意:我的问题非常详细且针对我们的系统,因此在我提前回答实际问题之前,我会为解释的长度道歉。)

我们有许多表需要将其中包含的部分数据汇总到另一个表中。

我已尽我所能确保触发器可以处理多行而不仅仅是单行事务:

TRIGGER [dbo].[TR_SavedFiles_PlanLibraryMetrics] on [dbo].[SavedFiles]
AFTER INSERT, UPDATE, DELETE
AS
SET NOCOUNT ON
BEGIN
    IF EXISTS (
        SELECT m.AccountID, m.PlanLibCode 
        FROM PlanLibraryMetrics AS m 
            JOIN PlanLibraryItems AS it 
                ON m.PlanLibCode = it.PlanLibCode
            JOIN inserted AS i 
                ON m.AccountID = i.AccountID 
                AND (it.ItemTypeID = i.CISFileID AND it.ItemType = 'FILE')
            JOIN Accounts AS a ON i.AccountID = a.AccountID
        WHERE a.Status = 0       
    )
    BEGIN
    UPDATE PlanLibraryMetrics 
    SET MetricValue = x.newValue
    FROM (SELECT COUNT(s.AccountID) AS newValue, it.PlanLibCode, i.AccountID
          FROM SavedFiles AS s
            JOIN PlanLibraryItems AS it 
                ON (s.CISFileID = it.ItemTypeID AND it.ItemType = 'FILE')
            JOIN inserted AS i
                ON s.AccountID = i.AccountID
                AND s.CISFileID = i.CISFileID
          Group By PlanLibCode, i.AccountID) AS x 
        JOIN PlanLibraryMetrics AS m 
            ON x.PlanLibCode = m.PlanLibCode 
            AND x.AccountID = m.AccountID
END
ELSE IF EXISTS (
    SELECT m.AccountID, m.PlanLibCode 
    FROM PlanLibraryMetrics AS m 
        JOIN PlanLibraryItems AS it 
            ON m.PlanLibCode = it.PlanLibCode
        JOIN deleted AS d
            ON m.AccountID = d.AccountID 
            AND (it.ItemTypeID = d.CISFileID AND it.ItemType = 'FILE')
        JOIN Accounts AS a ON d.AccountID = a.AccountID
    WHERE a.Status = 0
)
BEGIN
    UPDATE PlanLibraryMetrics 
    SET MetricValue = x.newValue
    FROM (SELECT COUNT(s.AccountID) AS newValue,it.PlanLibCode, d.AccountID
            FROM SavedFiles AS s
                RIGHT OUTER JOIN deleted AS d
                    ON s.AccountID = d.AccountID
                    AND s.CISFileID = d.CISFileID 
                JOIN PlanLibraryItems AS it 
                    ON (d.CISFileID = it.ItemTypeID AND it.ItemType = 'FILE')                   
            Group By PlanLibCode, d.AccountID) AS x 
        JOIN PlanLibraryMetrics AS m 
            ON x.PlanLibCode = m.PlanLibCode 
            AND x.AccountID = m.AccountID
END
ELSE IF NOT EXISTS (
    SELECT m.AccountID, m.PlanLibCode 
    FROM PlanLibraryMetrics AS m 
        JOIN PlanLibraryItems AS it ON m.PlanLibCode = it.PlanLibCode
        JOIN inserted AS i 
            ON m.AccountID = i.AccountID 
            AND (it.ItemTypeID = i.CISFileID AND it.ItemType = 'FILE')
        JOIN Accounts AS a ON i.AccountID = a.AccountID
    WHERE a.Status = 0 
)
BEGIN
    INSERT INTO PlanLibraryMetrics
    SELECT i.AccountID, it.PlanLibCode, COUNT(s.AccountID)  
    FROM SavedFiles AS s 
        JOIN PlanLibraryItems AS it
            ON (s.CISFileID = it.ItemTypeID AND it.ItemType = 'FILE')
        JOIN inserted AS i
            ON s.AccountID = i.AccountID
            AND s.CISFileID = i.CISFileID
    Group By PlanLibCode, i.AccountID 
END             
END

而且它似乎可以工作,直到我们建立一个系统来“合并”账户,旧账户中的所有记录都将其 accountid 更改为主账户。

我试图完成的行为是更新聚合表中的任何现有记录,并为主帐户不存在的任何内容插入新记录。

所以我的问题是:If exists/elseif exists 子句的存在是否真的使这个触发器(以及以此触发器为模型的其余触发器)实际上不能处理多行?如果是这样,我认为我的问题不会通过为每个表编写 2 或 3 个单独的触发器来解决,因为我仍然必须每次都检查现有记录(即更新/删除触发器和单独的插入触发器)?我是否应该将所有这些移动到一个存储过程并传递一个表变量,例如,一个包含插入/删除表中所有记录的表变量(我想在论坛周围巡游)?

SQL 不是我的强项,更不用说触发。非常感谢有经验的人提供任何帮助。

【问题讨论】:

  • 您是否考虑过使用索引视图来执行此聚合的可能性?如果您可以使用它们,我建议您这样做 - 在幕后,它们使用非常类似于触发器的东西,但该代码已被使用和测试比您希望的要多得多。跨度>
  • 我相信没有人想到过一个观点。我会把这个想法告诉我的主管讨论。谢谢!
  • 是的,存储可以计算的数据通常是不受欢迎的(因为你介绍机会计算数据与基础数据不一致)。如果你必须这样做,最好使用内置机制而不是自己编写(因为它们不太可能出现不一致或错过边缘情况)
  • 我应该在此处发布更多信息以便清楚起见还是创建一个新问题来详细说明这种情况?

标签: sql-server triggers sql-server-2008-r2


【解决方案1】:

简短的回答 - 不。使用“如果存在”不会妨碍正确处理多行语句。

但是说实话。如果您在使用 tsql 和触发器时遇到困难,那么您应该为 UNTIL 您对自己的方法和逻辑有信心的每个操作编写单独的触发器。

您的逻辑似乎过于复杂,但没有人知道您的架构以及您的表是如何使用的。它似乎也至少是一个错误。在您的插入部分中,您根据 a.Status = 0 检查是否存在。相关的插入语句中不包含相同的逻辑。这已传播到其他 2 组逻辑。

例如,假设在 SavedFiles 中插入了 5 行。 1 匹配 PlanLibraryMetrics 中的一行,其他不匹配。你的代码是做什么的?执行后,仅更新 PlanLibraryMetrics 中的一行 - 您已跳过聚合/插入其他 4 行所需的逻辑。使用正确的连接,您的删除逻辑似乎非常可疑;如果不知道各个表的实际和逻辑键,就很难理解。

关于这一点 - 现在是开始评论您的代码以帮助其他人(包括以后的您自己)了解代码应该做什么以及可能原因的好时机。是的 - 请考虑 Damien 的建议。一个视图,无论是否被索引,都可能是(可能是)一种更好的方法——当然也是一种更安全、更容易维护的方法。

【讨论】:

  • 谢谢。我很感激反馈。我将与高级程序员讨论视图的使用......我认为以前没有人考虑过。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 2015-11-06
  • 2013-06-11
  • 2017-07-22
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多