(이펙티브 자바 3) 27. 비검사 경고를 제거하라

할 수 있는 한 모든 비검사 경고를 제거하라

제네릭을 사용하기 시작하면 수많은 컴파일러 경고를 보게 된다.

• 비검사 형변환 경고
• 비검사 메서드 호출 경고
• 비검사 매개변수화 가변인수 타입 경고
• 비검사 변환 경고

할 수 있는 한 모든 비검사 경고를 제거하라. 모두 제거한다면 그 코드는 타입 안전성이 보장된다.
즉, 런타임에 ClassCastException이 발생할 일이 없으며, 의도한 대로 잘 동작한다.

@SuppressWarning

경고를 제거할 수는 없지만 타입 안전하다고 확신할 수 있다면 @SuppressWaring("unchecked") 애너테이션을 달아 경고를 숨기자.

• Type Safety를 검증하지 않은 채 경고를 숨기면 스스로에게 잘못된 보안 인식을 심어주게 된다.
• 코드는 경고 없이 컴파일되겠지만, 런타임에는 여전히 ClassCastException을 던질 수 있다.
• 한편, 안전하다고 검증된 비검사 경로를 숨기지 않고 그대로 두면, 진짜 문제를 알리는 새로운 경고가 나와도 눈치채지 못할 수 있다.

@SuppressWarings 애너테이션은 항상 가능한 한 좁은 범위에 적용하자.

• 보통은 변수 선언, 아주 짧은 메서드, 혹은 생성자가에 선언한다.
• 심각한 경고를 놓칠 수 있으니 절대로 클래스 전체에 적용해서는 안 된다.
• 애너테이션은 선언에만 달 수 있기 때문에 범위가 필요 이상으로 넓어지면 반환 값을 담을 지역변수를 하나 선언하고 그 변수에 애너테이션을 달아주자.

@SuppressWaring("unckecked") 애너테이션을 사용할 때면 그 경고를 무시해도 안전한 이유를 반드시 주석으로 남겨야 한다.

• 다른 사람이 그 코드를 이해하는데 도움이 되며, 다른 사람이 그 코드를 잘못 수정하여 타입 안정성을 잃는 상황을 줄여준다.

728x90

정리

비검사 경고는 중요하니 무시하지 말자. 모든 비검사 경고는 런타임에 ClassCastException을 일으킬 수 있으니 최선을 다해 제거하자. 경고를 없애지 못한다면 검증 후 @SuppressWaring 애너테이션으로 경고를 숨겨라.