Какие эксэпшены использовать?

Question

[NoMoneyException]

Привет) Какой ход мыслей/алгоритм должен быть при выборе Exception'a?
Вот я например написал класс:

import java.util.Scanner;

public class Scan {
    private  Scanner scanner = new Scanner(System.in);

    public int scanInt() {
        try{
            return scanner.nextInt();
        }
        catch (Exception exc){
            return 0;
        }
    }
}

Не обращайте внимания на код пожалуйста, это пример. С чего бы мне использовать Exception, а не сразу Throwable? Может все ошибки сразу перехватывать через Throwable? Какой смысл использования более низких в иерархии исключений? И еще вопрос касательно этого примера) Стоит использовать Error-handling внутри метода, или же использовать throws и отловить в main() ? Чем руководствоваться?
Спасибо!

Answer 1

[Дмитрий Александров]

Если класс не делает ничего критически важного в программе то можно и нужно обрабатывать все исключения внутри этого класса. Если же класс выполняет какие то критические действия то вероятнее всего обрабатывать исклюения внутри класса может быть очень опасно и лучше их ловить уровнем выше.
Не критичный пример: класс рисует картинку на экран, если пара пикселей будет каким либо образом при обработке null и ты перехватишь это исключение заменив ошибочный пиксель белым то ничего криминального не случится. А вот если ты будешь ловить исключение уровнем выше то будет много лишнего гемороя и кода.
Критичный пример: класс работает с базой данных в который хранятся данные о людях, происходит ошибка, скажем, с доступом к бд, ты поймал исключения внутри самого класса и вместо того чтобы вернуть, например, "Иванова А.П.", у тебя случилось исключение и ты вместо того чтобы выкинуть исключение вверх прикидываешься шлангом и возвращаешь что таких нет. Код где то дальше видя что все нормально и "Иванова А.П." нет в базе спокойно создает там нового(соединение с бд могло восстановиться к этому моменту) вместо того чтобы обновить запись. Т.е. в такой ситуации обрабатывать исключения внутри класса просто неправильно в большинстве случаев т.к. железо бетонно его надо обрабатывать выше и в случае ошибки обращения к бд отвалиться и пробовать все снова.

Answer 2

[Rou1997]

Стоит использовать Error-handling внутри метода, или же использовать throws и отловить в main() ?

В данном примере вы пишете свой Util-класс, конечно в нем нужно исправить недочет разработчиков Java, ведь nextInt который выбрасывает проверяемое (checked) исключение это явный перегиб, который сильно усложняет код, поэтому делайте обработку внутри своего Util-метода, в случае исключения делайте возврат null, можете разработать и применить свой механизм логирования, более гибкий и удобный.

Comments

[Artem Gapchenko]

Как раз таки усложнения кода тут никакого не происходит, наоборот упрощение. Если у вас в функции десяток строчек, каждая из которых потенциально может выбросить исключение, то проще написать все эти десять строчек, обернуть их в try, а затем в отдельных catch clauses писать реакцию на выброшенное исключение. Если же все методы в случае ошибки будут возвращать null-значение (или какой-то особый код, описанный в контракте метода), то у вас нормальный код смешается с кодом по обработке ошибок, то есть нормальный flow метода будет перемежаться конструкциями вида if (result == null) { doSomething(); }, что сильно усложняет чтение кода.
Более того, в случае nextInt() могут быть выброшены три разных исключения, и обрабатывать их нужно по-разному. Если вы будете возвращать null, то потеряете ценную информацию о том, что случилось, и не сможете описать правильную реакцию на возникшую ошибку.

[Rou1997]

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

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

Rou1997: Может ты тогда "лично свой" код будешь оставлять при себе, а не писать, что это правильно и верно?

[Rou1997]

Денис Загаевский:
int x;
do {
  print("Твои аргументы? ");
  boolean b = true;
  try {
    x = nextInt();
  } catch (Exception e) {
    b = false;
  }
} while (b != true);

int x;
do {
  print("Твои аргументы? ");
  x = org.rou1997.internal.Console.readInt();
} while (x == null);