Pourquoi ne peut-enum constructeur de l'accès aux champs statiques?

Pourquoi ne peut-enum constructeur de l'accès aux champs statiques et les méthodes? C'est parfaitement valide avec une classe, mais il n'est pas permis avec un enum.

Ce que je suis en train de faire est de stocker mes énumérer les instances dans une Carte statique. Prenons cet exemple de code qui permet la recherche par abbreivation:

public enum Day {
    Sunday("Sun"), Monday("Mon"), Tuesday("Tue"), Wednesday("Wed"), Thursday("Thu"), Friday("Fri"), Saturday("Sat");

    private final String abbreviation;

    private static final Map<String, Day> ABBREV_MAP = new HashMap<String, Day>();

    private Day(String abbreviation) {
        this.abbreviation = abbreviation;
        ABBREV_MAP.put(abbreviation, this);  //Not valid
    }

    public String getAbbreviation() {
        return abbreviation;
    }

    public static Day getByAbbreviation(String abbreviation) {
        return ABBREV_MAP.get(abbreviation);
    }
}

Cela ne fonctionne pas comme enum ne permet pas de faire des références statiques dans son constructeur. Il fonctionne cependant il suffit de trouver si elles sont appliquées comme une classe:

public static final Day SUNDAY = new Day("Sunday", "Sun");
private Day(String name, String abbreviation) {
    this.name = name;
    this.abbreviation = abbreviation;
    ABBREV_MAP.put(abbreviation, this);  //Valid
}
InformationsquelleAutor Steve Kuo | 2009-01-14
  1. 105

    Le constructeur est appelé avant de les champs statiques ont tous été initialisé, parce que les champs statiques (y compris ceux qui représentent les valeurs enum) sont initialisés dans l'ordre textuel, et les valeurs enum viennent toujours avant les autres champs. Notez que dans votre exemple de classe vous n'avez pas indiqué où ABBREV_MAP est initialisée si c'est après DIMANCHE, vous aurez une exception lorsque la classe est initialisée.

    Oui, c'est un peu de douleur, et il pourrait probablement avoir été conçu mieux.

    Cependant, la réponse habituelle dans mon expérience, c'est d'avoir un static {} bloc à la fin de tous les initialiseurs statiques, et de faire tous initialisation statique, en utilisant EnumSet.allOf d'obtenir toutes les valeurs.

    • Si vous ajoutez une classe imbriquée, puis la statique de qui sera initialisé à un moment approprié.
    • Ooh, une belle. Je n'avais pas pensé à cela.
    • Je ne comprends pas vraiment le rapport au sujet de l'initialiseur statique bloc. J'ai essayé de créer un initialiseur statique bloc dans un enum classe, et il a été appelé après le constructeur de la enum membre, apparemment faite selon l'ordre textuel comme vous l'avez dit.
    • Si vous essayez d'utiliser une variable statique dans l'enum constructeur (comme dans la question), vous constaterez que la valeur non initialisée. Au lieu de cela, utilisez un initialiseur statique à tout initialiser après enum valeurs ont tous été construits et le reste de la statique des variables sont initialisées.
    • Peu bizarre mais si vous appelez une méthode statique dans une enum constructeur qui retourne une valeur statique, elle compile très bien - mais la valeur qu'elle renvoie sera celui par défaut pour ce type (c'est à dire 0, 0.0, '\u0000' ou null), même si vous l'avez explicitement définie (sauf si elle est déclarée comme final). Devinez qui va être difficile à attraper!
    • quick spin-off de question @JonSkeet: Quelque raison que vous utilisez EnumSet.allOf au lieu de Enum.values()? Je demande parce que values est une sorte de fantôme de la méthode (ne peut pas voir la source dans Enum.class) et je ne sais pas où le son créé
    • C'est la seule chose qui fonctionne si vous souhaitez utiliser static final champs comme arguments pour une enum constructeur. (Bien sûr, il n'a pas à être une classe imbriquée; vous avez juste à enlever la statique de la enum lui-même dans une autre classe quelque part.)
    • Il y a un question à ce sujet. Il semble que Enum.values() est plus rapide si vous prévoyez sur l'itération sur eux avec un renforcement de la boucle for (depuis, elle retourne un tableau), mais surtout pour son sujet de style et de cas d'utilisation. Il est probablement préférable d'utiliser EnumSet.allOf() si vous voulez écrire du code qui existe dans la documentation de Java au lieu de juste dans les specs, mais beaucoup de gens semblent être familier avec Enum.values() de toute façon.
    • jfyi le faire en groovy, vous pouvez également voir cette question

    InformationsquelleAutor Jon Skeet
  2. 29

    Citation de JLS, section "Enum Corps Déclarations":

    Sans cette règle, apparemment raisonnable code échouerait au moment de l'exécution
    en raison de l'initialisation de la circularité inhérente dans les types d'énumérations. (Un
    la circularité existe dans une classe avec un "auto-tapé" champ statique.)
    Voici un exemple de code qui serait un échec:

    enum Color {
    RED, GREEN, BLUE;
    static final Map<String,Color> colorMap = new HashMap<String,Color>();

    Color() {
       colorMap.put(toString(), this);
    }
}

    Initialisation statique de ce type d'énumération serait jeter un NullPointerException parce que la variable statique palette de couleurs est
    non initialisée lorsque les constructeurs pour les enum constantes exécuter. L'
    la restriction ci-dessus assure que ce code ne compile pas.

    Noter que l'exemple peut facilement être reconstruit pour fonctionner correctement:

    enum Color {
    RED, GREEN, BLUE;
    static final Map<String,Color> colorMap = new HashMap<String,Color>();

    static {
        for (Color c : Color.values())
            colorMap.put(c.toString(), c);
    }
}

    La version remaniée est clairement, comme l'initialisation statique se produit haut vers le bas.

    InformationsquelleAutor Phani
  3. 7

    c'est peut-être ce que vous voulez

    public enum Day {
    Sunday("Sun"), 
    Monday("Mon"), 
    Tuesday("Tue"), 
    Wednesday("Wed"), 
    Thursday("Thu"), 
    Friday("Fri"), 
    Saturday("Sat");

    private static final Map<String, Day> ELEMENTS;

    static {
        Map<String, Day> elements = new HashMap<String, Day>();
        for (Day value : values()) {
            elements.put(value.element(), value);
        }
        ELEMENTS = Collections.unmodifiableMap(elements);
    }

    private final String abbr;

    Day(String abbr) {
        this.abbr = abbr;
    }

    public String element() {
        return this.abbr;
    }

    public static Day elementOf(String abbr) {
        return ELEMENTS.get(abbr);
    }
}
    • À l'aide de Collections.unmodifiableMap() est une très bonne pratique ici. +1
    InformationsquelleAutor user4767902
  4. 6

    Le problème a été résolu par une classe imbriquée. Avantages: il est plus court, et aussi de mieux en consommation CPU. Inconvénients: une classe plus dans la mémoire JVM.

    enum Day {

    private static final class Helper {
        static Map<String,Day> ABBR_TO_ENUM = new HashMap<>();
    }

    Day(String abbr) {
        this.abbr = abbr;
        Helper.ABBR_TO_ENUM.put(abbr, this);

    }

    public static Day getByAbbreviation(String abbr) {
        return Helper.ABBR_TO_ENUM.get(abbr);
    }
    • J'ai corrigé le code
    InformationsquelleAutor Pavel Vlasov
  5. 1

    Quand une classe est chargée dans la JVM statique champs sont initialisés dans l'ordre dans lequel ils apparaissent dans le code. Par exemple

    public class Test4 {
        private static final Test4 test4 = new Test4();
        private static int j = 6;
        Test4() {
            System.out.println(j);
        }
        private static void test() {
        }
        public static void main(String[] args) {
            Test4.test();
        }
    }

    La sortie sera de 0. Notez que test4 initialisation a lieu en statique processus d'initialisation et pendant ce temps, j n'est pas encore initialisée comme il apparaît par la suite. Maintenant, si nous passons de l'ordre des initialiseurs statiques tels que j arrive avant test4. La sortie sera de 6.Mais dans le cas des Énumérations, nous ne pouvons pas changer l'ordre des champs statiques. La première chose enum doit être l'une des constantes qui sont en réalité statique finale des instances de type enum.Ainsi, pour les enums son toujours garantie que les champs statiques ne sera pas initialisé avant enum constantes.Puisque nous ne pouvons pas donner n'importe quelle des valeurs pour les champs statiques pour une utilisation dans les enum constructeur, il serait inutile d'y accéder dans l'enum constructeur.

    InformationsquelleAutor Hitesh