Как написать generic фабрику?

Question

[Mozzarella]

Есть иерархия классов моделей из бд.

class Car {
    private String name;
}
class Ford extends Car {
    private String name;
}
class Audi extends Car {
    private String name;
}

Для каждой модели нужен класс который принимает как аргумент один из наследников и возвращает Object

class FordHandler {
    Object handle(Ford ford) {
        return new Object();
    }
class AudiHandler {
    Object handle(Audi ford) {
        return new Object();
    }
}

Где-то снаружи должен быть похожий на этот

List<Car> carList;
List<Object> result;
carList.forEach((car) -> {
    result.add(factory.get(car.getClass()).handle(car));
})

Как правильно это реализовать?

Comments

[roswell]

При указанных условиях через дженерики в одном классе -- никак, только через рефлексию или switch. А для чего вообще такой огород городить? Модель сама может реализовывать handle() из какого-нибудь интерфейса CarHandler, например.

[Mozzarella]

roswell, просто во-первых, не хотелось бы засорять модель данных, во-вторых, возмоjно расширение handler'ов, например, FordOfficialHandler, AudiOfficialHandler.
Для такой задачи думаю некорректно внедрять контейнер зависимостей

[roswell]

Mozzarella, ясно-понятно. Ну, а классы Handler-ов откуда будут браться? Тем более, что их там, оказывается, может быть несколько для каждого потомка Car.

[Mozzarella]

roswell, из отдельного пакета, просто возможно потребуется переключение в основном методе с OfficalHandler'ов на UnofficialHandler'ы

carList.forEach((car) -> {
    result.add(unofficialfactory.get(car.getClass()).handle(car));
})

[roswell]

Mozzarella, не, это только через рефлексию. Потому что я пока что понимаю задачу как "найти имя нужного класса по типу переданного в метод параметра", а дженериками это дело не обустроить.

[Mozzarella]

roswell, вроде как рефлексия - плохой тон, наверное лучше добавить нужные методы в саму модель, используя, например, паттерн visitor?

[roswell]

Mozzarella, ну, в общем-то, я сразу и предложил модифицировать модели. Тогда было бы всё прозрачней и проще -- передаём в метод интерфейса (или абстрактного класса) модели экземпляр нужного обработчика, а тот уже как нужно взаимодействует с моделью любым дозволенным образом.

[Mozzarella]

roswell, спасибо, просто все-таки хотелось разделить работу с данными и сами данные

Answer 1

[Дмитрий Александров]

Если я правильно понял задачу то что то типа такого.

public interface Handler<T> {
  Object handle(T obj);
}
class FordHandler implements Handler<Ford>{...}
/// в фабрику
Map<Class<T>, Handler<T>> registry = new HashMap<>();
public void registerHandler(Class<T> carType, Class<? extends Handler> handlerType) {
    registry.put(carType, handlerType);
}
public <T> Handler<T> getHandler(Class<T> clazz) {
  return registry.get(clazz).newInstance();
}
///наполняешь фабрику
factory.registerHandler(Ford.class, FordHandler.class);
factory.registerHandler(Audi.class, AudiHandler.class);
///
List<Car> carList;
List<Object> result;
carList.forEach((car) -> {
    result.add(factory.getHandler(car.getClass()).handle(car));
})

Answer 2

[Marat Khayrutdinov]

На вскидку не придумал, как избавиться от @SuppressWarnings("unchecked"), может только пойти другим путем.

public class Main {
  @SuppressWarnings("unchecked")
  public static void main(String[] args) {
    Map<Class<? extends Car>, Handler> carHandlers = new HashMap<>();
    carHandlers.put(Car.class, new CarHandler());
    carHandlers.put(Ford.class, new FordHandler());
    Car car = new Car();
    Car ford = new Ford();
    Object carTest = carHandlers.get(car.getClass()).handle(car);
    Object fordTest = carHandlers.get(ford.getClass()).handle(ford);
    System.out.println(carTest);
    System.out.println(fordTest);
  }

  interface Handler<T extends Car> {
    Object handle(T obj);
  }

  static class Car {
    String test() {
      return "I'm Car";
    }
  }

  static class Ford extends Car {
    String test() {
      return "I'm Ford";
    }
  }

  static class CarHandler implements Handler<Car> {
    @Override
    public Object handle(Car car) {
      return car.test();
    }
  }

  static class FordHandler implements Handler<Ford> {
    @Override
    public Object handle(Ford ford) {
      return ford.test();
    }
  }
}

Comments

[Marat Khayrutdinov]

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

private static <T extends Car> void putHandler(Map<Class<? extends Car>, Handler> carHandlers, Class<T> car, Handler<T> handler) {
    carHandlers.put(car, handler);
  }