SQL Server游标无限循环问题排查及不完整游标代码完善咨询

先直接给结论：你提供的这段代码肯定会触发无限循环，而且还有几个逻辑和语法问题，我来给你拆解问题所在，再给出修复后的完整代码。

问题分析 #

先看代码里的几个致命问题：

1. 缺少游标推进语句（核心原因）：在WHILE循环体的末尾，你没有执行FETCH NEXT FROM UnitPriceUpdateCursor INTO @ProductId;。这意味着游标永远停留在第一次获取的那一行，@@Fetch_Status会一直保持为0，循环永远不会终止，直接导致无限循环。
2. UnitPrice取值逻辑错误：SELECT @UnitPrice = UnitPrice FROM Products;这句没有加WHERE ProductID = @ProductId条件，它会把Products表中最后一行的UnitPrice赋值给变量，而不是当前游标指向的产品对应的价格，完全偏离了你的业务逻辑。
3. 语法不完整：ELSE IF分支的UPDATE语句里，WHERE ProductID = @...是不完整的，会直接导致语法报错。
4. 资源泄漏风险：代码末尾没有关闭和释放游标，会占用数据库资源。

修复后的完整代码 #

我已经把所有问题都修正了，你可以直接使用这段代码：

DECLARE @UnitPrice DECIMAL(5, 2); 
DECLARE @ProductId INT; 

-- 推荐使用LOCAL FORWARD_ONLY游标，提升性能并避免意外问题
DECLARE UnitPriceUpdateCursor CURSOR LOCAL FORWARD_ONLY FOR 
    SELECT ProductID FROM Products; 

OPEN UnitPriceUpdateCursor; 
FETCH NEXT FROM UnitPriceUpdateCursor INTO @ProductId; 

WHILE(@@Fetch_Status = 0) 
BEGIN 
    -- 获取当前游标指向产品的UnitPrice
    SELECT @UnitPrice = UnitPrice 
    FROM Products 
    WHERE ProductID = @ProductId; 

    IF(@UnitPrice BETWEEN 5 AND 10) 
    BEGIN 
        UPDATE Products 
        SET UnitPrice = 15.00 
        WHERE ProductID = @ProductId; 
    END 
    ELSE IF(@UnitPrice BETWEEN 15 AND 20) 
    BEGIN 
        UPDATE Products 
        SET UnitPrice = 25.00 
        WHERE ProductID = @ProductId; 
    END;

    -- 关键：每次循环结束后推进游标到下一行，避免无限循环
    FETCH NEXT FROM UnitPriceUpdateCursor INTO @ProductId; 
END;

-- 关闭并释放游标，释放数据库资源
CLOSE UnitPriceUpdateCursor;
DEALLOCATE UnitPriceUpdateCursor;

额外优化建议：尽量避免使用游标 #

其实这种批量更新操作完全不需要游标，用一条集合式的UPDATE语句就能搞定，效率比游标高得多（游标是逐行处理，集合操作是批量处理）：

UPDATE Products
SET UnitPrice = CASE
    WHEN UnitPrice BETWEEN 5 AND 10 THEN 15.00
    WHEN UnitPrice BETWEEN 15 AND 20 THEN 25.00
    ELSE UnitPrice -- 不满足条件的产品保持原价格
END
WHERE UnitPrice BETWEEN 5 AND 10 OR UnitPrice BETWEEN 15 AND 20;

这条语句会一次性更新所有符合条件的行，性能和可维护性都比游标好很多。

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