Utwórz metodę fabryki wyliczeniowej z unikalnej wartości instancji

Question

Utworzyłem Enum, aby zdefiniować określone akcje. Programowanie z zewnętrznego interfejsu API Aby wyrazić tę akcję, muszę użyć Integer. Właśnie dlatego dodałem pole instancji całkowitej do mojego Enum. Powinno to być zgodne z Efektywną Javą Joshua Blocha, zamiast polegać na ordinal() lub kolejności stałych Enum przy użyciu values()[index].

public enum Action { 

    START(0), 

    QUIT(1); 

    public final int code; 

    Protocol(int code) { 
     this.code = code; 
    } 
} 

uzyskać wartość całkowitą what z API i teraz chcę utworzyć wartość Enum z niego, w jaki sposób można zaimplementować to w najbardziej rodzajowe mody?

Oczywiście dodanie takiej metody fabrycznej nie będzie działać. Nie można utworzyć instancji Enum.

Action valueOf(int what) { 
    return new Action(what); 
} 

Oczywiście zawsze mogę utworzyć instrukcję switch-case i dodać wszystkie możliwe kody i zwrócić odpowiednią stałą. Ale chcę uniknąć definiowania ich w dwóch miejscach w tym samym czasie.

Answer 1

Jeśli masz zamiar mieć wiele z nich można użyć HashMap<Integer, Action>:

private static final Map<Integer, Action> actions = new HashMap<>(values().size, 1); 

static { 
    for (Action action : values()) 
     actions.put(action.code, action); 
} 

// ... 

public static Action valueOf(int what) { 
    return actions.get(what); 
} 

Jest to przydatne, jeśli masz zamiar mieć dużą liczbę Action wartości ponieważ jest wyszukiwanie HashMap O (1).

Answer 2

Jeżeli jesteś pewien, że kody będą zawsze sekwencyjny i od 0 to najbardziej efektywny rozwiązaniem byłoby

public enum Action { 
    START(0), 

    QUIT(1); 

    public static final Action[] ACTIONS; 
    static { 
     ACTIONS = new Action[values().length]; 
     for(Action a : values()) { 
     ACTIONS[a.code] = a; 
     } 
    } 

    public final int code; 

    Protocol(int code) { 
     this.code = code; 
    } 
} 

Answer 3

bym osobiście keep it simple (YAGNI) i użyć wartości porządkowej ale :

• chciałbym zachować logikę w wyliczenia, aby upewnić się, że kod na zewnątrz nie wie o tym szczegółowo realizacji i nie polegać na nim
• i wou ld upewnij się, że mam test, który się nie powiedzie, jeśli coś się zepsuje (tj. gdy numery nie zaczynają się od 0 lub nie są przyrostowe)

kod ENUM:

public enum Action { 

    START(0), 
    QUIT(1); 
    private final int code; 

    Action(int code) { 
     this.code = code; 
    } 

    public int getCode() { 
     return code; 
    } 

    public static Action of(int code) { 
     try { 
      return Action.values()[code]; 
     } catch (IndexOutOfBoundsException e) { 
      throw new IllegalArgumentException("not a valid code: " + code); 
     } 
    } 
} 

testu

@Test 
public testActionEnumOrder() { 
    int i = 0; 
    for (Action a : Action.values()) { 
     assertEquals(a.getCode(), i++); 
    } 
} 

Jeżeli zmienisz QUIT(1) do QUIT(2) na przykład , test się nie powiedzie. Kiedy tak się stanie, możesz użyć HashMap lub pętli odnośnika.