2021. 8. 24. 11:53ㆍ책/이펙티브자바
비검사 경고
unchecked warning을 비검사 경고라고 하며 컴파일러 경고 정도라고 생각하면 된다.
대부분의 비검사 경고는 쉽게 제거할 수 있다.
// ex.1
Set<Lark> exaltation = new HashSet();
HashSet exaltation = new HashSet();
// ex.2
Set<Lark> exaltation = new HashSet<>();
ex.1의 경우 컴파일러가 무엇이 잘못됐는지 알려주고 알려준 대로 수정하면 경고가 사라진다.
ex.2를 보면컴파일러가 알려준 타입 매개변수를 명시하지않고, 자바 7부터 지원하는 다이아몬드 연산자(<>)만으로 해결할 수 있다.
여기서 <> 은 컴파일러가 올바른 실제 타입 매개변수(이 경우는 Lark) 를 추론해준다.
경고 제거하기
곧바로 해결되지 않는 경고가 나타나도 포기하지 말자!
할 수 있는 한 모든 비검사 경고를 제거하라. 모두 제거한다면 타입 안전성이 보장된다. 즉, 런타임에 ClassCastException이 발생할 일이 없고, 의도한 대로 동작하리라 확신할 수 있다.
@SuppressWarnings
경고를 제거할 수는 없지만 타입 안전하다고 확신할 수 있다면 @SuppressWarnings("unckecked") 를 달아 경고를 숨기자.
단, 타입 안전함을 검증하지 않은 채 경고를 숨기면 스스로에게 잘못된 보안 인식을 심어주는 꼴이다.
한편, 안전하다고 검증된 비검사 경고를 숨기지않고 그대로 두면, 진짜 문제를 알리는 새로운 경고가 나와도 눈치채지 못할 수 있다. 제거하지 않은 수많은 거짓 경고 속에 새로운 경고가 묻힐 수 있기 때문이다.
@Target({TYPE, FIELD, METHOD, PARAMETER, CONSTRUCTOR, LOCAL_VARIABLE, MODULE})
@Retention(RetentionPolicy.SOURCE)
public @interface SuppressWarnings {
/**
* The set of warnings that are to be suppressed by the compiler in the
* annotated element. Duplicate names are permitted. The second and
* successive occurrences of a name are ignored. The presence of
* unrecognized warning names is <i>not</i> an error: Compilers must
* ignore any warning names they do not recognize. They are, however,
* free to emit a warning if an annotation contains an unrecognized
* warning name.
*
* <p> The string {@code "unchecked"} is used to suppress
* unchecked warnings. Compiler vendors should document the
* additional warning names they support in conjunction with this
* annotation type. They are encouraged to cooperate to ensure
* that the same names work across multiple compilers.
* @return the set of warnings to be suppressed
*/
String[] value();
}
@SuppressWarnings 은 개별 지역변수 선언부터 클래스 전체까지 어떤 선언에도 달 수 있다.
하지만, @SuppressWarnings 애노테이션은 항상 가능한 한 좁은 범위에서 적용하자.
보통, 변수선언, 아주짧은 메서드, 생성자가 될 것이다.
자칫 심각한 경고를 놓칠 수 있으니 절대로 클래스 전체에 적용해서는 안된다.
@SuppressWarnings("unchecked") 를 사용할때면 그 경고를 무시해도 안전한 이유를 항상 주석으로 남겨야한다.
결론
- 비검사 경고는 중요하니 무시하지 말자.
- 모든 비검사 경고는 런타임에 ClassCastException을 일으킬 수 있으니 최선을 다해 제거하라.
- 경고를 없앨 방법을 못찾겠다면, 그 코드가 타입 안전함을 증명하고 가능한 한 범위를 좁혀 @SuppressWarnings("unchecked") 로 경고를 숨겨라. 그런 다음 숨기기로 한 근거를 주석으로 남겨라.
'책 > 이펙티브자바' 카테고리의 다른 글
아이템35. ordinal 메서드 대신 인스턴스 필드를 사용하라 (0) | 2021.08.28 |
---|---|
아이템34. int 상수 대신 열거 타입을 사용하라 (0) | 2021.08.28 |
아이템26. 로 타입은 사용하지 말라 (0) | 2021.08.24 |
아이템25. 톱레벨 클래스는 한 파일에 하나만 담으라 (0) | 2021.08.24 |
아이템23. 태그 달린 클래스보다는 클래스 계층구조를 활용하라 (0) | 2021.08.24 |