이펙티브 자바 아이템35

한주영·2024년 2월 1일
0

이펙티브자바

목록 보기
27/33

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

대부분의 열거 타입 상수는 자연스럽게 하나의 정수값에 대응
모든 열거타입은 해당 상수가 그 열거 타입에서 몇번째 위치인지를 반환하는 ordinal메서드 제공

ordinal을 잘못 사용한 예

public enum Ensemble{
   SOLO,DUET,TRIO,QUARTET, QUINTET,
   SEXTET, SEPTET,OCTET, NONTET, 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; }
}

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

profile
백엔드개발자가 되고싶은 코린이:)

0개의 댓글