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

 

위 코드는 동작하지만 유지보수하기 어려운 코드다.

 

1. 상수 선언 순서를 바꾸는 순간 numberOfMusicians가 오동작하며, 이미 사용 중인 정수와 값이 같은 상수는 추가할 방법이 없다.
   
   
2. 값을 중간에 비워둘 수도 없다.
   → 12번째 상수를 추가할 경우 11번째의 더미 상수를 같이 추가해야 한다.
   이는 코드가 깔끔하지 못할 뿐 아니라 실용성이 떨어진다.

 

열거 타입 상수에 연결된 값은 ordinal 메서드로 얻지 말고, 인스턴스 필드에 저장하자.

 

public enum Ensemble {
	SOLO(l), DUET(2), TRI0(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 메서드는 절대 사용하지 말자.