올바르지 않은 방법

→ ordinal()을 배열 인덱스로 사용

public class Plant{

    enum LifeCycle { ANNUAL, PERNNIAL, BIENNIAL}

    final String name;
    final LifeCycle lifeCycle;

    public Plant(String name, LifeCycle lifeCycle) {
        this.name = name;
        this.lifeCycle = lifeCycle;
    }

    @Override
    public String toString() {
        return name;
    }
}

///

public static void usingOrdinalArray(List<Plant> garden) {
    Set<Plant>[] plantsByLifeCycle = (Set<Plant>[]) new Set[LifeCycle.values().length];
    for (int i = 0 ; i < plantsByLifeCycle.length ; i++) {
        plantsByLifeCycle[i] = new HashSet<>();
    }

    for (Plant plant : garden) {
        plantsByLifeCycle[plant.lifeCycle.ordinal()].add(plant);
    }

    for (int i = 0 ; i < plantsByLifeCycle.length ; i++) {
        System.out.printf("%s : %s%n", LifeCycle.values()[i], plantsByLifeCycle[i]);
    }
}

• 배열과 제네릭의 호환성 문제 발생
• ordinal() 메소드가 반환하는 값은 열거형의 선언 순서에 대한 정보만 제공하며, 이것이 실제 열거형 상수를 나타내진 않음
• 열거형의 순서를 바꾸거나 새로운 상수를 추가하는 등의 변경이 있을 때, ordinal()이 반환하는 값도 변경될 수 있다. 이러한 변경은 컴파일 타임이 아닌 런타임에 에러를 발생시키므로 디버깅이 어려움
• ordinal() 메소드가 반환하는 정수 값을 배열의 인덱스로 사용하는 것은 정확한 정수값을 사용하는지 개발자가 직접 보증해야 함

ordianl 대신 EnumMap을 사용하자

public static void usingEnumMap(List<Plant> garden) {
    Map<LifeCycle, Set<Plant>> plantsByLifeCycle = new EnumMap<>(LifeCycle.class);

    for (LifeCycle lifeCycle : LifeCycle.values()) {
        plantsByLifeCycle.put(lifeCycle,new HashSet<>());
    }

    for (Plant plant : garden) {
        plantsByLifeCycle.get(plant.lifeCycle).add(plant);
    }
    System.out.println(plantsByLifeCycle);
}

///

Map<LifeCycle, Set<Plant>> plantsByLifeCycle = new EnumMap<>(LifeCycle.class);

• 비검사 형변환이 필요 없음
• EnumMap은 내부적으로 배열을 사용하므로 배열의 성능을 유지하면서도, 열거 타입의 타입 안정성을 활용할 수 있음
• ordinal 메소드를 사용하지 않으므로 열거형 상수의 순서가 변경되었을 때 발생할 수 있는 문제를 피할 수 있음
• 열거형 상수 자체를 키로 사용하므로 명확하게 표현 가능
• EnumMap은 toString 메소드를 재정의하여 결과를 쉽게 출력할 수 있음