Как правильнее работать в сервисом, контрактом?

Question

[Evgeniy Golovin]

Помогите разобраться с контрактами (интерфейсами) и провайдерами.

Хочу сделать на их основе подгрузку каталога скажем брендов или скажем городов

Я делаю следующее

1) Создал контракт Contracts/Catalog.php

interface Catalog {
    public function show();
}

2) Создал два сервиса Services/Catalog/BrandService.php и Services/Catalog/CityService.php

class BrandService implements Catalog {
    public function show() {
        dd('Каталог брендов');
    }
}

class CityService implements Catalog {
    public function show() {
        dd('Каталог городов');
    }
}

И вот дальше у меня тупняк. Хотел создать общий контроллер Http/Controllers/CatalogController.php

class CatalogController extends Controller {
    function showBrand(BrandService $brand) {
      return $brand->show();
    }
   
   function showCity(CityService $city) {
      return $city->show();
    }
}

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

И ловить потом

Route::get('/catalog/city', [BaseCatalogController::class, 'city']);
Route::get('/catalog/brand', [BaseCatalogController::class, 'brand']);

Направьте, пожалуйста, на правильный подход в реализации :)

Comments

[Дмитрий]

Как бы теоретически контракты используются для того что бы у вас код который работает с этим контрактом не парился с какой конкретно реализацией он работает.
То бишь - есть смысл в таком коде

function showBrand(Catalog $service) {
      return $service->show();
    }

Тогда да - туда может прилететь CityService, BrandService - но методу похрен он четко знает что есть метод show потому что они выполняют контракт Catalog

В случае с этим

class CatalogController extends Controller {
    function showBrand(BrandService $brand) {
      return $brand->show();
    }
   
   function showCity(CityService $city) {
      return $city->show();
    }
}

какая хрен разница методу showCity что переданный параметр имплементирует контракт Catalog? Ему от этого ни горячо ни холодно, он ждет конкретную реализацию и будет работать с ней как с отдельной реализацией. В чем вообще причина связывать CityService и BrandService одним контрактом?

Второе. сильное подозрение что вы в метод show собираетесь захерачить работу с view. А нахрена?

Answer 1

[Дмитрий Гординский]

Вы странным образом используете интерфейсы.

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

А то выходит использование интерфейсов ради использования интерфейсов.