Рефакторинг кода Java: использование нескольких экземпляров оператора

Если вы можете изменить File и его подклассы, вы можете попробовать использовать шаблон посетителя. Поскольку вы не можете, я рекомендую вам консолидировать оператор switch type type в одном методе, так что вам нужно будет только изменить один коммутатор.

public Enum FileHandler {
   HTML(){
      public search(File file, String searchstr){ /* ... */}
      public prettyPrint(File file){ /* ... */}
   },
   XML(){ /* ... */};
   // ... end list of enum implementations

   public static FileHander get(File file){
      // Put your master switch statement here
      if (targetPage instanceof HtmlFile) {
         return HTML;
      }
      else if (targetPage instanceof XmlFile) {
         return XML;
      }
      // ...
   }
}

И ваш код для использования класса enum будет выглядеть примерно так, избегая опасного оператора switch:

public searchFile(SimpleFile targetFile, String searchStr) {
   FileHandler.get(targetFile).search(targetFile, searchStr);
}

Моим подходом было бы создать интерфейс FileSearcher с единственным методом, который выполняет бизнес-логику. Создайте одну реализацию этого интерфейса для каждого типа файла. Затем создайте реестр, который сопоставляет подклассы SimpleFile с их соответствующей реализацией FileSearcher. В вашем методе файла поиска, а не в выполнении проверок instanceof, найдите соответствующую реализацию FileSeacher во время выполнения и делегируйте ее.

С этой реализацией вы можете столкнуться с проблемами, когда есть подклассы HtmlFile и т.д., Потому что вам нужно будет сопоставить HtmlFileSearcher со всеми подклассами HtmlFile. В этом случае добавьте еще один метод в FileSearcher именем canHandle(Class<SimpleFile> runtimeClass) который позволяет реализациям FileSearcher сигнализировать, какой класс файлов они понимают. В реестре, а не просматривая его на карте, пройдите через все зарегистрированные FileSearchers.

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

пример шаблона-интерпретатора-шаблона, комментарий, если вам нужен пример шаблона цепочки ответственности-дизайна

Клиент:

public Integer wordCountOnPage (Page samplePage) 
{
    ArrayList list = new ArrayList<Page>();
    list.add(new XmlPage());

    list.add(new HtmlPage());

    list.add(new JsonPage());

    for (int index = 0 ; index < list.size(); index ++)

    {
        Page eachPage = (Page) list.get(index); 
        if (eachPage.intercept(samplePage)){
            Integer i = eachPage.wordCountOnPage(samplePage);
        }

    }
    return 1;
}


public class XmlPage implements Page {

@Override
public Boolean intercept(Page page) {
    // TODO Auto-generated method stub
    return page instanceof XmlPage;
}

@Override
public Integer wordCountOnPage(Page page) {
    // TODO Auto-generated method stub
    return null;
}

}

public interface Page {

Boolean intercept(Page page);
Integer wordCountOnPage(Page page);
}