SQL游标与if-else if语句执行异常问题求助

你遇到的是游标使用里的典型陷阱——默认的游标会实时感知原表的数据变化，当你在循环内更新t1表的记录后，这条被修改的记录会再次被游标抓取，导致重复进入逻辑分支。下面给你几个实用的解决方案：

方案1：改用静态游标（推荐） #

静态游标会在创建时生成结果集的快照，后续对原表的修改不会影响游标内的数据，从根本上避免重复处理。修改你的游标声明即可：

DECLARE cursor CURSOR STATIC FOR 
SELECT col1, col2, col3 ... FROM t1 WHERE...
OPEN cursor
FETCH NEXT FROM cursor INTO @col1, @col2, @col3
WHILE @@FETCH_STATUS = 0
BEGIN
    IF(@col1 = 1)
    BEGIN
        update u set u.col1=2 from t1 u where ...
    END
    ELSE IF(@col1 <> 1)
    BEGIN
        update u set u.col1=3 from t1 u where ...
    END
    FETCH NEXT FROM cursor INTO @col1, @col2, @col3
END
CLOSE cursor
DEALLOCATE cursor

另外注意：你代码里存在游标名称不一致的问题——声明的是cursor，但关闭和释放时用了cursor_sala_curso，这会直接报错，记得统一名称。

方案2：用集合操作替代游标（更推荐） #

游标在处理大量数据时效率极低，还容易出现这类逻辑问题。其实你可以用单条UPDATE语句完成完全相同的逻辑，完全不需要游标：

UPDATE t1
SET col1 = CASE 
    WHEN col1 = 1 THEN 2
    ELSE 3
END
WHERE -- 这里放入你原游标查询中的WHERE条件

这种集合操作不仅性能远高于游标，还从根源上规避了循环重复处理的问题。

方案3：标记已处理记录（若必须保留游标） #

如果因为特殊业务场景必须使用游标，可以给t1表新增一个IsProcessed布尔字段，或者利用现有字段标记状态，在游标查询时排除已处理的记录：

首先修改游标查询的WHERE条件：

DECLARE cursor CURSOR FOR 
SELECT col1, col2, col3 ... FROM t1 WHERE ... AND IsProcessed = 0

然后在更新时同步标记已处理状态：

IF(@col1 = 1)
BEGIN
    update u set u.col1=2, u.IsProcessed=1 from t1 u where ...
END
ELSE IF(@col1 <> 1)
BEGIN
    update u set u.col1=3, u.IsProcessed=1 from t1 u where ...
END

这样被处理过的记录就不会再被游标抓取到了。

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