czy moje wyliczenie jest poprawne?

Question

W całym naszym projekcie mamy tego rodzaju wyliczenia. Działają dobrze, ale nie jesteśmy do końca pewni.

Specjalnie z metodą getDocumentType (String).

Czy istnieje sposób na uniknięcie iteracji na wszystkich polach 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; 
    } 
} 

Edytuj: Sprawdź odpowiedź newacct. Ona też jest w porządku.

Answer 1

Będziesz musiał zrobić to gdzieś, ze względu na ograniczenia w pisaniu wyrażeń. W idealnym świecie wypełnisz statyczną mapę z poziomu konstruktora DocumentType, ale to niedozwolone.

Najlepszym mogę zaproponować jest wykonywanie iteracji raz w statycznego inicjatora i przechowywanie stałe teksty w tabeli odnośników:

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

Przynajmniej nie będzie robić iteracji za każdym razem, chociaż wątpię, że zauważysz znaczącą poprawę wydajności.

Answer 2

Z tego, co wiem (za to, co warto), jest to najlepszy sposób robienia tego, co chcesz.

Tak właśnie zrobiłbym to co najmniej.

Jeśli liczba enum rośnie znacząco (kilkaset tysięcy) - może warto dodać Map ing z Strings do enums zrobić rozglądać się trochę szybciej. Ale jeśli masz małą liczbę eunums, może to być przesada.

Answer 3

Wygląda dobrze dla mnie.

Pozostawiłbym iterację taką, jaka jest. Na pewno można dodać implementację "etykiety", "DocumentType"> do klasy wyliczeniowej i wykonać wyszukiwanie, ale nie zwiększy to znacząco wydajności.

Answer 4

Jeśli ciągi są znane w czasie kompilacji, a jeśli są one ważne identyfikatory, można po prostu użyć ich jako nazw bezpośrednio teksty stałe:

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

a następnie dostać go .valueOf(). Na przykład:

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