SQL Server 2022字符串转uniqueidentifier转换失败问题排查

核心错误点 #

• 变量未声明：原代码直接将游标结果存入@UserID，但未提前声明该变量，这是触发运行时错误的关键（右键执行存储过程时SSMS对变量声明的检查更严格）。
• 游标未循环：仅执行一次FETCH NEXT，只能处理第一个用户的数据，无法遍历所有PersonID完成批量操作。
• NOT EXISTS逻辑错误：原条件仅匹配ExerciseID，未关联当前用户的PersonID，导致判断逻辑完全偏离需求——只要系统中任意用户有该Exercise，就不给当前用户插入，而非判断当前用户是否已存在该记录。
• 列名拼写不一致：Exercise表主键为ExcerciseID，Workout表外键为ExerciseID，拼写差异易引发混淆和潜在错误。

修正后的代码 #

DECLARE @UserID uniqueidentifier; -- 必须提前声明变量

DECLARE IndexCursor CURSOR FOR
    SELECT PersonID
    FROM Person;

OPEN IndexCursor;
FETCH NEXT FROM IndexCursor INTO @UserID;

-- 循环处理所有用户
WHILE @@FETCH_STATUS = 0
BEGIN
    INSERT INTO Workout (PersonID, ExerciseID, ExerciseTypeID)
    SELECT 
        @UserID,
        A.ExcerciseID,
        A.ExerciseTypeID
    FROM Exercise A
    WHERE NOT EXISTS (
        SELECT 1 -- NOT EXISTS仅关心记录是否存在，用1更高效
        FROM Workout B
        WHERE B.PersonID = @UserID -- 关联当前用户
          AND B.ExerciseID = A.ExcerciseID -- 匹配ExerciseID，注意拼写统一
    );

    FETCH NEXT FROM IndexCursor INTO @UserID;
END;

CLOSE IndexCursor;
DEALLOCATE IndexCursor;

额外优化建议 #

• 用集合操作替代游标：游标效率较低，批量场景下用交叉连接+NOT EXISTS的集合写法性能更优：

INSERT INTO Workout (PersonID, ExerciseID, ExerciseTypeID)
SELECT 
    P.PersonID,
    E.ExcerciseID,
    E.ExerciseTypeID
FROM Person P
CROSS JOIN Exercise E
WHERE NOT EXISTS (
    SELECT 1
    FROM Workout W
    WHERE W.PersonID = P.PersonID
      AND W.ExerciseID = E.ExcerciseID
);

• 统一列名拼写：建议修改Exercise表的ExcerciseID为ExerciseID，与Workout表保持一致，避免后续拼写错误：

-- 先删除关联外键再重命名
ALTER TABLE Workout DROP CONSTRAINT FK_ExcerciseNameToWorkout;
EXEC sp_rename 'Exercise.ExcerciseID', 'ExerciseID', 'COLUMN';
ALTER TABLE Workout ADD CONSTRAINT FK_ExerciseToWorkout FOREIGN KEY(ExerciseID) REFERENCES Exercise(ExerciseID);

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