[Effective Java]ordinal 메소드 대신 인스턴스 필드를 사용하라

열거 타입과 애너테이션, 두 번째 아이템

Posted by SungBeom on April 12, 2020 · 3 mins read

ordinal 메소드

대부분의 열거 타입 상수는 자연스럽게 하나의 정숫값에 대응된다. 그리고 모든 열거 타입은 해당 상수가 그 열거 타입에서 몇 번째 위치인지를 반환하는 ordinal이라는 메소드를 제공한다. ordinal 메소드를 이용한 예를 보자.

1
2
3
4
5
6
7
// ordinal을 잘못 사용한 예 - 따라 하지 말 것!
public enum Ensemble {
    SOLO, DUET, TRIO, QUARTET, QUINTET,
    SEXTET, SEPTET, OCTET, NONET, DECTET;
 
    public int numberOfMusicians() { return ordinal() + 1; }
}
cs

동작은 하지만 유지보수하기가 끔찍한 코드다. 상수 선언 순서를 바꾸는 순간 numberOfMusicians가 오동작하며, 이미 사용 중인 정수와 값이 같은 상수는 추가할 방법이 없다. 예컨대 8중주(octet) 상수가 이미 있으니 똑같이 8명이 연주하는 복4중주(double quartet)는 추가할 수 없다.

또한, 값을 중간에 비워둘 수도 없다. 예컨대 12명이 연주하는 3중 4중주(triple quartet)를 추가한다고 하면, 중간에 11명짜리 상수도 채워야 하는데, 11명으로 구성된 연주를 일컫는 이름이 없다. 그래서 더미(dummy) 상수를 같이 추가해야만 하는데, 이는 코드가 깔끔하지 못할 뿐 아니라, 쓰이지 않는 값이 많아질수록 실용성이 떨어진다.

인스턴스 필드

해결책은 간단한데, 열거 타입 상수에 연결된 값은 ordinal 메소드로 얻지 말고, 인스턴스 필드에 저장하자.

1
2
3
4
5
6
7
8
9
10
// ordinal을 잘못 사용한 예 - 따라 하지 말 것!
public enum Ensemble {
    SOLO(1), DUET(2), TRIO(3), QUARTET(4), QUINTET(5),
    SEXTET(6), SEPTET(7), OCTET(8), DOUBLE_QUARTET(8),
    NONET(9), DECTET(10), TRIPLE_QUARTET(12);
 
    private final int numberOfMusicians;
    Ensemble(int size) { this.numberOfMusicians = size; }
    public int numberOfMusicians() { return numberOfMusicians; }
}
cs

ordinal 메소드는 EnumSet과 EnumMap 같이 열거 타입 기반의 범용 자료구조에 쓸 목적으로 설계되었다. 따라서 이런 용도가 아니라면 ordinal 메소드는 절대 사용하지 말자.


핵심 정리

열거 타입 상수에 연결된 값을 얻기 위해서 ordinal 메소드는 EnumSet과 EnumMap 같이 특수한 상황이 아니면 절대 사용하지 말자. 대신 인스턴스 메소드를 사용하자.