Сразу оговорюсь что нет золотого правила о том где и когда обрабатывать Exception, а следовательно всё это довольно субъективно.
Лично я стараюсь обрабатывать ошибку там, где это требуется логикой программы либо в месте ответственности за вызванную ошибку. Постараюсь объяснить свою мысль на абстрактном примере.Представим что есть математическая библиотека у которой есть метод для решения квадратного уравнения.
public Pair solveQuadratic (string input){
Quadratic quadratic = Quadratic.parse(input);
Pair result;
result = tryVieta(quadratic);
result = tryDiscriminant(quadratic);
return result;
}
В данном примере у нас возможны 3 ошибки:
- ParseException, когда мы преобразуем строку в объект
- InvalidParameter, когда мы пытаемся вызвать решение методом Виета для уравнения, которое нельзя решить теоремой Виета
- UnsolvedQuadraticException, если уравнение оказалось не решаемое
Первую ошибку надо выбрасывать методу, который его вызвал, т.к. ни метод решающий уравнения, ни математическая либа не хотят знать что делать если строка не верна. Это задача того, кто метод вызвал. В каком то случае главный процесс ещё раз запросит ввод у пользователя, в каком то преобразует строку, либо вообще заинициализирует Pair нулями. В любом случае, это не головняк приведенного в примере метода.
Вторую ошибку следует обрабатывать внутри метода, так как снаружи не должно быть известно и вообще не волнует, как именно решит уравнение метод, важен результат. Если запихнуть в метод корректное уравнение, а метод вернет что то типа "Это уравнение не вида Виета", то это будет нарушением ответственности за ошибку.
Обработка третьей ошибки зависит от вашего представления, что должно вернуться, если уравнение не решаемое: null или Exception. Если считаете что null, то ошибка обрабатывается внутри, а иначе иначе)) Я преподчитаю выбрасывать Exception
По итогу наш метод превратится в следующий
public Pair solveQuadratic (string input) throw ParseException, UnsolvedQuadraticException{
Quadratic quadratic = Quadratic.parse(input);
Pair result;
try{
result = tryVieta(quadratic);
}catch(InvalidParameterException e){
result = tryDiscriminant(quadratic);
}
return result;
}
В приведенном вами примере ошибки нужно выбрасывать, и обрабатывать снаружи, особенно если стоит задача написать эффективные тесты.