[effective java] 아이템 35. ordinal 메서드 대신 인스턴스 필드를 사용하라.

 

 

 

 

 

 

이 아이템은 정말 제목이 전부이다.. 정말 ordinal은 쳐다보지 말고 인스턴스 필드를 사용하면 된다.

 

 

 

 

ordinal을 사용할 경우
: 유지보수 극악

 

 

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

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

 

상수의 선언 순서를 변경하면 오동작..

 

이미 사용 중인 정수와 값이 같은 상수 추가 불가능..

 

값이 비어있는 중간 상수 추가 불가능..

 

결론적으로 유지보수에 적합하지 않다.

 

 

 

 

 

인스턴스 필드에 저장하자

 

 

 

 

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; }
}

 

 

 

 

 

 

 

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