5-1 제네릭을 그냥 Raw 로 사용하지 마라(+ 비검사 경고제거)

Raw 타입이란 제네릭 변수<T>를 사용할 때 적지 않고 그냥 사용하는 경우를 말합니다.

이는 JDK 1.5 이하 버전의 호환성을 위해 남겨둔 기능이지, 절대 사용해서는 안됩니다. 타입안정성이 보장되지 않습니다.

private final Collection stamps = ...;
stamps.add(new Coin(...)); // stamp 타입에 coin 타입을 넣고 있음.
// RAW 타입을 사용해도 unchecked call "경고"를 호출하지만 컴파일도 되고 실행도 됩니다.

private final Collection<Stamp> stamps = ...;
stamps.add(new Coin()); // 올바른 사용방법. 컴파일 오류가 발생합니다.

 

# 뭐든지 받는 Object 타입이 필요하다면 와일드카드를 쓰세요.

 List<Object> 대신에 List<?> 를 사용하면 타입 안정성을 보장합니다. 타입이 다른 객체가 섞여 들어가는걸 방지합니다.

 

 

# 다만 클래스 리터널(Class.class)은 예외입니다. 

제네릭은 자바 코드상에서 컴파일 이전까지에만 존재하고 바이트코드로 변환되면 없어지는 코드입니다. (Object로 변환됨) 그래서 만약 List<String>.class 같은 클래스 리터널은 사용이 불가능합니다. 제네릭이더라도 Raw 타입으로 List.class를 사용해야 합니다.

 

참고로 이는 instanceof 연산자와도 관련 있습니다. 런타임에는 제네릭정보가 지워지므로
o instanceof Set<MyClass> 같은 건 불가능합니다. 필요하다면 아래와 같이 RAW 타입인 Set으로 사용해야합니다.

// o의 타입이 Set인지 확인한 다음, 와일드카드 타입으로 형변환해야 합니다.
// 여기서 로 타입인 Set이 아닌 와일드카드 <?> 타입으로 변환함에 주의!
if( o instanceof Set) {
    Set<?> s = (Set<?>) o;
}

 

 

# 비검사 경고를 제거하라

컴파일 경고가 나온김에 하는 이야기인데, 비검사 경고는 제거하는게 맞습니다.

클래스를 개발할 때, 해당 클래스를 컴파일 할 때 나오는 경고가 전혀 문제가 될 일이 없다면 @SuppressWaring으로 경고를 제거해주세요. 굳이 필요없는 주의사항을 다른 개발자에게 전파할 필요는 없습니다.

 

물론 가능한 좁은범위로 경고를 제거하고, 왜 제거했는지 주석을 달아주는게 좋다.

   public <T> T[] toArray(T[] a){
        
        @SuppressWarnings("unchecked") T[] result = (T[]) Arrays.copyOf(a, a.length, a.getClass());
        
        ...
    }