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

wisdom·2022년 8월 18일
0

Effetctive Java

목록 보기
27/80
post-thumbnail

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

모두 제거한다면 그 코드는 타입 안전성이 보장된다. 
즉, 런타임에 ClassCastException이 발생할 일이 없다.

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

안전하다고 검증된 비검사 경고를 숨기지 않고 그대로 두면, 진짜 문제를 알리는 새로운 경고가 나와도 눈치채지 못할 수 있다. 

단, 경고를 무시해도 안전한 이유를 항상 주석으로 남겨야 한다.

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

@SuppressWarning 애너테이션은 개별 지역변수 선언부터 클래스 전체까지 어떤 선언에도 달 수 있다.

하지만 항상 가능한 한 좁은 범위에 적용하자.

보통은 변수 선언, 아주 짧은 메서드, 혹은 생성자가 될 것이다.
자칫 심각한 경고를 놓칠 수 있으니 절대로 클래스 전체에 적용해서는 안 된다.

한 줄이 넘는 메서드나 생성자에 달린 @SuppressWarning 애너테이션을 발견하면 지역변수 선언 쪽으로 옮기자.


📌 핵심 정리

  • 모든 비검사 경고는 런타임에 ClassCastException을 일으킬 수 있는 잠재적 가능성을 뜻하니 최선을 다해 제거하라.
  • 경고를 없앨 방법을 찾지 못하겠다면, 그 코드가 타입 안전함을 증명하고 가능한 한 범우를 좁혀 경고를 숨겨라.(@SuppressWarning("unchecked"))
    그리고 경고를 숨기기로 한 근거를 주석으로 남겨라.
profile
백엔드 개발자

0개의 댓글