Как выбрать действие для выполнения на основе объектов instanceof?

Question

У меня есть некоторые реализации класса Base, и все объекты собраны в List<Base>.

Как я могу назвать конкретный action на основе instanceof этих объектов без использования подробных instanceof чеков? Как я могу выбрать servicemethod для выполнения на основе экземпляра этих объектов, но без необходимости заботиться о том, на каком объекте выполняется действие. Правильный сервис-метод следует выбирать как можно автоматически без проверки типа или экземпляра.

class Base; 
class Foo extends Base; 
class Bar extends Base; 

class Service { 
    List<Base> bases; 

    public void someMethod() { 
     for (Base base : bases) { 
      //perform some instanceof dependend action. 
      //these actions cannot be inside of any Base class as it makes use of other objects too. 
      if (base instanceof Foo) { 
       fooService.action((Foo) base); 
      } 
      if (base instanceof Bar) { 
       barService.action((Bar) base); 
      } 
     } 
    } 
} 


//custom service methods 
class FooService { 
    void action(Foo foo) { 
    } 
} 

class BarService { 
    void action(Bar bar) { 
    } 
} 

Answer 1

Полиморфизм - одна из возможностей. Если класс Base содержит абстрактный метод action(), вы можете вызвать его напрямую, без каких-либо операторов if.

Другой - это реестр. Для каждого подкласса Base вы можете зарегистрировать сопоставление для класса сервиса. В someMethod() найдите класс в реестре, получите соответствующий класс сервиса и вызовите его.

Кстати, многословие - это не единственная проблема с операторами instanceof. Код, который вы пишете с ними, может быть поврежден, как только будет создан новый подкласс базы. Удовлетворение Open/Closed principle может привести к менее хрупкому коду.

Answer 2

Одним из возможных решений является использование шаблона Visitor. Вам нужно будет объединить FooService и BarService в один класс с перегруженными методами action(). Затем вам нужно добавить метод accept() для каждого из классови Bar, которые затем могут вызвать соответствующий метод action(). Для получения дополнительной информации см. http://en.wikipedia.org/wiki/Visitor_pattern.

Возможно, другие, более подходящие design patterns, которые могут решить эту проблему. Я предлагаю вам изучить их.

Answer 3

Ближайший я когда-либо получал, чтобы решить эту проблему:

class Base{}; 
class Foo extends Base{}; 
class Boo extends Base{}; 

Дополнительный интерфейс BaseService поэтому каждый класс обслуживания используют одни и те же методы:

public interface BaseService<T extends Base> { 

void action(T base); 

} 

BooService:

public class BooService implements BaseService<Boo> { 

public void action(Boo boo){ 
    System.out.println("Action performed by BooService"); 
    } 
} 

FooService:

public class FooService implements BaseService<Foo> { 

public void action(Foo foo){ 
    System.out.println("Action performed by FooService"); 
    } 
} 

Дополнительный класс ServiceSupplier, который будет ссылаться на надлежащую базу обслуживания на переданный объект:

public class ServiceSupplier { 

private Map<Class<? extends Base>, BaseService> services; 

public ServiceSupplier(){ 
    initializeServiceMap(); 
} 

public BaseService getServiceOfType(Class<? extends Base> clazz){ 
    return services.get(clazz); 
} 

private void initializeServiceMap() { 
    services = new HashMap<>(); 
    services.put(Foo.class, new FooService()); 
    services.put(Boo.class, new BooService()); 
    } 
} 

И ваш класс обслуживания:

public class Service { 
List<Base> bases; 
ServiceSupplier serviceSupplier; 

public Service(){ 
    serviceSupplier = new ServiceSupplier(); 

    bases = new ArrayList<>(Arrays.asList(new Foo(), new Boo())); 
} 

public void someMethod() { 
    bases.forEach(base -> serviceSupplier.getServiceOfType(base.getClass()).action(base)); 
    } 
} 

Может показаться много, как только для удаления нескольких «если» заявления, но имея больше услуг, все, что вам нужно сделать, это добавить их в метод initializeServiceMap().