Пусть есть метод, который может бросить проверяемое исключение при некорректных входных параметрах
void throwMethod(int count) throws MyException {
if (count < 0)
throw new MyException(...);
.......
}
и есть вызывающий метод, который передает гарантировано правильный параметр
void callMethod() {
int count = 15;
throwMethod(count);
}
Компилятор говорит, что callMethod
должен декларировать возможность возникновения исключения MyException
. Т.е. у меня есть два пути: либо задекларировать это
void callMethod() throws MyException {
int count = 15;
throwMethod(count);
}
либо поймать его
void callMethod() {
int count = 15;
try {
throwMethod(count);
} catch (MyException e) {}
}
первый вариант мне не нравится, т.к. декларируется событие, которое никогда не произойдет. Во втором варианте закапываются грабли, которые могут всплыть при рефакторинге. Скажем, в методе throwMethod
усилится проверка
void throwMethod(int count) throws MyException {
if (count < 0 && count > 10)
throw new MyException(...);
.......
}
или вызываемый метод перепишется
void callMethod() {
int count = 15 - List.count;
try {
throwMethod(count);
} catch (MyException e) {}
}
тогда исключение потеряется и ошибку придется искать долго. Ну и кроме того, у меня стабильное отвращение перед пустой catch
секцией.
Есть еще мысль, выбросить непроверяемое исключение
void callMethod() {
int count = 15;
try {
throwMethod(count);
} catch (MyException e) {
throw new RuntimeException(e);
}
}
Тогда если сейчас исключений не может быть, тогда и RuntimeException
не будет. А если что-то поменяется, то хоть ошибку увидим. Но это попахивает каким-то костылем.
Вопрос - как делать правильно?
UPDATE
Вопрос возник по мотивам конструктора класса URL(String)
, который может бросить MalformedURLException
если в строке не указан протокол, указан неизвестный протокол или передан null. И я создаю объект кодом
URL url = new URL('http://google.com/');