带有参数化游标的嵌套 for 循环正在插入重复记录

Posted

技术标签:

【中文标题】带有参数化游标的嵌套 for 循环正在插入重复记录【英文标题】:Nested for loop with parameterized cursors is inserting duplicate records 【发布时间】:2019-06-02 18:43:55 【问题描述】:

我们正在使用 2 个参数化游标,其中第一个游标的值需要在第二个游标中相应地获取数据,但它从两个游标中获取值。相反,它应该只从第二个游标中获取。

这里有 2 个光标。它需要从第一个游标中获取属性 1 的值并将其作为参数化游标传递给 cursor2。但是在插入时,它同时插入了第一个和第二个光标值。

实际结果和预期结果都在代码 sn-p 中给出。

/* Table Creation script:*/       
    create table tab1 (order_no number,order_item varchar2(40),header_id number)        
    /        
    create table tab2 (header_id number,line_id number,attribute1 number)        
    /        
    create table final_tab(order_no number, order_item varchar2(40), line_id number)        
    /

    /* Insertion script:*/         
    insert into tab1 values (1,'ABC',12345)
    /
    insert into tab1 values (11,'DEF',34567)
    /
    insert into tab2 values (12345,56789,11)
    /
    insert into tab2 values (12345,23489,11)
    /
    insert into tab2 values (34567,32156,null)
    /
    insert into tab2 values (34567,12534,null)
    /
    commit
    /

    /* Anonymous Block: */
    DECLARE
        CURSOR c1
        IS
            SELECT a.order_no,
                   a.order_item,
                   b.attribute1 end_ord_no,
                   a.header_id,
                   b.line_id
              FROM tab1 a, tab2 b
             WHERE a.header_id = b.header_id AND a.order_no = 1;
        CURSOR c2 (i_ord_no NUMBER)
        IS
            SELECT a.order_no,
                   a.order_item,
                   a.header_id,
                   b.line_id
             FROM tab1 a, tab2 b
             WHERE a.header_id = b.header_id AND a.order_no = i_ord_no;
    BEGIN
        FOR c1_rec IN c1
        LOOP
            FOR c2_rec IN c2 (c1_rec.end_ord_no)
            LOOP
                INSERT INTO final_tab (order_no, order_item, line_id)
                     VALUES (c2_rec.order_no, c2_rec.order_item, c2_rec.line_id);
            END LOOP;
        END LOOP;
        COMMIT;
    END;

/* Actual Result:*/

    Order_NO | Order_Item | Line_id
    11              |  DEF           | 32156
    11              |  DEF           | 12534
    11              |  DEF           | 32156
    11              |  DEF           | 12534

/*Expected Result:*/ 

    Order_NO | Order_Item | Line_id
    11              |  DEF           | 32156
    11              |  DEF           | 12534

【问题讨论】:

请编辑问题,并显示引用的代码以及问题所在。 添加了代码sn-p。 【参考方案1】:

问题是您的第一个游标返回两行,对于END_ORD_NO,这两行的值都是 11。第二个游标针对第一个游标返回的两行中的每一行执行,并且第二个游标的每次执行都返回两行,其中的值被适当地插入到FINAL_TAB

要解决这个问题,您应该将两个光标组合成一个光标:

SELECT a.order_no,
       a.order_item,
       a.header_id,
       b.line_id
  FROM tab1 a, tab2 b
  WHERE a.header_id = b.header_id AND
        a.order_no IN (SELECT b.attribute1
                         FROM tab1 a, tab2 b
                         WHERE a.header_id = b.header_id AND
                               a.order_no = 1)

这将您的代码块减少到

DECLARE
  CURSOR cc IS
    SELECT a.order_no,
           a.order_item,
           a.header_id,
           b.line_id
      FROM tab1 a, tab2 b
      WHERE a.header_id = b.header_id AND
            a.order_no IN (SELECT b.attribute1
                             FROM tab1 a, tab2 b
                             WHERE a.header_id = b.header_id AND
                                   a.order_no = 1);
BEGIN
  FOR rec IN cc LOOP
    INSERT INTO final_tab (order_no, order_item, line_id)
      VALUES (rec.order_no, rec.order_item, rec.line_id);
  END LOOP;

  COMMIT;
END;

这不仅会提供您想要的结果,而且还会通过简化代码来减少出错的机会,并通过消除嵌套循环来减少运行时间。

但是,即使这样也可以进一步简化为

INSERT INTO final_tab (order_no, order_item, line_id)
  SELECT a.order_no,
         a.order_item,
         b.line_id
    FROM tab1 a, tab2 b
    WHERE a.header_id = b.header_id AND
          a.order_no IN (SELECT b.attribute1
                           FROM tab1 a, tab2 b
                           WHERE a.header_id = b.header_id AND
                                 a.order_no = 1)

这样做的好处是它无需使用任何循环即可完成整个任务,并且速度更快,因为整个操作由数据库执行,而不必减慢与外部过程的接口。

dbfiddle here

祝你好运。


编辑

如果必须同时使用两个游标,或许可以使用 MERGE 语句,如下所示:

DECLARE
    CURSOR c1
    IS
        SELECT a.order_no,
               a.order_item,
               b.attribute1 end_ord_no,
               a.header_id,
               b.line_id
          FROM tab1 a, tab2 b
         WHERE a.header_id = b.header_id AND a.order_no = 1;
    CURSOR c2 (i_ord_no NUMBER)
    IS
        SELECT a.order_no,
               a.order_item,
               a.header_id,
               b.line_id
         FROM tab1 a, tab2 b
         WHERE a.header_id = b.header_id AND a.order_no = i_ord_no;
BEGIN
  FOR c1_rec IN c1 LOOP
    FOR c2_rec IN c2 (c1_rec.end_ord_no) LOOP
      MERGE INTO FINAL_TAB ft
        USING (SELECT c2_rec.order_no AS ORDER_NO,
                      c2_rec.order_item AS ORDER_ITEM,
                      c2_rec.line_id AS LINE_ID
                 FROM DUAL) d
          ON (ft.ORDER_NO = d.ORDER_NO AND
              ft.ORDER_ITEM = d.ORDER_ITEM AND
              ft.LINE_ID = d.LINE_ID)
        WHEN NOT MATCHED THEN
          INSERT (order_no, order_item, line_id)
          VALUES (d.order_no, d.order_item, d.line_id);
    END LOOP;  -- c2_rec
  END LOOP;  -- c1_rec

  COMMIT;
END;

【讨论】:

答案已编辑以包含使用 MERGE 语句执行条件 INSERT 的示例。

以上是关于带有参数化游标的嵌套 for 循环正在插入重复记录的主要内容,如果未能解决你的问题,请参考以下文章

具有参数化模式名称并用于目标表中的 BULK INSERT 的动态游标

For循环不打印游标中的最后一条记录

带有嵌套for循环的Javascript多维数组-无法正常工作

SQL 中将游标循环查询的结果插入到临时表中

spark中的嵌套for循环并行化

pl / sql如何使用带有游标的for循环进行更新操作