item 27 dodo4513

아이템27 비검사 경고를 제거하라

• 대부분의 비검사 경고는 아래와 같 쉽게 제거할 수 있다.

// bad
Set<Lark> exaltation = new HashSet();

// good
Set<Lark> exaltation = new HashSet<>();

• 제거하기 훨씬 어려운 경고도 있다. 곧바로 해결되지 않는 경고가 나타나도 포기하지 말자! 할 수 있는 한 모든 비검사 경고를 제거하라. 모두 제거한다면 그 코드는 타입 안전성이 보장된다. 즉, 런타임에 ClassCastException이 발생할 일이 없고, 여러분이 의도한 대로 잘 동작하리라 확신할 수 있다.

만약에 제거하기 힘들다면?

@Suppress
Warnings("unchecked")

• 경고를 제거할 수는 없지만 타입 안전하다고 확신할 수 있다면 위 애너테이션을 달아 경고를 숨기자.
• @SuppressWarnings 애너테이션은 개별 지역변수 선언부터 클래스 전체까지 어떤 선언에도 달 수 있다. 하지만 @SuppressWarnings 애너테이션은 항상 가능한 한 좁은 범위에 적용하자. 자칫 심각한 경고를 놓칠 수 있으니 절대로 클래스 전체에 적용해서는 안 된다.
• @SuppressWarnings("unchecked") 애너테이션을 사용할 때면 그 경고를 무시해도 안전한 이유를 항상 주석으로 남겨야 한다.

핵심정리

비검사 경고는 중요하니 무시하지 말자.
모든 비검사 경고는 런타임에 ClassCastException을 일으킬 수 있는 잠재적 가능성을 뜻하니 최선을 다해 제거하라.
경고를 없앨 방법을 찾지 못하겠다면, 그 코드가 타입 안전함을 증명하고 가능한 한 범위를 좁혀 @SuppressWarnings("unchecked") 애너테이션으로 경고를 숨겨라.
그런 다음 경고를 숨기기로 한 근거를 주석으로 남겨라.