は私の列挙は正しいですか？

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

enumを書く際の制限のため、どこかでその反復処理を行う必要があります。理想的な世界では、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数が大幅に大きくなった場合（数百 - 数千人）あなたが少し速くルックアップを行うためにenumsにStringsのMap INGを追加することもできます。しかし、あなたが持っているeunumsの少量のために、これは過度の可能性があります。

Answer 3

私にはうまく見えます。

私はそのまま繰り返します。もちろん、マップ< 'label'、 'DocumentType'>の実装を列挙型クラスに追加して検索することはできますが、パフォーマンスが大幅に向上することはありません。

Answer 4

文字列はコンパイル時に知られており、それらが有効な識別子であれば、あなただけの直接列挙型の名前としてそれらを使用することができる場合：

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