我需要优化我的第一个T-SQL更新触发器

Question

如何在不使用大量变量的情况下重写此更新触发器？

我写了我的第一个SQL Server触发器,它工作正常,但我认为,必须有一个更简单的解决方案.

如果更改了5列中的最小值,则在另一个表中写入两个新行.第1行=旧的Fahrer(=驱动程序)和旧的dispodate和更新时间第2行=新的Fahrer和新的dispodate和updatedatetime我的解决方案只是foxpro-trigger的副本,但在T-SQL中必须有一个更简单的解决方案检查是否有一个列更改.

ALTER TRIGGER [dbo].[MyTrigger]
ON [dbo].[tbldisposaetze]
AFTER UPDATE
AS
SET NOCOUNT ON;
/*SET XACT_ABORT ON 
SET ARITHABORT ON 
*/
DECLARE @oldfahrer varchar(10)
DECLARE @oldbus varchar(10)
DECLARE @olddispodat date
DECLARE @oldvzeit decimal(4,0)
DECLARE @oldbzeit  decimal(4,0)
DECLARE @oldbeschreibk varchar(255)

DECLARE @newfahrer varchar(10)
DECLARE @newbus varchar(10)
DECLARE @newdispodat date
DECLARE @newvzeit decimal(4,0)
DECLARE @newbzeit  decimal(4,0)
DECLARE @newbeschreibk varchar(255)

    SELECT  @oldfahrer = fahrer,@oldbeschreibk=beschreibk,@oldbus=bus,@oldbzeit=bzeit,@olddispodat=dispodat,@oldvzeit=vzeit
        FROM DELETED D
    SELECT  @newfahrer = fahrer,@newbeschreibk=beschreibk,@newbus=bus,@newbzeit=bzeit,@newdispodat=dispodat,@newvzeit=vzeit
        FROM inserted I

if @oldbeschreibk <> @newbeschreibk or @oldbus <> @newbus or @oldbzeit <> @newbzeit or @oldfahrer <> @newfahrer or @oldvzeit <> @newvzeit 
begin
  IF (SELECT COUNT(*) FROM tbldispofahrer where fahrer=@oldfahrer and dispodat=@olddispodat) > 0 
    update tbldispofahrer  set laenderung = GETDATE()  where fahrer=@oldfahrer and dispodat=@olddispodat
  else
    INSERT into tbldispofahrer (fahrer,dispodat,laenderung) VALUES (@oldfahrer,@olddispodat,getdate())

  IF (SELECT COUNT(*) FROM tbldispofahrer where fahrer=@newfahrer and dispodat=@newdispodat) > 0 
    update tbldispofahrer  set laenderung = GETDATE()  where fahrer=@newfahrer and dispodat=@newdispodat
  else
    INSERT into tbldispofahrer (fahrer,dispodat,laenderung) VALUES (@newfahrer,@newdispodat,getdate())
 end 

Answer 1

我假设您拥有 SQL Server 2008 或更高版本。您可以在一条语句中完成这一切，无需任何变量。

您可以轻松地将其作为子句的一部分来完成，而不是首先获取变量并查看它们是否不匹配where。正如人们在评论中所说，您可以将多行作为插入和删除的一部分。为了确保您使用的是相同的更新行，您需要按主键进行匹配。

为了插入或更新行，我使用了一条MERGE语句。合并的来源是与上面的 where 子句的联合，联合中的顶部表具有旧的 fahrer，底部表具有新的 Farher。就像您的内部 IF 一样，现有行在 farher 和 dispodat 上进行匹配，并适当插入或更新。

我注意到的一件事是，在您的示例中，newfahrer 和 oldfahrer 可能完全相同，因此只应发生一次插入或更新（即，如果只有 bzeit 不同）。联合应防止尝试插入重复数据。我确实相信如果有的话合并会出错。

MERGE tbldispofahrer AS tgt
USING (
    SELECT d.farher, d.dispodat, GETDATE() [laenderung]
    INNER JOIN inserted i ON i.PrimaryKey = d.PrimaryKey
        AND (i.fahrer <> d.fahrer OR i.beschreibk <> d.beschreik ... )
    UNION
    SELECT i.farher, i.dispodat, GETDATE() [laenderung]
    INNER JOIN inserted i ON i.PrimaryKey = d.PrimaryKey
        AND (i.fahrer <> d.fahrer OR i.beschreibk <> d.beschreik ... )
) AS src (farher, dispodat, laenderung)
ON tgt.farher = src.farher AND tgt.dispodat = src.dispodat
WHEN MATCHED THEN UPDATE SET
    laenderung = GETDATE()
WHEN NOT MATCHED THEN
    INSERT (fahrer,dispodat,laenderung)
    VALUES (src.fahrer, src.dispodat, src.laenderung)