如何创建触发器以减少计数器，我的触发器有什么问题？

Question

当登记表中有插入时，我需要减少表计划中的计数器：

CREATE TRIGGER [UpdateEnrollmentsTrigger]
ON [TBLENROLLMENT_ENR]
FOR INSERT
AS
BEGIN
    DECLARE @ScheduleCode NVARCHAR
    DECLARE @TotalSlots INT

    IF EXISTS(SELECT SCH_CODE FROM inserted)
        BEGIN
            SELECT @ScheduleCode = SCH_CODE FROM inserted

            SELECT @TotalSlots = SCH_TOTALSLOTS FROM TBLSCHEDULES_SCH
            WHERE SCH_CODE = @ScheduleCode

            UPDATE TBLSCHEDULES_SCH
            SET SCH_FREESLOTS = @TotalSlots - 1
            WHERE SCH_CODE = @ScheduleCode
        END
END

当我尝试创建此触发器时，VS12 的查询窗口显示：

SQL46010 :: Incorrect syntax near ].

提前致谢。

Answer 1

具体的错误是因为您使用的是FOR INSERT 而不是AFTER INSERT，但是您应该在触发器上改进其他一些事情。

首先，始终始终写入NVARCHAR 的长度，将其留空会根据使用位置而有所不同。所以将DECLARE @ScheduleCode NVARCHAR替换为DECLARE @ScheduleCode NVARCHAR(n)，其中n是需要的长度。

我也不确定您为什么要执行 IF EXISTS，因为它正在读取 INSERTED 伪表，因为触发了触发器，所以它一定会有记录。

要改进的另一件事是，您假设只插入了一行，因为您将其存储在一个变量中，这是错误的，如果您插入多行，它的行为就会不正确。

哦，我差点忘了，您还应该始终指定架构，例如：CREATE TRIGGER [UpdateEnrollmentsTrigger] ON [TBLENROLLMENT_ENR] 应该是 CREATE TRIGGER dbo.[UpdateEnrollmentsTrigger] ON dbo.[TBLENROLLMENT_ENR]（当然使用正确的架构）