Блог

Ошибка:

 ...
Caused by: java.lang.ExceptionInInitializerError
...
Caused by: java.lang.ClassCastException: 
class com.evopulse.ds2150.TechTrees$BuildingTechTree
not an enum
at java.util.EnumSet.noneOf(Unknown Source)
at java.util.EnumSet.of(Unknown Source)
at com.evopulse.ds2150.TechTrees$BuildingTechTree.<clinit>(TechTrees.java:38)
  

Вот фрагмент моего перечисления

 public enum BuildingTechTree {
//Name                      SoftName                    Requirements    
NONE                        ("NULL",                    null),
  

—> В следующей строке происходит сбой

 BARRACKS                    ("Barracks",                EnumSet.of(NONE),
WALLS_SANDBAGS              ("Sandbag wall",            EnumSet.of(NONE),

POWERPLANT                  ("Power plant",             EnumSet.of(BARRACKS)),
GUARDTOWER                  ("Guard Tower",             EnumSet.of(BARRACKS));
  

Замена EnumSet.of(NONE) и EnumSet.of(BARRACKS) на null позволяет инициализации работать, но нарушает мой код из-за отсутствия структуры данных… очевидно, но я сделал это, чтобы протестировать остальную часть моего кода, и это каким-то образом не было причиной.

Удаление EnumSet.of(NONE) и замена просто NONE, и то же самое для BARRACKS, и изменение всех связанных переменных, конструктора и методов, которые тоже не сработали… (и даже не смог использовать contains.все, поскольку это не было «применимо к моей измененной переменной» … )

Я расширил этот пример, используя вторую реализацию: https://gamedev.stackexchange.com/a/25652/48573

Я также попытался повторить свои шаги, скопировав пример дословно. добавлено

 private static Set<BuildingTechTree> techsKnown;

techsKnown = (BuildingTechTree.BIODOME);
test = TechTrees.researchTech(techsKnown);
  

в другой класс, который будет вызван для инициализации тестирования. и пришлось изменить

 public boolean researchTech(BuildingTechTree tech) {
  

для статического

Это привело к той же ошибке «не в перечислении». У меня нет представителя, чтобы прокомментировать его ответ, чтобы указать на ошибку инициализации…

Добавлена информация для обоих текущих ответов, поскольку оба решения вызывают одну и ту же новую ошибку:

 public class TechTrees {
private static Set<BuildingTechTree> techsKnown;

public TechTrees() {
    techsKnown = EnumSet.of(BuildingTechTree.NONE);       //Using this
    techsKnown = EnumSet.noneOf(BuildingTechTree.class);  //Or this
}

public static boolean researchTech(BuildingTechTree tech) {
    if (techsKnown.containsAll(tech.requirements)) {      //Causes null pointer
        return true;                                      //exception @ techsKnown  
    }
    return false;
}
  

Ваша структура объявления настолько умна, что жаль, что она не работает. Но EnumSet очевидно, сначала необходимо полностью инициализировать перечисление. Он пытается извлечь массив констант из перечисления, чтобы, среди прочего, узнать, сколько места требуется для его внутреннего набора битов.

Вот одно из решений. Он использует вспомогательный метод, который сначала создает обычный набор ( HashSet ), а затем в блоке статической инициализации повторяет константы перечисления и заменяет все наборы на EnumSet s.

 public enum BuildingTechTree {
    // Named constants
    //Name                      SoftName                        Requirements
    NONE                        ("NULL",                        null),
    BARRACKS                    ("Barracks",                    setOf(NONE)),
    WALLS_SANDBAGS              ("Sandbag wall",                setOf(NONE)),
    POWERPLANT                  ("Power plant",                 setOf(BARRACKS)),
    GUARDTOWER                  ("Guard Tower",                 setOf(BARRACKS));

    private final String softName;
    private Set<BuildingTechTree> requirements;

    private BuildingTechTree(String softName, Set<BuildingTechTree> requirements) {
        this.softName = softName;
        this.requirements = requirements;
    }

    private static Set<BuildingTechTree> setOf(BuildingTechTree... values) {
        return new HashSet<>(Arrays.asList(values));
    }

    static {
        for (BuildingTechTree v : values()) {
            if (v.requirements == null) {
                v.requirements = EnumSet.noneOf(BuildingTechTree.class);
            } else {
                v.requirements = EnumSet.copyOf(v.requirements);
            }
        }
    }
}
  

У вас проблема с курицей и яйцом. Вы могли бы реорганизовать свое перечисление на что-то вроде этого:

 public enum BuildingTechTree {

    NONE("NULL"),
    BARRACKS("Barracks"),
    WALLS_SANDBAGS("Sandbag wall"),
    POWERPLANT("Power plant"),
    GUARDTOWER("Guard Tower");

    static {
        NONE.trees = EnumSet.noneOf(BuildingTechTree.class);
        BARRACKS.trees = EnumSet.of(NONE);
        WALLS_SANDBAGS.trees = EnumSet.of(NONE);
        POWERPLANT.trees = EnumSet.of(BARRACKS);
        GUARDTOWER.trees = EnumSet.of(BARRACKS);
    }

    private String name;
    private Set<BuildingTechTree> trees;

    private BuildingTechTree(String name) {
        this.name = name;
    }

    public String getName() {
        return name;
    }

    public Set<BuildingTechTree> getTrees() {
        return Collections.unmodifiableSet(trees);
    }
}
  

Редактировать:

что касается вашей второй проблемы: вы обращаетесь к статической переменной из статического метода. Но эта переменная инициализируется при вызове конструктора класса (что является огромной проблемой проектирования). Не используйте статические поля, не являющиеся окончательными. И не инициализируйте статические поля из методов экземпляра или конструкторов. Это не имеет смысла. Вы не задаете цвет, который должен быть у всех автомобилей при создании автомобиля. Инициализируйте свои статические поля статически:

 public class TechTrees {
    private static final Set<BuildingTechTree> TECHS_KNOWN =
        EnumSet.of(BuildingTechTree.NONE);

    public static boolean researchTech(BuildingTechTree tech) {
        return TECHS_KNOWN.containsAll(tech.requirements));
    }
}