在Java返回枚举是不好的做法吗？

Question

假设我有一个类来为游戏中的一个项目建模，如下所示：

public class Item {
    private final EnumItem type;
    
    public Item(EnumItem type) {
        this.type = type;
    }
        
    public Item(String name) {
        this.type = EnumItem.fromName(name);
    }
}

public enum EnumItem {
    MACHINE_GUN("machine_gun"),
    SWORD("sword"),
    BAT("bat"),
    DEFAULT("default");
    
    private final String name;
    
    public EnumItem(name) {
        this.name = name;
    }

    public String getName() { return name; }

    public static EnumItem fromName(String name) {
        for(EnumItem i: EnumItem.values()) {
            if(i.name.equals(name)) {
                return i;
            } else {
                return EnumItem.DEFAULT;
            }
        }
    }
}

（假设正确重写item的.equals()和.hashcode()以比较内部枚举）

现在我想要一种方法来用item中的getter来区分这些项：我应该返回enum还是string名称？一般情况下返回枚举是好的做法吗？或者是否有更好的方法来区分这些items？因为返回枚举类似于向我公开rep，而且我不希望我的同事直接使用EnumItem来比较Item的

我想到的办法如下：

1. 字符串getName()执行类似item1.getName().equals（“machine_gun”）；
2. EnumItem getEnum()要执行item1.getEnum().equals(enumitem.machine_gun)；
3. item1.equals（新项目（“machine_gun”））；
4. 静态名称（字符串名称）{new Item（name）}要执行item1.equals(Item.name（“machine_gun”））；

我不知道该怎么做，我希望有经验的程序员能给我一些启示。

编辑：我知道它们看起来像是从上下文中得到的，但是在我的用例中，这些项没有特殊的功能来证明从基项类扩展是合理的

Answer 1

这种做法好吗？当然，您使用的是聚合，因为Item不依赖于EnumItem，这很好。话虽如此，还能做得更好吗？当然可以。我提供的替代方案是唯一的解决方案吗？没有。

备选方案

如果希望它是可扩展的，请考虑使用接口来表示项。然后允许接口扩展此接口以提供一些标准类型。

interface Item {

    String name();

}

enum EnumItem implement Item {

    private final String name;

    EnumItem(String name) {
        this.name = name;
    }

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

}

class AbstractItem implements Item {
    
    // constructor, override name()
    
}

Item item = EnumItem.FOO_BAR;

Item item2 = new AbstractItem("FooBar");

Item item3 = () -> "FooBar";