PL/SQL存储过程AVG_MARK参数错误及最后记录重复问题咨询

我来帮你逐一解决遇到的两个问题：

问题1：调用时提示参数数量/类型错误 #

这个问题是因为存储过程的定义存在多处语法错误，导致实际的参数签名不符合预期：

• 缺少核心关键字：CREATE OR REPLACE后必须加上PROCEDURE才能正确声明存储过程；
• 参数列表格式错误：CAVG_SALARY OUT TEST_TBL.SALARY%type后面遗漏了逗号，使得CRANK参数的定义被错误解析；
• 窗口函数语法错误：RANK() OVER ORDER BY SALARY DESC)多了一个多余的右括号，正确写法应为RANK() OVER (ORDER BY SALARY DESC)。

问题2：最后一条记录重复显示 #

这是游标遍历的典型逻辑错误：当前代码先执行FETCH获取数据到变量，立即输出后才判断是否退出。当游标读取完最后一条记录后，下一次FETCH会触发C1%NOTFOUND，但此时变量中仍保留着最后一条记录的值，循环会再执行一次输出操作，导致重复。

修正后的完整代码 #

CREATE OR REPLACE PROCEDURE AVG_MARK (
    CROLL_NUM OUT TEST_TBL.ROLL_NUM%TYPE,
    CNAME OUT TEST_TBL.NAME%TYPE,
    CAVG_SALARY OUT TEST_TBL.SALARY%TYPE,
    CRANK OUT INTEGER
) AS
    CURSOR C1 IS
        SELECT ROLL_NUM, NAME, SALARY,
               RANK() OVER (ORDER BY SALARY DESC) AS RNK
        FROM TEST_TBL;
BEGIN
    OPEN C1;
    LOOP
        FETCH C1 INTO CROLL_NUM, CNAME, CAVG_SALARY, CRANK;
        -- 先判断是否无数据，无数据直接退出，避免重复输出
        EXIT WHEN C1%NOTFOUND;
        DBMS_OUTPUT.PUT_LINE(CROLL_NUM || ',' || CNAME || ',' || CAVG_SALARY || ',' || CRANK);
    END LOOP;
    CLOSE C1;
END;
/

更简洁的优化方案 #

如果你不需要通过OUT参数返回数据，只是要输出结果，推荐使用PL/SQL的游标FOR循环，它会自动处理游标的打开、遍历和关闭，代码更简洁且不易出错：

CREATE OR REPLACE PROCEDURE AVG_MARK AS
    CURSOR C1 IS
        SELECT ROLL_NUM, NAME, SALARY,
               RANK() OVER (ORDER BY SALARY DESC) AS RNK
        FROM TEST_TBL;
BEGIN
    FOR rec IN C1 LOOP
        DBMS_OUTPUT.PUT_LINE(rec.ROLL_NUM || ',' || rec.NAME || ',' || rec.SALARY || ',' || rec.RNK);
    END LOOP;
END;
/

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