Блог

У меня есть следующее…

 IF CONDITION1  THEN

        -- SELECT  STATEMENT MIGHT RETURN DATA

         IF CONDITION2 THEN

                -- SELECT COUNT

             IF CONDITION3 THEN



                    INSERT INTO TABLE
                            (
                            --- 
                            )
                    VALUES  (
                            ---
                            );


            End IF;


    END IF;


 END IF;


EXCEPTION
   WHEN NO_DATA_FOUND THEN
 RETURN;

END of Trigger
  

Это правильный способ обработки исключения для оператора select внутри CONDITION1?

В PL / SQL нет способа вернуться на место ошибки, поэтому вам нужно создать блок вокруг части, которую вы хотите игнорировать конкретные ошибки:

 IF CONDITION1  THEN
         BEGIN
         -- SELECT  STATEMENT MIGHT RETURN DATA
         EXCEPTION
             WHEN NO_DATA_FOUND THEN
                 NULL;
         END;
         IF CONDITION2 THEN
                -- SELECT COUNT
             IF CONDITION3 THEN
                    INSERT INTO TABLE
                            (
                            --- 
                            )
                    VALUES  (
                            ---
                            );
            End IF;
    END IF;
 END IF;
END TRIGGER_NAME;
  

Альтернативой является использование явного курсора, который не возвращает ошибку, когда он пуст:

 DECLARE
   CURSOR cur_sample is select dummy from dual where 1=0;
   v_dummy dual.dummy%type;
BEGIN
   IF CONDITION1  THEN
         open cur_sample;
         fetch cur_sample into v_dummy;
         close cur_sample;
         IF CONDITION2 THEN
                -- SELECT COUNT
             IF CONDITION3 THEN
                    INSERT INTO TABLE
                            (
                            --- 
                            )
                    VALUES  (
                            ---
                            );
            End IF;
       END IF;
    END IF;
END;
  

Зависит от того, что вы подразумеваете под «правильным». То, что вы представили, синтаксически допустимо, да. Но вы не сказали нам, что вы на самом деле хотите, чтобы произошло, поэтому мы не можем сказать вам, действительно ли опубликованный вами код будет делать то, что вы хотите.

С точки зрения бизнес-логики, вы уверены, что это действительно не ошибка, если ваш SELECT INTO возвращает 0 строк? Если вы перехватываете и проглатываете исключение, это означает, что вы знаете, что на самом деле это не ошибка. Однако, если вы кодируете SELECT В, это подразумевает, что вы ожидаете ровно одну строку. Конечно, возможно, что оба эти утверждения верны, но было бы более распространенным, что это действительно исключение и что его не следует просто проглатывать и игнорировать.

В общем, я бы предпочел поместить обработчик исключений как можно ближе к запросу, который может вызвать исключение, насколько это возможно. Я бы счел более чистым иметь что-то вроде

 IF condition1
THEN
  BEGIN
    <<select statement>>
  EXCEPTION
    WHEN no_data_found
    THEN
      <<do something>>
  END;

  IF condition2
  THEN
    ...
  END IF;
END IF;
  

Таким образом, если вы в конечном итоге получите несколько мест в вашей процедуре, где может быть сгенерировано исключение NO_DATA_FOUND, будет ясно, какие исключения ожидаемы, а какие неожиданны.

Когда вы доходите до того, что у вас есть три уровня вложенных операторов IF, я склонен подозревать, что вы хотите преобразовать код в несколько процедур, чтобы сделать код более понятным. Например, вместо того, чтобы иметь вложенный блок PL / SQL, который выполняет инструкцию SELECT, перехватывает исключение и обрабатывает исключение, вероятно, было бы понятнее иметь отдельную функцию, которая все это делала, а затем вызывать эту функцию из вашего триггера.