1

День добрый!

Подскажите, как наиболее оптимально и логично обрабатывать исключения в Java? Понятное дело, что каждый делает по–своему, но хотелось бы услышать советы опытных программистов.

Суть состоит в том, чтобы сделать такую систему сообщений для пользователя, чтобы иметь возможность классифицировать их по меньшей мере на три группы: «критические ошибки» - при возникновении которых работа программы должна быть прервана; «ошибки» - завершение работы блока программы (например, цикла); «предупреждение» - обычное информирование пользователя.

Собственно, исходя из сказанного, понятно, что обрабатываться эти ошибки должны на разных уровнях программы. Как сие организовать?

UPD1: Не сочтите глупцом, поэтому я и задал сей вопрос. Приложение было консольным т.к. ориентировано на работу на сервере. Делал я следующим образом:

  1. Создал класс, который умеет выводить сообщения в соответствии с заданным при старте уровнем логирования (всего 8 уровней, 256 комбинаций, бла-бла-бла)
  2. Создал класс для обработки исключительных ситуаций, который передавал текст и уровень в (1)
  3. В процессе работы программы (5 шагов обработки) подготавливались и обрабатывались данные, соответственно функция main(...) занималась исключительно последовательным запуском функций и передачей им параметров. В данной функции обрабатывались критические ситуации, выводилось сообщение (2) с полным stack-trace и программа завершалась (например, невозможность чтения файла, без которого дальше продолжать обработку невозможно).
  4. Также было создано несколько классов которые отдельно обрабатывали данные, в каждом из них могли произойти исключительные ситуации, но уже двух типов:

4.1. Ошибки (т.е. ситуация, когда выполнение какой-то функции должно быть прервано), например извлечение из файла ID некоторого множества. Т.е. продолжать обработку следующих множеств можно. Пример:

public void startPreparations(LinkedList<MyDataEntity> dataEntityList) {
    for(int i=0; i<dataEntityList.size(); i++) {
       MyDataEntity dataEntity = dataEntityList.get(i);
       try {
           int id = getId(dataEntity);
       } catch (MyDataEntityException mdee) {
          MyExceptionHandler(mdee, LogLevel.ERROR);
          continue;
       }
       ...
    }
}
private int getId(MyDataEntity dataEntity) throws MyDataEntityException {
    //... throw (new ...) где-то тут
}

Это лишь пример.

4.2. Предупреждения. В них похожая ситуация как и в (4.1) только предупреждения обрабатываются локально (например, в функции getId()) или без прерывания цикла (continue).

В общем как-то так. Скажите, насколько это решение было благоразумным?

2 ответа 2

2

В зависимости от ситуации и типа приложения.

По мне так в GUI-приложениях лучше всего сделать какой-нибудь ErrorNotificationManager, которому следует сообщать ошибку по мере отлова, а уж он пусть принимает решение, как отображать её. В момент передачи менеджеру можно и передавать аналог лог-левела, чтобы намекнуть ему о критичности ошибки в данном конкретном контексте.

Если же речь идёт о логгировании, то просто логгируйте с нужным логлевелом и получите то что надо.

UPD1

Я думаю, что нельзя дать общую рекомендацию о гранулярности try-catch на все случаи жизни. Всегда приходится искать разумный баланс. В некоторых случаях разумно передавать ошибку в виде колл-бэка. Как, например, это сделано в GWT AsyncCallback. В таких случаях, вызывающий точно знает насколько ошибка фатальна.

UPD2

Что вы, я мало кого считаю глупым :)

Итак, у нас имеется три вида ошибок: критические, просто ошибки и ошибки уровня warning.

О критических ошибках. В случае подобных ошибок можно заставить handler всё завершать (можно даже жёстко через System.exit). С другой стороны, можно протащить такую ошибку сквозь все уровни или сделать её от RuntimeException, чтобы не пришлось всюду протягивать её через throws. А ловить тогда прямо в корне. Если нет потоков, то на мой вкус второй вариант лучше. Если есть, то можно скомбинировать оба подхода (только без System.exit)

Просто ошибки и ворнинги. Если ворнинг не связан с необходимостью прервать некую деятельность, то можно и вовсе не делать исключения (думаю, это и так понятно). В остальных случаях ваш вариант вполне пригоден, хотя на мой вкус можно было бы оформить обработку поаккуратнее

Рассматривая ваш пример, мне кажется, его можно сделать красивее и тогда сомнения по поводу такого подхода могут быть развеяны:

public void startPreparations(List<MyDataEntity> dataEntityList) throws MyFatalPreparationException {
    for(MyDataEntity dataEntity : dataEntityList) {
       try {
           processId(getId(dataEntity));
       } catch (MyDataEntityException mdee) {
          MyExceptionHandler(mdee, LogLevel.ERROR);
       }
    }
}

private void processId(MyDataEntity dataEntity) throws MyFatalPreparationException {
    // ...
}
private int getId(MyDataEntity dataEntity) throws MyDataEntityException {
    //... throw (new ...) где-то тут
}
4
  • Так и было сделано, только вот вопрос, как быть, когда, например, вложенность вызовов функций достаточно велика? Что выбрать: отлов ситуации «на месте» (try-catch-finally), передача иксепшн в вызвавшую функцию (throws)?
    – Dex
    15 мар 2011 в 18:43
  • см. UPD1
    – cy6erGn0m
    15 мар 2011 в 18:59
  • Посмотрите пожалуйста мой UPD в вопросе.
    – Dex
    15 мар 2011 в 20:14
  • Спасибо:) В общем-то я и не хотел использовать System.exit поэтому и завернул все это так.
    – Dex
    15 мар 2011 в 22:04
-1

«критические ошибки» - при возникновении..

Intel® 64 and IA-32 Architectures Software Developer’s Manual Volume 3A, 5.5 EXCEPTION CLASSIFICATIONS: "Exceptions are classified as faults, traps, or aborts..". Матчасть.

Раз скрипт значит юзермод, посему есть три механизма: глобальный VEH, локальный SEH и глобальный финальный UEF. Это под NT. В убогом линусе походу есть тока финальный хэндлер, локально в этом поделии сепшин низя захэндлить :D

4
  • 1
    Я ничего не понял. Речь идёт о яве или нет?
    – cy6erGn0m
    15 мар 2011 в 17:44
  • Спасибо за матчасть, рад, что Вы разбираетесь в вопросе, но, к сожалению, не в моем. Ибо я не просил учить меня «жить», а спросил либо конкретную реализацию, применительно к выбранному языку програмиирования, либо пояснение, причем, не менее конкретное.
    – Dex
    15 мар 2011 в 18:40
  • Конкретное пояснение: механизм исключений локален для оси и не зависит от компилятора, таким образом выбрать золотую середину не получится. Эти механизмы привязаны к архитектуре, посему будут проблемы с использованием.
    – Indy
    16 мар 2011 в 9:38
  • Блин, речь идёт о яве. Забудьте про компилятор, ось и какие-то особенности обработки исключений. В яве исключения работают всегда одинаково.
    – cy6erGn0m
    16 мар 2011 в 10:41

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.