Creare un metodo factory enum con un valore di istanza univoco

Question

Ho creato un Enum per definire determinate azioni. Programmazione contro un'API esterna Sono costretto a utilizzare un Integer per esprimere questa azione. Ecco perché ho aggiunto un campo istanza intero al mio Enum. Questo dovrebbe essere d'accordo con Effective Java di Joshua Bloch, invece di fare affidamento su ordinal() o sull'ordine delle costanti Enum usando values()[index].

public enum Action { 

    START(0), 

    QUIT(1); 

    public final int code; 

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

ottengo un valore intero what dalle API e ora voglio per creare un valore Enum fuori di esso, come posso implementare questo nel modo più generica?

Ovviamente, l'aggiunta di un tale metodo di fabbrica, non funzionerà. Non puoi istanziare un Enum.

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

Naturalmente, posso sempre fare un'istruzione switch-case e aggiungere tutti i possibili codici e restituire la costante appropriata. Ma voglio evitare di definirli in due posti allo stesso tempo.

Answer 1

Se avete intenzione di avere un sacco di loro, è possibile utilizzare un 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); 
} 

Ciò è utile se si sta andando ad avere un gran numero di Action valori dal momento che la ricerca è HashMap O (1).

Answer 2

Se si è sicuri che i vostri codici saranno sempre sequenziale e partendo da 0, allora l'opzione più efficiente sarebbe

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

Io personalmente mantenere le cose semplici (YAGNI) e utilizzare il valore ordinale ma :

• vorrei mantenere la logica all'interno del enum per assicurarsi che il codice di fuori non sa su questo dettaglio di implementazione e non si basa su di esso
• I wou Assicurati di avere un test che fallisce se qualcosa si rompe (ad es. se i numeri non partono da 0 o non sono incrementali)

codice 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); 
     } 
    } 
} 

prova

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

Se si cambia QUIT(1)-QUIT(2) per esempio , il test fallirà. Quando ciò accade, puoi usare una HashMap o un ciclo di ricerca.