[이펙티브자바 3판] ITEM35. ordinal 메서드 대신 인스턴스 필드를 사용하라

이번장의 핵심은...

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

'대부분 프로그래머는 이 메서드를 쓸 일이 없다'

이 메서드는 EnumSet과 EnumMap 같이 열거 타입 기반의 범용 자료구조에 쓸 목적으로 설계되었다

따라서 이런 용도가 아니면 ordinal 메서드는 절대 사용하지 말자

---

Ordinal() 메서드

해당 상수가 그 열거 타입에서 몇 번째 위치인지를 반환하는 메서드

// ordinal 잘못 사용한 예 - 따라 하지 말것
public enum Ensemble {
	SOLO, DUET, TRIO, QUARTER ...
	
	public int numberOfMusicians() {
		return ordinal()+1;
	}
}

• 동작은 하지만 유지보수가 끔찍한 코드임

• 상수 선언 순서를 바꾸는 순간 오작동하며, 이미 사용 중인 정수와 값이 같은 상수는 추가할 방법이 없다

  ex) 8중주 상수가 있을 때, 똑같이 8명이 연주하는 복4중주는 추가할 수 없음

• 또한, 값을 중간에 비워둘 수 없음

 

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

 

이 글은 “이펙티브 자바 3판” 책 내용을 정리한 글입니다.

만약 저작권 관련 문제가 있다면 “shk3029@kakao.com”로 메일을 보내주시면, 바로 삭제하도록 하겠습니다.