存储过程中对比两表列时IF判断失效问题排查

需求：检查students表的std_id是否与student_grace_marks表的std_id匹配，匹配则将对应grace_marks加到students表的marks列。

创建了如下存储过程，通过两个游标循环取数并判断std_id是否匹配，匹配时执行更新，但IF条件未生效，求排查原因：

create or replace procedure std_info
IS
    CURSOR stdcur IS SELECT std_id,std_name, marks,mark_status FROM students;
    CURSOR gracer IS SELECT std_id,grace_marks from student_grace_marks;
    myvar  stdcur%ROWTYPE;
    mycur gracer%ROWTYPE;

BEGIN
    OPEN stdcur;
    OPEN gracer;

    LOOP
        FETCH stdcur INTO myvar;
        FETCH gracer INTO mycur;
        EXIT WHEN stdcur%NOTFOUND;

        DBMS_OUTPUT.PUT_LINE( myvar.std_id || '  '|| myvar.std_name||' '||myvar.marks );

        if(myvar.std_id=mycur.std_id) then
            update students set marks=myvar.marks+mycur.grace_marks;
        end if;

    END LOOP;

    CLOSE stdcur;
    CLOSE gracer;
END;

问题原因分析 #

• 双游标同步遍历逻辑错误：同时对两个游标执行FETCH，意味着每次循环只会取两个游标各自的下一条记录，只有当两个表的std_id在完全相同的行索引位置时才会匹配，完全忽略了数据本身的对应关系。比如students表第1条是std_id=1，student_grace_marks表第1条是std_id=2，就永远不会触发匹配，这是核心问题。
• UPDATE语句缺少WHERE条件：即使匹配成功，UPDATE语句没有指定WHERE std_id = myvar.std_id，会把整个students表的所有记录的marks都更新成当前myvar.marks+mycur.grace_marks的值，这是严重的逻辑错误。
• 游标终止条件不完整：只判断了stdcur%NOTFOUND，但如果gracer游标先遍历完，后续mycur会保持最后一条记录的值，可能导致错误的匹配判断。

修复方案 #

方案1：修复双游标逻辑（不推荐，效率较低） #

针对每个学生单独查询对应的加分，避免同步遍历的错误：

create or replace procedure std_info
IS
    CURSOR stdcur IS SELECT std_id, std_name, marks, mark_status FROM students;
    myvar stdcur%ROWTYPE;
    v_grace_marks student_grace_marks.grace_marks%TYPE;
BEGIN
    OPEN stdcur;
    LOOP
        FETCH stdcur INTO myvar;
        EXIT WHEN stdcur%NOTFOUND;

        DBMS_OUTPUT.PUT_LINE( myvar.std_id || '  '|| myvar.std_name||' '||myvar.marks );

        -- 查询当前学生对应的加分
        BEGIN
            SELECT grace_marks INTO v_grace_marks
            FROM student_grace_marks
            WHERE std_id = myvar.std_id;

            -- 仅更新当前学生的成绩
            UPDATE students 
            SET marks = myvar.marks + v_grace_marks
            WHERE std_id = myvar.std_id;
        EXCEPTION
            WHEN NO_DATA_FOUND THEN
                CONTINUE; -- 无对应加分则跳过
            WHEN OTHERS THEN
                RAISE; -- 抛出其他异常
        END;
    END LOOP;
    CLOSE stdcur;
END;

方案2：使用关联UPDATE（推荐，效率极高） #

完全不需要游标，用单条SQL语句实现需求，这是数据库操作的最优方式：

-- Oracle 关联UPDATE写法
UPDATE students s
SET marks = marks + (
    SELECT grace_marks 
    FROM student_grace_marks g 
    WHERE g.std_id = s.std_id
)
WHERE EXISTS (
    SELECT 1 
    FROM student_grace_marks g 
    WHERE g.std_id = s.std_id
);

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