Считается ли хорошим подходом создавать исключения в Java?

Question

[UltimateOrb]

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

1. Через исключения

// callback от сканера
public void callbackScaner(String data) {
  try {
     scanBarcode(String barcode);
   } catch (Exeprion e) {
      showMsg(e.getMessage());
   }
}

// обработка бар-кода
public void scanBarcode(String barcode) throws Exception {
   Box box= getProd(barcode);
   if (box== null) throw new Exeption("Коробки нет на складе");
   if (box.ready) throw new Exeption("Коробка уже отсканирована");
   ...   
}

2. Через проверку

// callback от сканера
public void callbackScaner(String data) {
     scanBarcode(String barcode);
}

// обработка бар-кода
public void scanBarcode(String barcode) throws Exception {
   Box box= getProd(barcode);
   if (box== null) {
      showMsg("Коробки нет на складе");
      return;
   }
   if (box.ready) {
      showMsg("Коробка уже отсканирована");
      return;
   }
   ...   
}

Честно говоря мне нравится первый подход. Код более лаконичный. Знаю что плохо закладывать логику в блок catch {}, но здесь ее вроде бы и нет. Судя по другим отзывам, блок try {} не оказывает влияние на производительность, если не происходит исключений.

Ваше мнение?

Answer 1

[aol-nnov]

иди дальше!
throw new BoxNotFoundException
throw BoxAlreadyScannedException

:)

Comments

[UltimateOrb]

Судя по смайлику это был стеб? Для этого и был задан вопрос, что бы узнать как делать правильно

[aol-nnov]

UltimateOrb: отнюдь. mitekgrishkin уже прокомментировал это.

[UltimateOrb]

aol-nnov: тогда ясно, спасибо.

[nirvimel]

UltimateOrb: Ничего смешно, это как раз хорошая практика, только важно построить правильное дерево наследования исключений, типа BoxNotFoundException и BoxAlreadyScannedException наследуются от BoxProcessingException, который наследуется от GeneralProcessingException. И отлавливать следует более общие классы, чем объявлены в функции, вызываемой в блоке try. Например: функция oneOfBoxProcessingSpeps бросает BoxNotFoundException, но в том месте, где она вызывается, отлавливается BoxProcessingException. Дальше функция processBox бросает BoxProcessingException, но в том месте, где она вызывается, отлавливается GeneralProcessingException. Это дает огромную гибкость в том, что позволяет в больших пределах менять реализацию конкретной функции/класса, ничего не трогая на вызывающей стороне. Это очень актуально в крупном проекте из кучи модулей.

Answer 2

[mitekgrishkin]

Как вам самому удобно. Единственное - просто кидать Exception - это издевательство над тем, кто будет это дело сопровождать. В вашем случае можно создать что-то типа UserException - это позволит отличать ваши исключения от общих остальных. Заодно подключите вывод сообщения.

Answer 3

[sirs]

if (box== null) throw new Exeption("Коробки нет на складе");
if (box.ready) throw new Exeption("Коробка уже отсканирована");

Куча if не есть хорошо. Почему бы вам просто не ввести понятие State, создать под него Enum со всеми возможными состояниями и применить шаблон, например, Strategy . Под каждое конкретное состояние объекта нужно будет написать отдельный обработчик и из метода кидать не Exception, а просто передавать управление в нужный обработчик.

Как альтернативный вариант - посмотрите фреймворки, который позволяют организовать т.н. Route, может быть вам понравится такая реализация. Если коробка есть на складе - пройти по пути 1, если коробки нет на складе - пройти по пути 2, если с коробки уже считан код - пройти по роуту 3 и т.д. Такая реализация, например, поможет наладить транзакционность из коробки.

Comments

[UltimateOrb]

Проблема в том что, "в любой непонятной ситуации" (т.е. когда нет коробки и др.) я должен просто вывести внятное сообщение пользователю и завершить выполнение функции. Да, при использовании Strategy, у меня будут отдельные обработчики на каждое состояние, но делать они будут одно и тоже.

[sirs]

UltimateOrb: Ну тут уж вам решать, я внутренних деталей не знаю. Но просто написав кучу if и кидая исключения вы поставите себя в трудное положение в будущем. Например сегодня это не нужно, а завтра понадобится залогировать это сообщение в Elasticsearch, Logstash, Kibana ... и т.д. и т.п. и вам придется лазить в этот код и что-то менять, а потом переписывать тесты. А можно просто расширить или добавить новый обработчик, не меняю методов callbackScaner() и scanBarcode().

Answer 4

[Денис Загаевский]

Нет, кидать исключения - однозначно плохой вариант. Само слово "исключение" говорит об этом - ситуации в приложении должны быть не нормой, а исключением для броска исключения. Исходите из этого постулата.

Comments

[UltimateOrb]

Если говорить в целом, то ситуация, когда нужно выкинуть сообщение типа "Коробки нет на складе" и др. не является нормой. Скорее всего это значит что кладовщик сканирует что-то не то.

[Денис Загаевский]

UltimateOrb: для вашей программы "юзер делает что-то не то" должно быть правилом. Они всегда делают что-то не то, и это не повод кидать исключение.