我的枚举正确吗？

Question

在我们的整个项目中，我们都有这种枚举。 它们工作得很好，但我们不确定它们。

特别是使用 getDocumentType(String) 方法。

有没有办法避免对所有 Enums 字段进行迭代？

public enum DocumentType {

    UNKNOWN("Unknown"),
    ANY("Any"),
    ASSET(Asset.class.getSimpleName()),
    MEDIA(Media.class.getSimpleName()),
    MEDIA35MM(Media.class.getSimpleName() + " 35mm");


    private String label;

    private DocumentType(String label) {
        this.label = label;
    }

    public String getLabel() {
        return label;
    }

    public static DocumentType getDocumentType(String label){
        for(DocumentType documentType : DocumentType.values()){
            if(documentType.getLabel().equals(label)){
                return documentType;
            }
        }
        return UNKNOWN;
    }
}

编辑 ： 检查 newacct 响应。 她也很好。

Answer 1

由于编写枚举的限制，您将不得不在某个地方进行迭代。 在理想的情况下，您可以从 DocumentType 的构造函数中填充静态 Map，但这是不允许的。

我能建议的最好方法是在静态初始化程序中执行一次迭代，并将枚举存储在查找表中：

public enum DocumentType {

    .... existing enum stuff here

    private static final Map<String, DocumentType> typesByLabel = new HashMap<String, DocumentType>();
    static {
        for(DocumentType documentType : DocumentType.values()){
            typesByLabel.put(documentType.label, documentType);
        }
    }

    public static DocumentType getDocumentType(String label){
        if (typesByLabel.containsKey(label)) {
            return typesByLabel.get(label);
        } else {
            return UNKNOWN;
        }
    }
}

至少您不会每次都进行迭代，尽管我怀疑您是否会看到任何有意义的性能改进。

Answer 2

据我所知（无论它的价值），这是做你想做的事的最好方法。

至少我会这样做。

如果您的 enum 数量显着增长（数百至数千），您可能需要将 Strings 的 Map 添加到 enum code> 使查找速度更快一些。 但对于您拥有的少量 eunum 来说，这可能有点过分了。

Answer 3

如果字符串在编译时已知，并且它们是有效的标识符，则可以直接将它们用作枚举的名称：

public enum DocumentType { Unknown, Any, Asset, Media, Media35mm }

然后通过 .valueOf() 获取它。 例如：

String label = "Asset";
DocumentType doctype;
try {
    doctype = DocumentType.valueOf(label);
} catch (IllegalArgumentException e) {
    doctype = DocumentType.Unknown;
}

Answer 4

对我来说看起来不错。

我会让迭代保持原样。 当然，您可以添加 Map<'label','DocumentType'> 实现枚举类并进行查找，但不会显着提高性能。