Обернутое исключение из компонента

Это чертовски уродливо. Думаю, особо нечего комментировать этот уродливый стиль. Если у вас уже есть весь код для решения различных проблем с использованием причины, это сработает. Это просто использование

try {
  componentCall();
} catch (ComponentException e) {
  Throwable t = e.getCause();
  //Handle each possible cause
}

менее удобен в обслуживании и более запутан, чем другой способ, и главное, что я не вижу преимуществ в его использовании.

Учитывая, что вы ограничены в его использовании, я бы, по крайней мере, попытался избежать выброса ComponentException как с причиной, так и без нее, что делает его более запутанным, чем должно быть, в вашем примере я бы добавил InvalidFooException и добавил его как причина общего ComponentException.

try {
    int foo = foo();
    if (foo != expectedValue) {
        throw new InvalidFooException("bad result from foo(): " + foo);
    }
    bar();
}
catch (Exception e) { 
    throw new ComponentException(e); 
}

getCause () предназначен для цепочки исключений, которые связаны только причинно, а не семантически.

Если вам нужно, чтобы они различали разные исключения (например, SQLExecutionException должно быть исправлено другим действием, отличным от AccessDeniedException), тогда ваш метод неприменим (потому что принудительное использование getCause () в каждом улове, чтобы увидеть, что должно быть done ужасен и не предлагает никакой выгоды, кроме как просто перехватить правильное исключение).

Если все вызывающие классы должны сообщить об ошибке и отменить, тогда упаковка может быть в порядке, но я бы не стал этого делать, так как это приносит мало пользы, и если вам позже понадобится дифференцировать, вы переписываете много вещей.

Что полезно, так это создать иерархия исключений для исключения того же типа в строках:

Если DataAccessException будет корнем, то у вас может быть, например, DataSourceUnavailableException, InvalidDataSourceException, InvalidDataException, а затем вы можете решить перехватить только одного из родителей (если действие одинаково) или перехватить каждое исключение отдельно.

Кстати, зачем вам это, чтобы так себя вести?

Возможно, это было бы менее уродливо (если функция foo () является той, которая была объявлена ​​как вызывающая исключение):

int foo;
try {
    foo = foo();
}
catch (Exception e) { 
    throw new ComponentException(e); 
}
if (foo != expectedValue) {
    throw new ComponentException("bad result from foo(): " + foo);
}
bar();