0

我的触发器有点问题。它应该：

  • 更新我的主表中插入的历史表并添加时间戳。
  • 如果历史表中的一行已经具有相同的值，则不应添加任何内容，但应增加计数器并更新时间。
  • 如果距离上次更新超过 24 小时，将创建一个新行。

马虎的伪代码：

IF NOT EXISTS ( --if the value isn't in the history table
    SELECT History.value1 FROM History, INSERTED
    WHERE History.value1 LIKE INSERTED.value1
    AND History.value2 LIKE INSERTED.value2
       ) 
       OR EXISTS ( --or if it has been added over 24h ago
        SELECT History.value1 FROM History, INSERTED
        WHERE History.value1 LIKE INSERTED.value1
        AND History.value2 LIKE INSERTED.value2
        AND DATEDIFF(HOUR,History.time, GETDATE()) > 24
       )     
    BEGIN --Insert it
        INSERT INTO History(value1, value2, counter, time)
        SELECT value1, value2, counter GETDATE() FROM INSERTED
    END
    ELSE
    BEGIN  -- else, increase counter and add new time
        UPDATE History
        SET History.time = GETDATE(), 
                               History.Items = History.Items + INSERTED.Items
        FROM History
        JOIN INSERTED ON History.value1 = INSERTED.value1
        AND History.value2 = INSERTED.value2
            AND DATEDIFF(HOUR, _History.time, GETDATE()) < 24;
    END

示例表：

__________________________________________________
| value1 | value2 | counter | time(last updated) |
+------------------------------------------------+
| test1  | test2  |    1    |        < 24h       |
| test3  | test4  |    1    |        > 24h       |
| test3  | test4  |    1    |        < 24h       |
+------------------------------------------------+

输入： 

INSERT INTO main_table(value1, value2, counter)
VALUES ('test3', 'test4', 1);

结果表：

__________________________________________________
| value1 | value2 | counter | time(last updated) |
+------------------------------------------------+
| test1  | test2  |    1    |        < 24h       |
| test3  | test4  |    1    |        > 24h       |
| test3  | test4  |    1    |        < 24h       | <--This counter+time should be updated
| test3  | test4  |    1    |        < 24h       | <--This row shouldn't be added
+------------------------------------------------+

我理解为什么会发生这种情况（因为代码找到了超过 24 小时的历史值，而忽略了较新的值），但我不知道如何修复它。

4

3 回答 3

1

inserted我认为如果包含混合行，您的触发器仍然会损坏- 因为您的IF/ELSE结构对要采取的操作做出单一决定。

最好有一个类似的MERGE东西：

;MERGE INTO History h USING INSERTED i
   ON h.Value1 = i.Value1 and h.Value2 = i.Value2 and
      DATEDIFF(HOUR,h.time, GETDATE()) <= 24
WHEN MATCHED THEN
   UPDATE SET time = GETDATE(), Items = h.Items + i.Items
WHEN NOT MATCHED THEN
   INSERT (Value1,Value2,Items,time)
   VALUES (i.Value1,i.Value2,i.Items,GETDATE());

哪个应该取代你的整个触发器体。

顺便，DATEDIFF计算跨越边界的转换次数，而不是计算精确的差异（例如DATEDIFF(hour,'00:59','01:01')为 1）。如果您想更接近同时考虑分钟和秒的 24 小时截止时间，更好的比较是：

h.Time >= DATEADD(day,-1,GETDATE())

顺便说一句，这也将允许使用包含History'time列的索引。

于 2013-06-13T08:38:43.167 回答
1

--像这样改变你的标准陈述。它会起作用的。反正你写的触发器真不好。触发器应设计为处理集合而不是记录，因为一批插入/更新/删除只能触发一次。

IF NOT EXISTS ( --if the value isn't in the history table
SELECT History.value1 FROM History, INSERTED
WHERE History.value1 LIKE INSERTED.value1
AND History.value2 LIKE INSERTED.value2
   ) 
   OR EXISTS ( --or if it has been added over 24h ago
    SELECT INSERTED.value1 
    FROM INSERTED
    cross apply
        (select max(History.[time]) mx_time
        from    History
        where   History.value1 LIKE INSERTED.value1
                AND History.value2 LIKE INSERTED.value2
        ) as t
    WHERE DATEDIFF(HOUR,t.mx_time, GETDATE()) >= 24
   )     
BEGIN --Insert it
    INSERT INTO History(value1, value2, items, time)
    SELECT value1, value2, items, GETDATE() FROM INSERTED
END
ELSE
BEGIN  -- else, increase counter and add new time

    UPDATE History
    SET History.time = GETDATE(), 
                           History.Items = History.Items + INSERTED.Items
    FROM History
    JOIN INSERTED ON History.value1 = INSERTED.value1
    AND History.value2 = INSERTED.value2
        AND DATEDIFF(HOUR, History.time, GETDATE()) < 24;
END
于 2013-06-13T09:44:49.593 回答
0

我的天啊，我解决了它^^一直在尝试修复它大约 2 小时，就在我将它发布到 stackoverflow 时，我设法修复它^^

我做了什么：

将第二个“EXIST”测试从：

OR EXISTS ( --or if it has been added over 24h ago
        SELECT History.value1 FROM History, INSERTED
        WHERE History.value1 LIKE INSERTED.value1
        AND History.value2 LIKE INSERTED.value2
        AND DATEDIFF(HOUR,History.time, GETDATE()) > 24
       )

至：

OR NOT EXISTS ( --or if it has been added over 24h ago
        SELECT History.value1 FROM History, INSERTED
        WHERE NOT History.value1 LIKE INSERTED.value1
        OR History.value2 LIKE INSERTED.value2
        OR DATEDIFF(HOUR,History.time, GETDATE()) > 24
       )
于 2013-06-13T08:30:34.843 回答