Item27 - 비검사 경고를 제거하라

 

비검사(unchecked)경고 : 컴파일러가 타입 안정성을 확인하는데 필요한 정보가 충분치 않을 때 발생시키는 경고

 

 

제네릭을 사용하기 시작하면 수많은 컴파일러 경고를 보게 된다. 비검사 형변환 경고, 비검사 메서드 호출 경고, 비검사 매개변수화 가변인수 타입 경고, 비검사 변화 경고 등이다.

 

 

Set<String> strings = new HashSet(); => Set<String> strings = new HashSet<String>();

컴파일러가 지시하는 대로 수정하면 대부분의 경고는 사라진다.

 

제거하기 어려운 경고도 있다. 곧바로 해결되지 않는 경고가 나타나도 할 수 있는 모든 비검사 경고를 제거하자

 => 타입 안전성을 보장하기 위해!

 

경고를 제거할 수 없지만 안전하다고 확신한다면 @SuppressWarnings("unchecked") 어노테이션을 달아 경고를 숨기자

 이러면 컴파일러의 경고가 뜨지 않는다. 만일 @ SuppressWarnings 어노테이션을 달지 않는다면 컴파일러의 경고가 어디서 발생하는지 헷갈릴 수 있다. 다만 어노테이션을 붙이기 전에 반드시 타입 안전 검증을 해야 한다.

 

@SuppressWarnings 어노테이션은 로컬 변수부터 클래스 전체까지 어떤 선언에도 달 수 있다. 하지만 가능한 한 좁은 범위에 적용하자

 클래스 전체에 적용을 하면 자칫 심각한 경고를 놓칠 수도 있기 때문이다.

 

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

    @SuppressWarnings("unchecked")
    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;
    }
    
    ====================================================================
    
    public <T> T[] toArray(T[] a) {
        if (a.length < size) {
            /**
             *  T[] 데이터 타입으로 매개변수를 받기 때문에
             *  @SuppressWarnings 애너테이션을 붙여 비검사 경고를 제거한다.
             */
            @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;
    }

 

 

 

@SuppressWarnings ("unchecked")  어노테이션을 사용할 때면 그 경고를 무시해도 안전한 이유를 항상 주석으로 남겨 다른 사람이 코드를 이해하는 데 도움이 되게 하자.