[EffectiveJava] item77 - 예외를 무시하지 말라

너무 뻔한 조언 같지만 반복해 각인시켜야 할 정도로 사람들이 자주 어기는 방법이 있다.

API 설계자가 메서드 선언에 예외를 명시하는 까닭은, 그 메서드를 사용할 때 적절한 조치를 취해달라고 말하는 것이다.

예외 무시

안타깝게도 예외를 무시하기란 아주 쉽다. 해당 메서드 호출을 try 문으로 감싼 후 catch 블록에서 아무 일도 하지 않으면 끝이다.

/* catch 블록을 비워두면 예외가 무시된다. 아주 의심스러운 코드다! */
try {
    ...
} catch (SomeException e) {
}

예외는 문제 상황에 잘 대처라기 위해 존재하는데 catch 블록을 비워두면 예외가 존재할 이유가 없어진다.

비유하자면 화재경보를 무시하는 수준을 넘어 아예 꺼버려, 다른 누구도 화재가 발생했음을 알지 못하게 하는 것과 같다.

예외를 무시해야할 때

예외를 무시해야 할 때도 있다. 예를 들면 FileInputStream을 닫을 때가 그렇다.

파일의 상태를 변경하지 않았으니 복구 할 것이 없으며, 필요한 정보는 이미 다 읽었다는 뜻이니 남은 작업을 중단할 이유도 없다.

같은 예외가 자주 발생한다면 조사해보는 것이 좋을 테니 파이을 닫지 못했다는 사실을 로그로 남기는 것도 좋은 생각이다.

예외를 무시하기로 했다면 catch 블록 안에 그렇게 결정한 이유를 주석으로 남기고 예외 변수의 이름도 ignored로 바꿔놓도록 하자.

Future<Integer> f = exec.submit(planarMap::chromaticNumber);
int numColors = 4; /* 기본값. 어떤 지도라도 이 값이면 충분하다. */
try {
    numColors = f.get(1L, TimeUnit.SECONDS);
} catch (TimeoutException | ExcutionException ignored) {
    /* 기본값을 사용한다(색상 수를 최소화 하면 좋지만, 필수는 아니다.) */
}

이번 절의 내용은 검사와 비검사 예외에 똑같이 적용된다. 예측할 수 있는 예외 상황이든 프로그래밍 오류든, 빈 catch 블록으로 못 본 척 지나치면 그 프로그램은 오류를 내재한 채 동작하게 된다. 그러다 어느 순간 문제의 원인과 아무 상관없는 곳에서 갑자기 죽어버릴 수도 있다.

무시하지 않고 바깥으로 전파되게만 놔둬도 최소한 디버깅 정보를 남긴 채 프로그램이 신속히 중단되게는 할 수 있다.