Item 35: ordinal 메서드 대신 인스턴스 필드를 사용하라

열거 타입은 해당 상수가 열거 타입에서 몇 번째 위치인지를 반환하는 ordinal이라는 메서드를 제공한다.

열거 타입 상수와 연결된 정숫값이 필요하면 ordinal 메서드를 이용하고 싶은 유혹에 빠진다.

public enum Ensemble {
    SOLO, DUET, TRIO, QUARTET, QUINTET, SEXTET, SEPTET, OCTET, NONET, DECTET;

    public int numberOfMusicians() {
        return ordinal() + 1;
    }
}

문제점

• 상수 선언 순서를 바꾸는 순간 numberOfMusicians가 오동작한다.
• 이미 사용 중인 정수와 값이 같은 상수는 추가할 방법이 없다.
• 값을 중간에 비워둘 수도 없다.

 

해결책은 간단하다.

열거 타입 상수에 연결된 값은 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;
    }
}

 

Enum의 API 문서를 보면 ordinal에 대해 이 렇게 쓰여 있다.

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

https://docs.oracle.com/javase/8/docs/api/java/lang/Enum.html#ordinal--

Enum (Java Platform SE 8 )