仅使用SQL数据库中的审核触发器显示已编辑的列

问题描述

我正在使用Microsoft SSMS。

我正在尝试为我的TestTable创建一个审计触发器，以便当用户编辑或插入任何数据时，它将记录在此ChangeLogTable中。下面的查询有效，并且我能够记录对表的任何更改。

但是，每当用户编辑一行时，它将记录所有内容，包括未编辑的列。例如，如果用户编辑TestDateTime，则TestStatus会出现在ChangeLogTable中，即使没有更改。

如何更改查询，以便仅记录已编辑的列？

Create trigger test_Audit
on TestTable
after Insert,update
not for replication
as

Declare @operation char(10)

if exists(select * from deleted)
set @operation = 'Edit'
else
set @operation = 'Insert'


if UPDATE (TestDateTime)
insert ChangeLog
(TableName,ModifiedBy,ChangeLogDateTime,Auditaction,WorkScheduleID,ChangedColumn,OldValue,NewValue)
select
'TestTable',SUSER_SNAME(),GETDATE(),@operation,inserted.TestID,'TestDateTime',Deleted.TestDateTime,Inserted.TestDateTime
from inserted
left outer join deleted
on inserted.TestID = deleted.TestID
and inserted.TestDateTime <> deleted.TestDateTime 


if UPDATE (TestStatus)
insert ChangeLog
(TableName,'TestStatus',Deleted.TestStatus,Inserted.TestStatus
from inserted
left outer join deleted
on inserted.TestID = deleted.TestID
and inserted.TestStatus<> deleted.TestStatus


if UPDATE (StaffID)
insert AuditLog
(TableName,AuditDateTime,ID,'StaffID',OSN.StaffName,NSN.StaffName
from inserted
left outer join deleted
on inserted.TestID= deleted.TestID
and inserted.StaffID <> deleted.StaffID
-- Fetch Staff Name
left outer join dbo.Staff OSN
on deleted.StaffID = OSN.StaffID
join dbo.Staff NSN
on inserted.StaffID = NSN.StaffID

解决方法

问题是您将连接on子句与where子句混淆了。因为例如inserted.TestStatus<> deleted.TestStatus作为on子句的一部分，所以如果数据发生更改，则永远不会匹配Deleted行。而您想要的是匹配ID上的行，然后检查值是否已更改。

注意，为了清楚起见，我已使用通用的最佳做法重新编写了触发器。

1. 使用表别名
2. 将其设置为基于集合而不是过程。
3. 处理null值，因为当任意一方为<>时都不能使用null运算符。
4. 与使用的大小写（小写/大写/混合）保持一致。
5. 使用分号终止语句。
6. 模式限定您的对象-我假设使用dbo。
7. 强制将varchar列转换为datetime，因为显式转换比隐式转换更好。

alter trigger dbo.test_Audit
on dbo.TestTable
after insert,update
not for replication
as
begin
  set nocount on;

  declare @operation char(10) = case when exists (select * from deleted) then 'Edit' else 'Insert' end;

  insert ChangeLog (TableName,ModifiedBy,ChangeLogDateTime,AuditAction,WorkScheduleID,ChangedColumn,OldValue,NewValue)
    select
      'TestTable',suser_sname(),getdate(),@operation,I.TestID,'TestDateTime',convert(varchar(21),D.TestDateTime),I.TestDateTime)
    from inserted I
    left outer join deleted D on I.TestID = D.TestID
    where coalesce(I.TestDateTime,'1 jan 1900') <> coalesce(D.TestDateTime,'1 jan 1900')
    and update(TestDateTime)
    union all
    select
    'TestTable','TestStatus',D.TestStatus,I.TestStatus
    from inserted I
    left outer join deleted D on I.TestID = D.TestID
    where coalesce(I.TestStatus,'') <> coalesce(D.TestStatus,'')
    and update(TestStatus)
end;