使用WHILE循环+动态SQL遍历Users表更新DB1失效，求解决方法

你的代码核心问题有两个：

1. 循环次数错误：你用@RowCount = (SELECT COUNT(1) FROM [DB1])，但实际需要遍历的是[Users]表的记录，循环次数应该对应[Users]的行数。
2. 未正确遍历单条记录：每次循环中执行SELECT @Field = ... FROM [USERS]会返回所有行，变量会被最后一行的值覆盖，导致每次执行的都是最后一条配置的更新逻辑。

修正后的代码 #

DECLARE @Field VARCHAR(50)
DECLARE @FieldType VARCHAR(10)
DECLARE @Operator CHAR(5)
DECLARE @Value VARCHAR(50)
DECLARE @Status VARCHAR(10)
DECLARE @StartDate DATETIME
DECLARE @EndDate DATETIME

DECLARE @Count INT = 1  
DECLARE @RowCount INT 
-- 改为获取Users表的行数
SET @RowCount = (SELECT COUNT(1) FROM [Users])  

DECLARE @SQL VARCHAR(MAX)

WHILE @Count <= @RowCount
BEGIN
    -- 给Users表生成行号，每次循环取对应行号的记录
    SELECT 
        @Field = [Field],
        @FieldType = [FieldType],
        @Operator = [Operator],
        @Value = [Value],
        @Status = [Status],
        @StartDate = [StartDate],
        @EndDate = [EndDate]
    FROM (
        SELECT *,
               ROW_NUMBER() OVER(ORDER BY [ConfigOrder] ASC) AS RowNum
        FROM [Users] (NOLOCK)
    ) AS UserRows
    WHERE RowNum = @Count

    -- 修正动态SQL的语法空格问题，避免拼接后出现语法错误
    SET @SQL= 'UPDATE [DB1]
        SET [Status] = ''' + @Status + '''
        FROM [DB1] AS a (NOLOCK)
        INNER JOIN [DB2] AS b (NOLOCK) ON a.ID = b.ID
        WHERE b.[status] = ''UNK''
        AND ' + @Field + ' ' + @Operator + ' ''' + @Value + '''
        AND  a.[start_date] >= ''' + CONVERT(VARCHAR(11),@StartDate,106)  + '''
        AND  a.[start_date] <= ''' + CONVERT(VARCHAR(11),@EndDate,106)  + '''' ;

    PRINT @SQL -- 调试用，可查看每次生成的动态SQL语句是否正确
    EXEC (@SQL)

    SET @Count = @Count + 1
END

关键改进点 #

• 行号遍历：通过ROW_NUMBER()函数给[Users]表的每条记录生成唯一行号，循环时根据@Count精准获取单条配置记录，避免变量被覆盖。
• 循环次数匹配：将循环次数改为[Users]表的总行数，确保每条配置规则都被执行一次。
• 语法修正：修复了动态SQL拼接时的空格缺失问题，避免出现类似Field=Value的语法错误。

额外建议 #

当前动态SQL存在SQL注入风险，如果[Users]表中的字段值不可信，建议改用sp_executesql进行参数化查询，示例核心部分如下：

SET @SQL= 'UPDATE [DB1]
    SET [Status] = @StatusParam
    FROM [DB1] AS a (NOLOCK)
    INNER JOIN [DB2] AS b (NOLOCK) ON a.ID = b.ID
    WHERE b.[status] = ''UNK''
    AND ' + @Field + ' ' + @Operator + ' @ValueParam
    AND  a.[start_date] >= @StartDateParam
    AND  a.[start_date] <= @EndDateParam' ;

EXEC sp_executesql @SQL,
    N'@StatusParam VARCHAR(10), @ValueParam VARCHAR(50), @StartDateParam DATETIME, @EndDateParam DATETIME',
    @StatusParam = @Status,
    @ValueParam = @Value,
    @StartDateParam = @StartDate,
    @EndDateParam = @EndDate;

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