Ораклисты, help pls

[J++]

Пишу процедуру на PL/SQL Оракла.
пока в них не сильна, нужен совет в расстановке commit-rollback внутри процедуры:

сейчас сделано примерно так (если писать человеческим языком):

PROCEDURE какая-то (список параметров)
IS
BEGIN
 
  IF проверка_1_не_прошла THEN
       RETURN;
  END IF;


  IF проверка_2_не_прошла THEN
       RETURN;
  END IF;

  все_проверки_прошли_что-то_делаем_с записью_в_базу;
  commit; 

EXCEPTION -- отловить всякую бяку, которая не предусмотрена процедурой и предыдущими блоками begin-end
     WHEN OTHERS THEN
        -- тут_присвоить_коды_ошибок_выходным_переменным и т.п.
        an_err_code:=SQLCODE;
        as_err_text:=SQLERRM;            

        rollback;
END;

при этом внутри процедуры могут быть вложенные блоки
BEGIN-EXCEPTION-END, которые обрабатывают только "нужные" исключения, а WHEN OTHERS оставляют самому внешнему блоку.

вопрос:

1) правильно ли я поставила rollback в обработке самого внешнего Exception?
Теоретически, мимо него не должна проскочить никакая нештатная сиуация - а все штатные exception-ы обрабатываются раньше, в проверках и блоках IF.

2) мне обязательно нужно возвращать коды ошибок Oracle, если хоть что-то произошло.

Вот в этой ситуации коды ошибок присваиваются выходным переменным:

        -- тут_присвоить_коды_ошибок_выходным_переменным и т.п.
        an_err_code:=SQLCODE;
        as_err_text:=SQLERRM;            

        rollback;

а в этой что? может ли rollback пере-инициализировать коды ошибок Oracle?
и следовательно, если rollback идет первым - то увижу не реальную ошибку, из-за которой случился exception, а код завершения операции rollback?

        rollback;

        -- тут_присвоить_коды_ошибок_выходным_переменным и т.п.
        an_err_code:=SQLCODE;
        as_err_text:=SQLERRM;

Может кто-нибудь увидит другие некорректные вещи, - тогда тоже скажите плз.

[J++]

Знаю, что плохо делать rollback-commit внутри процедуры. Но я крутила-крутила варианты, всё равно получается программный код так себе. Не лапша, но явно и не дзен.

Если не понравится, сделаю то же самое на уровне вызывающих.

А исключения В ДАННОМ случае и не нужно передавать никуда наверх. Там пока что достаточно кодов ошибок. Это все - довольно тупая либа, которая работает по принципу "получилось - не получилось". Если она получит развитие, то буду смотреть дальше.

В общем, я так понимаю - нехорошо, но не криминально? Если так, то пока оставлю этот вариант.

[n0frost]

Сообщение от Ismail

Фиксация-откат внутри функций - плохо. Обработка всех исключений без передачи ошибки наверх - плохо.

Эт точно, еще много ретурнов из разных мест - тоже плохо.

Я обычно в похожих случаях делаю функцию, возвращает 0 - значит саксесс, иначе код ошибки.