[Effective Java/아이템27] 비검사 경고를 제거하라.
2022. 3. 19. 16:12
반응형
# 아이템27. 비검사 경고를 제거하라
제네릭을 사용하기 시작하면 수많은 컴파일러 경고를 보게될 것이다. 비검사 형변환 경고, 비검사 메서드 호출 경고, 비검사 매개변수화 가변인수 타입 경고, 비검사 변환 경고 등이다.
Set<String> set = new HashSet();
$ javac WarningTest.java -Xlint:unchecked
WarningTest.java:8: [unchecked] unchecked conversion
Set<String> set = new HashSet();
^
required: Set<String>
found: HashSet
1 warning
* javac 명령 인수에 -Xlint:uncheck 를 추가하면 해당 에러를 볼 수 있다.
컴파일러가 알려준 대로 수정하면 경고가 사라진다. 사실 컴파일러가 알려준 타입 매개변수를 명시하지 않고, 자바 7부터 지원하는 다이아몬드 연산자(<>)만으로 해결할 수 있다. 그러면 컴파일러가 올바른 실제 타입 매개변수를 추론해준다.
Set<String> set = new HashSet<>();
제거하기 훨씬 어려운 경고도 있다. 할 수 있는 한 모든 비검사 경고를 제거하라. 모두 제거한다면 그 코드는 타입 안정성이 보장된다. 즉, 런타임에 ClassCastException이 발생할 일이 없고, 여러분이 의도한 대로 잘 동작하리라 확신할 수 있다.
경고를 제거할 수는 없지만 타입이 안전하다고 확신할 수 있다면 @SuppressWarning("unchecked") 어노테이션을 달아 경고를 숨기자.
단, 타입 안전함을 검증하지 않은 채 경고를 숨기면 스스로에게 잘못된 보안 인식을 심어주는 꼴이다. 그 코드는 경고없이 컴파일이 되겠지만, 런타임에는 여전히 ClassCastException을 던질 수 있다. 한편 안전하다고 검증된 비검사 경고를 그대로 두면, 진짜 문제를 알리는 새로운 경고가 나와도 눈치채지 못할 수 있다. 제거하지 않은 수많은 거짓 경고 속에 새로운 경고가 파묻힐 것이기 때문이다.
@SuppressWarnings 어노테이션은 개별 지역변수 선언부터 클래스 전체까지 어떤 선언에도 달 수 있다. 하지만 @SuppressWarnings 어노테이션은 항상 가능한 좁은 범위에 적용하자. 보통은 변수 선언, 아주 짧은 메서드, 혹은 생성자가 될 것이다. 자칫 심각한 경고를 놓칠 수 있으니 절대로 클래스 전체에 적용해서는 안된다.
한줄이 넘는 메서드나 생성자에 달린 @SuppressWarnings 어노테이션을 발견하면 지역변수 선언 쪽으로 옮기자.
## 지역변수를 추가해 @SuppressWarning의 범위를 좁힌다.
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, 1, 0, size);
if (a.length > size)
a[size] = null;
return a;
}
이 코드는 깔끔하게 컴파일되고 비검사 경고를 숨기는 범위로 좁혔다.
@SuppressWarnings("unchecked") 어노테이션을 사용할 때면 그 경고를 무시해도 안전한 이유를 항상 주석으로 남겨야 한다.
728x90
반응형
'dev book > Effective Java' 카테고리의 다른 글
[Effective Java/아이템26] 로타입은 사용하지 마라. (0) | 2022.03.04 |
---|---|
[Effective Java/아이템25] 톱레벨 클래스는 한 파일에 하나만 담아라. (0) | 2022.03.03 |
[Effective Java/아이템24] 멤버 클래스는 되도록 static으로 만들라. (0) | 2022.03.02 |
[Effective Java/아이템23] 태그 달린 클래스보다는 클래스 계층 구조를 활용하라. (0) | 2021.12.10 |
[Effective Java/아이템22] 인터페이스는 타입을 정의하는 용도로만 사용하라. (0) | 2021.12.09 |
Written by ner.o
개발자 네로의 개발 일기,
자바를 좋아합니다 !
댓글 개