caxapa.ru :: Для чего существуют AdsService.filterByVehicleTypeByPurpose(), AdsService.filterByVehicleTypeByBody(), AdsService.filterByVehicleTypeByFuel() с парными им VehicleAd.getVehicleTypeByPurpose() VehicleAd.getVehicleTypeByBody() VehicleAd.getVehicleTypeByFuel и что ты будешь делать если появятся фары, багажники, бамперы и проч. критерии и надо построить фильтрацию по ним.

RxTx (27.04.2020 19:27, просмотров: 564) ответил Argon на А кто сказал, что расширяться добавлением методов? В том-то и дело, что всего лишь добавлением значений в enum и соответствующих классов-потомков для классов вида TrackType. А все методы бы оставались неизменными.

Для чего существуют AdsService.filterByVehicleTypeByPurpose(), AdsService.filterByVehicleTypeByBody(), AdsService.filterByVehicleTypeByFuel() с парными им VehicleAd.getVehicleTypeByPurpose() VehicleAd.getVehicleTypeByBody() VehicleAd.getVehicleTypeByFuel и что ты будешь делать если появятся фары, багажники, бамперы и проч. критерии и надо построить фильтрацию по ним.

Зачем и почему в AdsService повторяется один и тот же код вида:

for (VehicleAd ad : adList) {
    if (ad.getVehicleTypeByPurpose().equals(vehicleType)) {

Я не знаю какой там уровень, изучен ли полиморфизм, а вдруг за этим последует пример "это было как делать не надо", "вместо этого делайте вот так"?

но вот это

    private VehicleTypeByPurpose vehicleTypeByPurpose;
    private VehicleTypeByBody vehicleTypeByBody;
    private VehicleTypeByFuel vehicleTypeByFuel;

с методами

 VehicleAd.getVehicleTypeByBody()

Это именно то, как в ООП делать не надо.

Спасибо, князь. Вы настоящий дворянин. И программист.

Ответить

- Фары, багажники, бамперы - это не типы/подтипы машин и к задаче не относятся. Я имел ввиду возможность расширения TruckType на подтипы добавлением потомков TruckType, далее классов-внуков, классов-правнуков и т.п. с сохранением возможности фильтрации как по подтипу, так и по типу TruckType. Уже сделал, все получилось. - Argon(27.04.2020 20:48)

Средства и методы разработки