이펙티브 자바 Item 35 : ordinal 메서드 대신 인스턴스 필드를 사용하라

 

열거 타입 상수는 하나의 정수값에 대응되며, 모든 열거 타입은 해당 상수가 몇 번째 위치인지 반환하는 ordinal()메서드를 제공한다. 

 

• ordinal()의 잘못 사용한 예 
  • 해당 코드는 상수 선언 순서를 바꾸는 순간 오작동하며, 이미 사용 중인 정수와 값이 같은 상수는 추가할 방법이 없다. 
  • 동작은 하지만 유지보수의 어려움이 있다! 

public enum Ensemble {
    SOLO, DUET, TRIO, QUARTET, QUINTET,
    SEXTET, SEPTER, OCTET, DOUBLE_QUARTET,
    NONET, DECTET, TRIPLE_QUARTET;
    
    public int numberOfMusicians() { return ordinal() + 1; }

 

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

 

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

 

---

참고자료

https://javabom.tistory.com/50

[아이템 36] 비트 필드 대신 EnumSet을 사용하라

https://velog.io/@semi-cloud/Effective-Java-%EC%95%84%EC%9D%B4%ED%85%9C-35-ordinal-%EB%A9%94%EC%84%9C%EB%93%9C-%EB%8C%80%EC%8B%A0-%EC%9D%B8%EC%8A%A4%ED%84%B4%EC%8A%A4-%ED%95%84%EB%93%9C%EB%A5%BC-%EC%82%AC%EC%9A%A9%ED%95%98%EB%9D%BC

[Effective Java] 아이템 35 : ordinal 메서드 대신 인스턴스 필드를 사용하라