0

テーブルに重複した最後のレコードを挿入するmuプロシージャに問題があります

INSERT を入れたときの例..... 'AAA' テーブル 'AAA' と 'AAA' の行に到達しました

DBMS()... をコードに配置すると、2 つのレコードが得られます。HistoriaDismissDate の列 ID にトリガーとシーケンスを使用しますが、状態は良好です。トリガーとシーケンスをドロップしたかどうかを確認しますが、同じ状況です

私もviewDateを使用していますが、このビューは2つではなく1つのレコードを取得します

私のコード

CREATE OR REPLACE PROCEDURE ChangeDismissDate
IS

          v_id VARCHAR2(11);
          v_dateBhd DATE := TO_DATE('20491231','yyyymmdd');
          v_dateDismiss DATE := TO_DATE('20491231','yyyymmdd');
          v_login VARCHAR2(50);
          last_id NUMBER :=0;

        CURSOR cur IS
                select EMP_NO, LOGIN, ODEJSCIE_BHD, ODEJSCIE_OLD FROM viewDate;

        BEGIN
                       OPEN cur;
                           LOOP
                                FETCH cur INTO v_id,v_login,v_dateBhd,v_dateDismiss;
                                  DBMS_OUTPUT.put_line(v_id || ' ' || v_login || ' ' || v_dateBhd || ' ' || v_dateDismiss);
                                 UPDATE employee_tab SET DISMISS_DATE = v_dateBhd WHERE EMP_NO = v_id;
                                 COMMIT;

                                 INSERT INTO HistoriaDismissDate(CUSTOMER_ID,LOGIN, DATE_CHANGE, DATE_BHD, DATE_DISMISS)
                                 VALUES(v_id,v_login, sysdate, v_dateBhd, v_dateDismiss);
                                 COMMIT;

                           EXIT WHEN cur%NOTFOUND;
                           END LOOP;
                           CLOSE cur;


    EXCEPTION
    WHEN NO_DATA_FOUND
    THEN
        DBMS_OUTPUT.put_line (DBMS_UTILITY.format_error_stack);
    WHEN OTHERS
    THEN
        DBMS_OUTPUT.put_line (DBMS_UTILITY.format_error_stack);
    END;
/
4

1 に答える 1

0

元のコードに関する 2 つのヒント:

1) カーソルは、PL/SQLの非常に古いプログラミング手法です。FOR ... LOOP 構造を使用することをお勧めします。よりクリーンで、エラーが発生しにくくなっています。それがどのように機能するかを見てください：

CREATE OR REPLACE PROCEDURE ChangeDismissDate IS
BEGIN
  for cur in (select EMP_NO, LOGIN, ODEJSCIE_BHD, ODEJSCIE_OLD FROM viewDate) loop

     DBMS_OUTPUT.put_line(cur.EMP_NO || ' ' || cur.login || ' ' || cur.ODEJSCIE_BHD || ' ' || cur.ODEJSCIE_OLD);
     UPDATE employee_tab 
        SET DISMISS_DATE = cur.ODEJSCIE_BHD
      WHERE EMP_NO = cur.EMP_NO;

     INSERT INTO HistoriaDismissDate
       ( CUSTOMER_ID,LOGIN, DATE_CHANGE, DATE_BHD, DATE_DISMISS )
     VALUES
       ( cur.EMP_NO, cur.LOGIN, sysdate, cur.ODEJSCIE_BHD, cur.ODEJSCIE_OLD, );
  end loop;
end;
/

2)決して、つまり、プロシージャ内にコミットを入れないでください。コミットは、呼び出し元ブロックまたはクライアント側アプリで行う必要があります。コミットをプロシージャ内に置くと、実行後にロールバックする機会を逃し、他のプロシージャがトランザクション フローを制御したい場合にそれを呼び出すことができません。

于 2012-12-07T18:44:34.063 に答える