Tôi đã được chỉ định để duy trì một ứng dụng được viết trước đây bởi các nhà phát triển lành nghề hơn. Tôi đã xem qua đoạn mã này:
public Configuration retrieveUserMailConfiguration(Long id) throws MailException {
try {
return translate(mailManagementService.retrieveUserMailConfiguration(id));
} catch (Exception e) {
rethrow(e);
}
throw new RuntimeException("cannot reach here");
}
Tôi tò mò nếu ném RuntimeException("cannot reach here")là hợp lý. Có lẽ tôi đang thiếu một cái gì đó rõ ràng khi biết rằng đoạn mã này đến từ đồng nghiệp dày dạn hơn.
EDIT: Đây là cơ thể suy nghĩ lại mà một số câu trả lời đề cập đến. Tôi cho rằng nó không quan trọng trong câu hỏi này.
private void rethrow(Exception e) throws MailException {
if (e instanceof InvalidDataException) {
InvalidDataException ex = (InvalidDataException) e;
rethrow(ex);
}
if (e instanceof EntityAlreadyExistsException) {
EntityAlreadyExistsException ex = (EntityAlreadyExistsException) e;
rethrow(ex);
}
if (e instanceof EntityNotFoundException) {
EntityNotFoundException ex = (EntityNotFoundException) e;
rethrow(ex);
}
if (e instanceof NoPermissionException) {
NoPermissionException ex = (NoPermissionException) e;
rethrow(ex);
}
if (e instanceof ServiceUnavailableException) {
ServiceUnavailableException ex = (ServiceUnavailableException) e;
rethrow(ex);
}
LOG.error("internal error, original exception", e);
throw new MailUnexpectedException();
}
private void rethrow(ServiceUnavailableException e) throws
MailServiceUnavailableException {
throw new MailServiceUnavailableException();
}
private void rethrow(NoPermissionException e) throws PersonNotAuthorizedException {
throw new PersonNotAuthorizedException();
}
private void rethrow(InvalidDataException e) throws
MailInvalidIdException, MailLoginNotAvailableException,
MailInvalidLoginException, MailInvalidPasswordException,
MailInvalidEmailException {
switch (e.getDetail()) {
case ID_INVALID:
throw new MailInvalidIdException();
case LOGIN_INVALID:
throw new MailInvalidLoginException();
case LOGIN_NOT_ALLOWED:
throw new MailLoginNotAvailableException();
case PASSWORD_INVALID:
throw new MailInvalidPasswordException();
case EMAIL_INVALID:
throw new MailInvalidEmailException();
}
}
private void rethrow(EntityAlreadyExistsException e)
throws MailLoginNotAvailableException, MailEmailAddressAlreadyForwardedToException {
switch (e.getDetail()) {
case LOGIN_ALREADY_TAKEN:
throw new MailLoginNotAvailableException();
case EMAIL_ADDRESS_ALREADY_FORWARDED_TO:
throw new MailEmailAddressAlreadyForwardedToException();
}
}
private void rethrow(EntityNotFoundException e) throws
MailAccountNotCreatedException,
MailAliasNotCreatedException {
switch (e.getDetail()) {
case ACCOUNT_NOT_FOUND:
throw new MailAccountNotCreatedException();
case ALIAS_NOT_FOUND:
throw new MailAliasNotCreatedException();
}
}
Bên cạnh đó, một AssertsError có thể là một lựa chọn tốt hơn về mặt ngữ nghĩa so với RuntimeException.
—
JvR
Cú ném cuối cùng là dư thừa. Nếu bạn bật findbugs hoặc các công cụ phân tích tĩnh khác, nó sẽ gắn cờ các loại đường này để xóa.
—
Bon Ami
Bây giờ tôi thấy phần còn lại của mã, tôi nghĩ rằng có lẽ bạn nên thay đổi "nhà phát triển lành nghề hơn" thành " nhà phát triển cao cấp hơn ". Đánh giá mã sẽ được hạnh phúc để giải thích tại sao.
—
JvR
Tôi đã cố gắng tạo ra các ví dụ giả thuyết về lý do tại sao các ngoại lệ là khủng khiếp. Nhưng không có gì tôi từng làm giữ một ngọn nến cho điều này.
—
Paul Draper
rethrowkhông thực sựthrowlà một ngoại lệ. (điều này có thể xảy ra vào một ngày nào đó, nếu việc triển khai thay đổi)