Обычно вы хотите сконцентрировать обработку ошибок для конкретной ошибки в одном месте.Таким образом, вы либо выполняете здесь всю обработку ошибок, а затем не перебрасываете свое исключение или просто не перехватываете его и не позволяете ему перехватить свое первоначальное место на более высоких уровнях и полностью обработать его.Иногда делается то, что вы ловите определенный тип исключения, а затем оборачиваете его в другое исключение, и это приемлемое поведение:
try {
doSomething();
} catch (IOException e) {
doSomethingElse();
throw new MyCustomException("My custom error message", e);
}
Таким образом, ваш случай кажется плохим.Однако я не хочу исключать это на 100%, так как это может быть действительно редкий случай, когда вам может понадобиться что-то подобное.Но, как обычно, я бы сказал, что это плохая практика