[이펙티브 자바] Item27 비검사 경고를 제거하라

핵심 정리

 

비검사 경고는 중요하니 무시하지 말자.
모든 비검사 경고는 런타임에 ClassCastException을 일으킬 수 있는 잠재적 가능성을 뜻하니 최선을 다해 제거하라.

 

 

비검사 경고를 제거하는 방법

1. 컴파일러가 알려준 대로 수정하면 경고가 사라진다. 
2. 경고를 제거할 수는 없지만 타입 안전하다고 확신할 수 있다면 @SuppressWarnings("unchecked") 애너테이션을 달아 경고를 숨기자.
   
   • 단, 타입 안전함을 검증하지 않은 채 경고를 숨기면 스스로에게 잘못된 보안 인식을 심어주는 꼴이다.
   • 안전하다고 검증된 비검사 경고를 (숨기지 않고) 그대로 두면, 진짜 문제를 알리는 새로운 경고가 나와도 눈치채지 못할 수 있다.
   • @SuppressWarnings 애너테이션은 항상 가능 한 좁은 범위에 적용하자. 자칫 심각한 경고를 놓칠 수 있으니 절대로 클래스 전체에 적용해서는 안 된다.
   • 한 줄이 넘는 메서드나 생성자에 달린 @SuppressWarnings 애너테이션을 발견하면 지역변수 선언 쪽으로 옮기자.
   • 사용 할 때면 그 경고를 무시해도 안전한 이유를 항상 주석으로 남겨야 한다.

// 컴파일시 에러 발생
// return (T[]) Arrays.copyOf(elements, size, a.getClass());
public <T> T[] toArray(T[] a) {
    if (a.length < size)
    	return (T[]) Arrays.copyOf(elements, size, a.getClass());
    System.arraycopy(elements, 0, a, 0, size);
    if (a.length > size)
    	a[size] = null;
    return a;
}

// 메서드 전체에 달지 않고,
// 지역 변수를 추가해 @SuppressWarnings의 범위를 좁힌다.
public <T> T[] toArray(T[] a) {
    if (a.length < size) {
        // 생성한 배열과 매개변수로 받은 배열의 타입이 모두 T[]로 같으므로
        // 올바른 형변환이다.
        @SuppressWarnings("unchecked") T[] result =
        	(T[]) Arrays.copyOf(elements, size, a.getClass());
        return result;
    }
    System.arraycopy(elements, 0, a, 0, size);
    if (a.length > size)
    	a[size] = null;
    return a;
}