[교재 EffectiveJava] 아이템 27. 비검사 경고를 제거하라

비검사 경고

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

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

 

오류 코드 (경고 발생)

Set<Test> test = new HashSet();

/**
    [unckecked] unchecked conversion 
    required : Set<Test>
    found : HashSet
*/

 

수정 코드

Set<Test> test = new HashSet<>();

 

Java7 부터 지원하는 다이아몬드 연산자(<>)로 해결하였다. 컴파일러가 올바른 실제 타입 매개변수를 추론한다. (Test)

 

 

비검사 경고 제거 및 무시

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

@SuppressWarnings("unchecked")
...

 

경고를 제거할 수 없지만 타입이 안전하다고 확신이 들면 위 어노테이션을 사용하여 경고를 숨기자. 해당 어노테이션은 개별 지역변수 선언부터 클래스 전체까지 어떤 선언에도 달 수 있다. 하지만 @SuppressWarnings 어노테이션은 항상 가능한 한 좁은 범위에 적용하자. 보통은 변수 선언, 아주 짧은 메서드, 혹은 생성자가 될 것이다. 심각한 경고를 놓칠 수 있으니 클래스 전체에 적용해서는 안된다. 

 

 

지역변수 위에 @SuppressWarnings 어노테이션 선언

한 줄이 넘는 메서드나 생성자에 달린 @SuppressWarnings 어노테이션을 발견하면 지역변수 선언 쪽으로 옮기자. 이를 위해 지역변수를 새로 선언해야 하지만, 장점이 더 많다.

 

ArrayList.java

....
    @SuppressWarnings("unchecked")
    public <T> T[] toArray(T[] a) {
        if (a.length < size)
            // Make a new array of a's runtime type, but my contents:
            return (T[]) Arrays.copyOf(elementData, size, a.getClass());
        System.arraycopy(elementData, 0, a, 0, size);
        if (a.length > size)
            a[size] = null;
        return a;
    }
....

 

ArrayList.java 의 toArray() 메서드에 @SuppressWarnings 어노테이션을 지역변수로 옮겨보자.

 

ArrayList.java

....
    public <T> T[] toArray(T[] a) {
        if (a.length < size)
            @SuppressWarnings("unchecked")
            T[] result = (T[]) Arrays.copyOf(elementData, size, a.getClass());
            // Make a new array of a's runtime type, but my contents:
            return result;
        System.arraycopy(elementData, 0, a, 0, size);
        if (a.length > size)
            a[size] = null;
        return a;
    }
....

 

return 문에는 어노테이션을 달 수 없으니, 지역변수를 하나 선언하고 그 변수에 어노테이션을 달자. 위 코드로 변경되면서, 경고를 숨기는 범위를 최소화했다. @SuppressWarnings("unchecked") 어노테이션을 사용할 때면 그 경고를 무시해도 안전한 이유를 항상 주석으로 남겨야한다.