遇到了 SQL Server 中一个非常经典的变量复用陷阱。
在 WHILE 循环中,SELECT @变量 = 列名 这种写法,如果查询结果为空(0行),它不会把变量清空,也不会将其设置为 NULL,而是会"固执"地保持该变量在上一次循环中被赋的值。
因此,当循环进行到某个不存在的登录名时,@local_sid 就会偷偷沿用上一个账号的 SID。
解决方案
解决这个问题有两种最标准、最安全的方法:
方法 A:在每次循环开始时,手动将变量重置为 NULL(最常用)
在 WHILE 循环内部、执行 SELECT 语句之前,明确地给变量赋一个 NULL 值。这样即使查询没找到数据,它也只会是 NULL,不会污染后面的逻辑。
WHILE @@FETCH_STATUS = 0
BEGIN
-- 1. 【核心:每次循环开始前,必须初始化重置】
SET @local_sid = NULL;
-- 2. 正常的查询赋值
SELECT @local_sid = sid FROM master.sys.sql_logins WHERE name = @name;
-- 3. 判断是否查到了值
IF @local_sid IS NULL
BEGIN
PRINT '登录名 ' + @name + ' 在本地不存在!';
END
ELSE
BEGIN
PRINT '登录名 ' + @name + ' 的 SID 为:' + CONVERT(VARCHAR(MAX), @local_sid, 1);
END
-- 读取游标下一行
FETCH NEXT FROM ... INTO @name;
END
方法 B:改用 SET @变量 = (SELECT ...) 语法(最安全)
如果你使用 SET @local_sid = (SELECT sid FROM ...) 这种子查询的赋值方式,SQL Server 的行为会完全不同:如果子查询找不到值,它会自动将变量赋值为 NULL,从而天然地规避了循环污染的问题。
WHILE @@FETCH_STATUS = 0
BEGIN
-- 使用 SET + 子查询,找不到值时会自动变成 NULL,不需要手动重置
SET @local_sid = (SELECT sid FROM master.sys.sql_logins WHERE name = @name);
IF @local_sid IS NULL
BEGIN
-- 找不到时的逻辑
END
FETCH NEXT FROM ... INTO @name;
END
为什么会这样?(原理速览)
SELECT @var = col FROM table:这属于"基于行的赋值"。如果WHERE条件过滤后一条数据都没有,SQL Server 认为"没有发生任何赋值动作",因此变量保持原样。SET @var = (SELECT col FROM table):这属于"标量标量赋值"。SQL Server 会把括号里的子查询当成一个整体。当子查询返回空集合时,该集合在表达式中代表NULL,因此SET会强制把NULL赋给变量。