Суммарные суммы удвоения триггера SQL

Глядя на вашу схему, я вижу огромную кучу проблем. В произвольном порядке:

• Большинство колонок NULLспособны. Почему? Что бы это значило, если бы у Cart не было DateTimeCreated, почему вообще был бы такой ряд? А как насчет Deleted, существует ли третье неопределенное состояние (квантовая механика?), которое ни удалено, ни не удалено?
• Использование типа данных money, который имеет серьезные проблемы с округлением. Вместо этого используйте decimal с соответствующей точностью и масштабом.
• Использование префикса tbl раздражает, все и так знают, что это таблицы.
• Ваш существующий код триггера также имеет проблемы. Абсолютно отсутствует надлежащее форматирование, что делает его нечитаемым. Белое пространство бесплатно, вы знаете.
• Вы не проверяете виртуальную таблицу deleted в случае обновлений и удалений. Вам нужно присоединиться к нему по первичному ключу.
• Нет необходимости повторно присоединяться к таблице несколько раз, вы можете просто использовать inserted и deleted и вычесть разницу.
• NOLOCK это неправильно. Если вы беспокоитесь о блокировке, вам, вероятно, следует использовать изоляцию SNAPSHOT, а если вы беспокоитесь о производительности, вы можете использовать WITH (TABLOCK) для получения тех же преимуществ.
• Кажется, триггеру не нужно изменять CartLine, вы можете просто использовать вычисляемые столбцы:

  ALTER TABLE tblCartLine
      ADD ExtCost AS (Quantity * Cost);
  ALTER TABLE tblCartLine
      ADD TaxTotal AS (Quantity * Price) * (TaxRate / 100.0);
  ALTER TABLE tblCartLine
      ADD LineTotal AS (Quantity * i.Price);
  

И тогда ваш триггер должен выглядеть так

CREATE TRIGGER [dbo].[UtblCartLine] 
   ON  [dbo].[tblCartLine] 
   AFTER INSERT,DELETE,UPDATE
AS 

SET NOCOUNT ON;
IF TRIGGER_NESTLEVEL(OBJECT_ID('dbo.UtblCartLine')) > 0
    RETURN;
IF NOT EXISTS (SELECT 1 FROM inserted) AND NOT EXISTS (SELECT 1 FROM deleted)
    RETURN;

UPDATE tblCart
SET
  TotalCost  += i.DiffGoods,
  TotalPrice += i.DiffPrice,
  TotalTax   += i.DiffTax,
  TotalTotal += i.DiffTotal
FROM tblCart c
JOIN (
    SELECT
      ISNULL(i.CartID, d.CartID) CartID,
      ISNULL(SUM(i.GoodsTotal), 0) - ISNULL(SUM(d.GoodsTotal), 0) DiffGoods,
      ISNULL(SUM(i.LineTotal), 0) - ISNULL(SUM(d.LineTotal), 0) DiffPrice,
      ISNULL(SUM(i.TaxTotal), 0) - ISNULL(SUM(d.TaxTotal), 0) DiffTax,
      ISNULL(SUM(i.LineTotal + i.TaxTotal), 0) - ISNULL(SUM(d.LineTotal + d.TaxTotal), 0) DiffTotal
    FROM inserted i
    FULL JOIN deleted d ON d.CartLineID = i.CartLineID
    GROUP BY
      ISNULL(i.CartID, d.CartID)
) i ON i.CartID = c.CartID;

Добавление Deleted = 0 требует корректной условной агрегации.

Однако я рекомендую вам вообще не использовать триггеры.

Вместо этого используйте представление. Если это необходимо для производительности, вы можете использовать индексированное представление. Сервер может поддерживать индекс в представлении в соответствии с любыми обновлениями/вставками и эффективно выполнять весь вышеуказанный код автоматически.

Индексированные представления имеют некоторые ограничения. Особенно:

• Должен быть привязан к схеме, поэтому вы не можете изменять базовые столбцы, не удаляя представление.
• Только внутренние соединения.
• Никаких подзапросов или производных таблиц.
• Агрегация разрешена, но у вас должен быть столбец COUNT_BIG(*), а единственная разрешенная агрегация — это SUM.

CREATE VIEW CartTotal
WITH SCHEMABINDING AS

SELECT
  cl.CartID,
  COUNT_BIG(*) NumberOfLines,
  SUM(cl.GoodsTotal) TotalGoods,
  SUM(cl.LineTotal) TotalPrice,
  SUM(cl.TaxTotal) TotalTax,
  SUM(cl.LineTotal + cl.TaxTotal) TotalTotal
FROM tblCartLine cl
WHERE cl.Deleted = 0
GROUP BY
  cl.CartID;

go

CREATE UNIQUE CLUSTERED INDEX CX_CartTotal ON CartTotal (CartID);