Есть ли что-то не так с локальным классом определяется с п тела конструктора

Question

У меня есть следующие перечисления:

enum FilterFactory { 

    INSTANCE; 

    private final Map<FilterType, Creator> creators; 

    private FilterFactory() { 
     creators = new HashMap<>(); 

     class SimplCreator implements FilterCreator{ 
      @Override 
      public FilterDTO createDto() { 
       return new FilterDTO(); 
      } 
     } //Local class within the constructor 

     creators.put(FilterType.DATE, new FilterCreator(){ 
      @Override 
      public FilterDTO createDto() { 
       return new DynamicDTO(); 
      } 
     }); 

     creators.put(FilterType.DROP_DOWN_LIST, new SimplCreator()); 
     creators.put(FilterType.FIELD, new SimplCreator()); 
    } 

    private static interface Creator{ 
     public FilterDTO createDto(); 
    } 
    //Other staff 
} 

Дело в том, я никогда не использовал местные классы в застройщиках органов. Может ли это вызвать некоторые ошибки, неужели это плохо? Кроме того, конструктор конструктора enu.

Answer 1

Ваш подход прекрасно, но в Java 8 вы можете сделать это немного лучше, используя ссылки метод или лямбды (и замене Творца с более стандартным Supplier<FilterDTO>):

import java.util.function.Supplier; 

enum FilterFactory { 

    INSTANCE; 

    private final Map<FilterType, Supplier<FilterDTO>> creators; 

    private FilterFactory() { 
     creators = new EnumMap<>(FilterType.class); // a bit more efficient :) 
     creators.put(FilterType.DATE, DynamicDTO::new); 
     creators.put(FilterType.DROP_DOWN_LIST, SimpleDTO::new); 
     creators.put(FilterType.FIELD, SimpleDTO::new); 
    } 

    // other stuff ... 
} 

Здесь я использовал застройщика, method reference для конкретизации Supplier<FilterDTO>. Я мог бы так же хорошо использовать лямбда-выражения, который не говорит «дал ничего, дайте мне FilterDTO»:

creators.put(FilterType.DATE,() -> new DynamicDTO()); 
... 

два варианта (метод ссылки против lamdbas) в основном эквивалентны, и вы должны использовать в зависимости от того понятнее вы (Java's language architect himself). Я лично нахожу метод эталонным визуально более ясным, хотя он немного привыкает.

Answer 2

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

enum FilterFactory { 

    INSTANCE; 

    private final Map<FilterType, Creator> creators = new HashMap<>(); 

    private static final SimplCreator DEFAULT_CREATOR = new Creator() { 
     @Override 
     public FilterDTO createDto() { 
      return new FilterDTO(); 
     } 
    } 

    private static final FilterCreator DYNAMIC_CREATOR = new Creator(){ 
     @Override 
     public FilterDTO createDto() { 
      return new DynamicDTO(); 
     } 
    } 

    private FilterFactory() { 
     creators.put(FilterType.DATE, DYNAMIC_CREATOR); 
     creators.put(FilterType.DROP_DOWN_LIST, DEFAULT_CREATOR); 
     creators.put(FilterType.FIELD, DEFAULT_CREATOR); 
    } 

    private static interface Creator{ 
     FilterDTO createDto(); 
    } 
    //Other stuff 
}