如何在PL/SQL中用FOR循环取表行ID并传入存储过程（报错修正）

错误原因分析 #

• 游标FOR循环的查询语句中错误使用了INTO子句：INTO仅用于单行查询赋值给变量，游标FOR循环会自动遍历查询结果集，无需该子句
• 循环体内错误引用表别名a.ID：游标循环中，需通过游标变量（此处为c）访问每行数据的列，表别名a仅在查询语句内部有效，循环体中无法直接引用
• 存在冗余的嵌套BEGIN/END结构，以及不必要的NO_DATA_FOUND异常捕获：游标FOR循环在无数据时不会执行循环体，无需处理该异常

正确代码示例 #

DECLARE
   p_id     NUMBER(30);
   p_status VARCHAR(20);
BEGIN
   -- 游标FOR循环自动遍历查询结果
   FOR c IN (
      SELECT a.ID, a.STATUS
        FROM USER_COMMISSIONS a
        JOIN order_line b ON a.order_line_id = b.id
       WHERE a.status = 'unconfirmed'
   ) LOOP
      -- 通过游标变量c获取当前行的列值
      p_id := c.ID;
      p_status := c.STATUS;

      DBMS_OUTPUT.PUT_LINE(p_id);
      -- 调用存储过程，传入获取到的参数
      -- PSTK_PAYMENT_PACKAGE.add_payment(p_amt, p_user_id, p_reference, p_name, p_narration, p_payment_date, p_net_amt, p_payment_type_id, p_transaction_type_id, p_payment_id, p_status);
   END LOOP;
END;
/

代码优化说明 #

• 使用显式JOIN语法替代旧的逗号分隔表写法，提升代码可读性
• 移除了冗余的嵌套块和不必要的异常处理
• 直接通过游标变量c访问列数据，避免变量未声明的错误
• 如果不需要额外存储p_id和p_status，可以直接在存储过程调用中使用c.ID和c.STATUS，进一步简化代码：

DECLARE
BEGIN
   FOR c IN (
      SELECT a.ID, a.STATUS
        FROM USER_COMMISSIONS a
        JOIN order_line b ON a.order_line_id = b.id
       WHERE a.status = 'unconfirmed'
   ) LOOP
      DBMS_OUTPUT.PUT_LINE(c.ID);
      -- PSTK_PAYMENT_PACKAGE.add_payment(p_amt, p_user_id, p_reference, p_name, p_narration, p_payment_date, p_net_amt, p_payment_type_id, p_transaction_type_id, p_payment_id, c.STATUS);
   END LOOP;
END;
/

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