PL/SQL代码未执行所有IF语句，薪资调整逻辑异常求助

核心错误点 #

• 循环仅执行一次：原脚本仅在循环前执行一次FETCH，循环内未重新获取下一条数据，且EXIT;直接终止循环，导致只处理第一条员工数据。
• 条件逻辑互相覆盖：嵌套IF的写法导致部门10的判断结果会被后续部门20的IF-ELSE覆盖，无论部门10的条件是否满足，最终都会执行部门20的薪资计算逻辑。
• 批量更新错误：UPDATE语句仅通过deptno过滤，会将整个部门的所有员工薪资统一改为当前遍历到的员工的新薪资，而非针对单个员工更新。

修复后的代码 #

SET SERVEROUTPUT ON

DECLARE
    v_newsal emp.sal%TYPE;
    v_sal emp.sal%TYPE;
    v_deptno emp.deptno%TYPE;
    v_empno emp.empno%TYPE; -- 新增员工编号，用于精准定位单个员工
    CURSOR raise_cursor IS
        SELECT empno, sal, deptno
          FROM emp;

BEGIN
    OPEN raise_cursor;
    LOOP
        -- 循环内每次获取下一条数据
        FETCH raise_cursor INTO v_empno, v_sal, v_deptno;
        -- 无数据时退出循环
        EXIT WHEN raise_cursor%NOTFOUND;

        -- 重新组织条件判断，用ELSIF避免逻辑覆盖
        IF v_deptno = 10 THEN
            IF v_sal > 2000 THEN
                v_newsal := v_sal * 1.06;
            ELSE
                v_newsal := v_sal * 1.07;
            END IF;
        ELSIF v_deptno = 20 THEN
            IF v_sal > 2500 THEN
                v_newsal := v_sal * 1.05;
            ELSE
                v_newsal := v_sal * 1.055;
            END IF;
        -- 可添加其他部门的逻辑，这里保留原脚本未处理的情况（如需默认涨幅可补充）
        END IF;

        -- 通过empno精准更新单个员工的薪资
        UPDATE emp
           SET sal = v_newsal
         WHERE empno = v_empno;
    END LOOP;
    CLOSE raise_cursor;
    -- 提交事务（如果需要永久生效）
    COMMIT;
END;
/
SET SERVEROUTPUT OFF

修复说明 #

1. 循环逻辑修正：将FETCH放入循环内，每次迭代获取新的员工数据，通过raise_cursor%NOTFOUND判断是否遍历完毕，确保处理所有员工。
2. 条件结构优化：用ELSIF拆分不同部门的判断逻辑，每个部门的薪资计算独立执行，不会互相覆盖。
3. 精准更新：新增empno字段，UPDATE时通过员工编号定位单个员工，避免批量更新整个部门的错误。
4. 事务提交：添加COMMIT语句，确保更新操作永久生效（根据业务需求可调整）。

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