Блог

Итак, у меня есть этот метод, который выполняется неоднократно

 public static boolean isReady(String dirPath, int numPdfInPrintJob){
    File dir = new File(dirPath);
    String[] fileList = dir.list(new FilenameFilter(){
        public boolean accept(File file, String filename) {
            return (filename.toLowerCase().endsWith(".pdf"));
        }
    });
    if(fileList.length >= numPdfInPrintJob) return true;
    else return false;  
}
  

Этот метод использует anonymous class , который будет создавать новый экземпляр FilenameFilter при каждом вызове, и я часто использую этот метод. Итак, я хочу превратить это anonymous class в singleton . Итак, моя первоначальная мысль — создать новый singleton класс, который будет выглядеть следующим образом

 public class PdfFileNameFilter implements FilenameFilter{
    private PdfFileNameFilter(){} //non-instantible

    //guarantee to only have one instance at all time
    public static final PdfFileNameFilter INSTANCE = new PdfFileNameFilter();

    public boolean accept(File dir, String name) {
        return (name.toLowerCase().endsWith(".pdf"));
    }
}
  

Могу ли я еще немного реорганизовать это. Мне также нужно сделать ZipFileNameFilter , и, возможно, много разных фильтров расширений файлов. Не хочу создавать класс для каждого фильтра. Мне нужно еще немного реорганизовать этот дизайн. Может быть, interface где-то здесь это встанет на место.

Если все, что вы хотели сделать, это уменьшить использование памяти, вы могли бы сделать

 private static final FilenameFilter PDF_FILES = new FilenameFilter(){
    public boolean accept(File file, String filename) {
        return (filename.toLowerCase().endsWith(".pdf"));
    }
}
  

Если вы хотите создать singleton, самый простой способ

 public enum PdfFileNameFilter implements FilenameFilter {
    INSTANCE;

    public boolean accept(File dir, String name) {
        return (name.toLowerCase().endsWith(".pdf"));
    }
}
  

Мне кажется проще просто использовать ваш существующий анонимный класс и создать один экземпляр, который используют все ваши вызовы методов.

 private static final FilenameFilter PDF_FILTER = new FilenameFilter() {
    public boolean accept(File file, String filename) {
        return (filename.toLowerCase().endsWith(".pdf"));
    }
}

public static boolean isReady(String dirPath, int numPdfInPrintJob){
    File dir = new File(dirPath);
    String[] fileList = dir.list(pdfFilter);
    if(fileList.length >= numPdfInPrintJob) return true;
    else return false;  
}
  

Это тот случай, когда создание подклассов и синглтона кажется немного излишним: вы просто хотите использовать только один экземпляр прямо здесь, тогда как singleton используется, когда есть только один экземпляр, который вы когда-либо захотите использовать.

Для этого вы могли бы использовать enum :

 public enum ExtensionFilter implements FilenameFilter {
    PDF(".pdf"),
    ZIP(".zip");

    private final String extension;

    private ExtensionFilter(String extension) {
        this.extension = extension;
    }

    @Override
    public boolean accept(File dir, String name) {
        return (name.toLowerCase().endsWith(extension));
    }
}
  

Теперь вы сможете использовать его как:

 dir.list(ExtensionFilter.PDF)
  

Вы также можете выполнить итерацию по ним, если вам нужно:

 for ( FilenameFilter fileNameFilter : ExtensionFilter.values() ) {
    ....
}
  

Вы также могли бы использовать vararg в качестве аргумента конструктора, чтобы разрешить несколько расширений для одного и того же фильтра, и использовать имя константы по умолчанию, чтобы упростить его использование:

 public enum ExtensionFilter implements FilenameFilter {
    PDF,
    ZIP(".zip", ".jar", ".war", ".ear");

    private final String[] extensions;

    private ExtensionFilter(String... extensions) {
        if (extensions.length == 0) {
            extensions = new String[] {"."   name().toLowerCase()};
        }
        this.extensions = extensions;
    }

    @Override
    public boolean accept(File dir, String name) {
        for (String extension : extensions) {
            if (name.toLowerCase().endsWith(extension)) {
                return true;
            }
        }
        return false;
    }
}
  

вы можете отказаться от полного singleton и использовать личное поле для настройки расширения

 public class ExtensionFileNameFilter implements FilenameFilter{
    private String extension;
    private ExtensionFileNameFilter (String extension){this.extension=extension;} 

    public static final ExtensionFileNameFilter PDFINSTANCE = new ExtensionFileNameFilter (".pdf");
    public static final ExtensionFileNameFilter ZIPINSTANCE = new ExtensionFileNameFilter (".zip");
    //add instances as you need

    public boolean accept(File dir, String name) {
        return (name.toLowerCase().endsWith(extension));
    }
}
  

Могу ли я еще немного реорганизовать это.

Да, да, вы можете.

Предполагая, что это не тот ответ, который вы искали (вам следует обновить yoru question, чтобы задать более конкретный вопрос), я бы не стал его реорганизовывать, пока он вам не понадобится; YAGNI.

Как только у вас будет больше кода, например, больше фильтров имен файлов, вы увидите возможные рефакторинги. Вы увидите общий код, возможно, интерфейс, что-то в этом роде. Не пытайтесь преждевременно перенастроить.

TDD также является лучшим способом безопасного проведения рефакторинга. Если у вас есть тесты, показывающие, какой код вам действительно нужен, многие дополнительные материалы исчезнут, если таковые имеются. И с полным набором тестов вы можете рефакторинговать без колебаний, потому что вы знаете, работают ли ваши изменения или нет, основываясь на том, продолжают ли ваши тесты проходить.

Для интереса, эта альтернативная accept реализация будет выполняться намного быстрее в тестовом тестировании. Он не создает новые объекты с сохранением состояния и не несет других накладных расходов String.toLowerCase , которые не требуются в вашем случае.

   public boolean accept(File file, String filename) {
    int offset = s.length() - 4;
    if (offset >= 0) {
      if (s.charAt(offset) == '.') {
        offset  = 1;
        if (s.regionMatches(offset, "pdf", 0, 3)) {
          return true;
        } else if (s.regionMatches(offset, "PDF", 0, 3)) { 
          return true;
        }
      }
    }
    return false;
  }
  

Если бы это была точка доступа к выполнению, и вы искали оптимизацию, что-то подобное помогло бы.