SQL Server查询字符串添加变量求助：触发器执行报错

嘿，我来帮你搞定这个触发器的问题！你遇到的错误主要是因为动态SQL拼接时的类型转换问题，还有没考虑到inserted表可能有多行数据的情况，另外字符串类型的变量拼接时没加单引号也会导致语法错误。咱们一步步来修正：

你的需求看起来是把UP_PLAN中插入的数据同步到TOTAL表，其实完全不需要用动态SQL，直接用普通的批量INSERT语句就能搞定，还能处理一次性插入多条数据的场景（你原来的写法只会取inserted里的第一条数据，其他行都会丢失）：

CREATE TRIGGER UPLtoTOTAL 
ON UP_PLAN 
FOR INSERT 
AS 
BEGIN
    SET NOCOUNT ON;
    -- 直接从inserted表批量同步数据，支持多行插入
    INSERT INTO TOTAL(CODE, COL2)
    SELECT CODE, QTY 
    FROM inserted;
END

这种写法既简洁又安全，完全不会有类型转换的问题，效率也更高。

如果因为某些业务需求必须用动态SQL，那一定要用参数化查询，而不是直接拼接变量，这样能避免类型转换错误和SQL注入风险：

CREATE TRIGGER UPLtoTOTAL 
ON UP_PLAN 
FOR INSERT 
AS 
BEGIN
    SET NOCOUNT ON;
    -- 注意：这里先处理单条数据的情况，多行场景后面会补充
    DECLARE @code nvarchar(10), @qty decimal(18,9);
    SELECT TOP 1 @code = CODE, @qty = QTY FROM inserted;

    DECLARE @sqlstr nvarchar(1000);
    -- 使用sp_executesql传递参数，避免直接拼接的问题
    SET @sqlstr = N'INSERT INTO TOTAL(CODE, COL2) VALUES(@p_code, @p_qty)';
    EXEC sp_executesql @sqlstr, 
                       N'@p_code nvarchar(10), @p_qty decimal(18,9)',
                       @p_code = @code, 
                       @p_qty = @qty;
END

为什么你原来的写法会报错？ #

• @qty是decimal类型，直接拼接进字符串时，SQL Server会尝试隐式转换，一旦数值长度超过默认转换的限制，就会触发算术溢出错误；
• @code是字符串类型，拼接时没加单引号，会导致生成的SQL语法错误（比如变成values(ABC123, 100)，正确应该是values('ABC123', 100)）；
• 没处理inserted表的多行数据，只会同步第一条插入的记录，其他行的数据都会丢失。

如果需要对inserted的每一行都执行动态SQL，可以用游标遍历处理（还是建议优先用批量操作，效率更高）：

CREATE TRIGGER UPLtoTOTAL 
ON UP_PLAN 
FOR INSERT 
AS 
BEGIN
    SET NOCOUNT ON;
    -- 用游标遍历inserted的每一行
    DECLARE @code nvarchar(10), @qty decimal(18,9);
    DECLARE cur CURSOR LOCAL FAST_FORWARD FOR
        SELECT CODE, QTY FROM inserted;
    
    OPEN cur;
    FETCH NEXT FROM cur INTO @code, @qty;
    WHILE @@FETCH_STATUS = 0
    BEGIN
        DECLARE @sqlstr nvarchar(1000);
        SET @sqlstr = N'INSERT INTO TOTAL(CODE, COL2) VALUES(@p_code, @p_qty)';
        EXEC sp_executesql @sqlstr, 
                           N'@p_code nvarchar(10), @p_qty decimal(18,9)',
                           @p_code = @code, 
                           @p_qty = @qty;
        FETCH NEXT FROM cur INTO @code, @qty;
    END
    CLOSE cur;
    DEALLOCATE cur;
END

内容的提问来源于stack exchange，提问作者Hoan