SQL Server动态查询执行及存储过程变量赋值报错解决问询

首先，你的问题有两个核心原因：动态SQL的变量作用域问题，以及循环中缺少游标下移逻辑导致的无限循环（这也是程序无响应的根源）。咱们一步步来修复：

1. 问题根源拆解 #

• 当你用EXEC(@SQL)执行动态SQL时，这段代码会在独立的执行上下文里运行，无法直接访问主存储过程中声明的@uid、@isactive等变量，所以会抛出“must declare the scalar variable @uid”的错误。
• 你的循环里没有在末尾添加FETCH NEXT INTO @clientid，游标永远停留在第一条记录，@@FETCH_STATUS始终为0，直接导致无限循环，这就是程序卡死无响应的原因。
• 你尝试的sp_executesql写法存在参数拼写错误（比如@clienid少了一个t，@createDate和存储过程里的@createdDate名称不匹配），而且没有完整声明所有输出参数。

2. 修正后的完整存储过程代码 #

USE DATABASE1
GO
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
CREATE PROCEDURE [User].[update_client_details]
AS
-- 修正变量声明的语法错误：补上缺失的逗号
DECLARE 
    @clientdata CURSOR, 
    @clientid INT, 
    @SQL NVARCHAR(2000), 
    @uid INT, 
    @isactive INT, 
    @createdDate Date
BEGIN
    SET @clientdata = CURSOR FOR SELECT clientuserid FROM [User].queen_client
    OPEN @clientdata
    FETCH NEXT FROM @clientdata INTO @clientid

    WHILE @@FETCH_STATUS = 0
    BEGIN
        -- 用QUOTENAME安全拼接数据库名，避免SQL注入风险
        SET @SQL = N'SELECT @uid = userid, @isactive = isactive, @createdDate = createddate FROM ' 
                   + QUOTENAME(@clientid) + N'.[User].queen_user';
        
        -- 通过sp_executesql绑定输出参数，让动态SQL的结果能回写到主存储过程变量
        EXEC sys.sp_executesql 
            @SQL,
            -- 声明参数类型：所有需要输出的变量都要标记OUTPUT
            N'@uid INT OUTPUT, @isactive INT OUTPUT, @createdDate DATE OUTPUT',
            -- 传递主存储过程的变量作为输出载体
            @uid = @uid OUTPUT,
            @isactive = @isactive OUTPUT,
            @createdDate = @createdDate OUTPUT;

        IF(@isactive = 1)
        BEGIN
            -- 在这里编写你的业务逻辑
            -- 示例：PRINT '处理clientid: ' + CAST(@clientid AS NVARCHAR(20)) + ', 用户ID: ' + CAST(@uid AS NVARCHAR(20))
        END

        -- 必须添加这行！让游标移动到下一条记录，避免无限循环
        FETCH NEXT FROM @clientdata INTO @clientid
    END

    CLOSE @clientdata
    DEALLOCATE @clientdata
END

3. 关键修复点说明 #

• 语法错误修正：在@uid INT后添加逗号，解决变量声明的语法问题。
• 动态SQL参数绑定：通过sp_executesql的参数列表明确声明输出参数，让动态SQL执行结果能赋值回主存储过程的变量，解决作用域隔离问题。
• 终止无限循环：在循环末尾添加游标下移逻辑，确保遍历完所有clientuserid后退出循环。
• 安全拼接数据库名：用QUOTENAME(@clientid)替代直接字符串拼接，既防止SQL注入，又能处理带特殊字符的数据库名。

4. 额外注意事项 #

• 要确保每个@clientid对应的数据库真实存在，否则会抛出“数据库不存在”的错误。
• 如果queen_user表可能无数据，建议在IF判断前先检查@uid是否为NULL，避免空值逻辑错误。

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