Как спроектировать следующую логику?

Question

[magary4]

есть страница поиска отображаемая SearchControllerom
есть сам сервис поиска SearchService::search( $params ) внутри которого реализована логика сортировки, пагинации и еще много другой логики которая будет одинакова для всех проектов

базывое параметры поиска будут менятся от проекта к проекту

сейчас это как-то так

class SearchController extends Controller
{
    private function getParameterForSearchQuery()
    {
        return [ "type"=>["post","folder","group"], "subtree"=>"events" ]; 
    }

    public function indexAction()
   {
        . . .
        . . .
       $this->render('::index.html.twig',  $this->searchService->search( $this->getParameterForSearchQuery() )  );
   }
}

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

getParameterForSearchQuery() { return ["type"=>"file"] }

на сколько это правильно/неправильно?

подскажите лучшую практику. паттерны и т д

спасибо

Answer 1

[voidMan]

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

Comments

[magary4]

Есть ли принципиальная разница сделать это в конфиге и переопределять его или в методе класса и наследовать?

[voidMan]

Оба способа будут работать, но конфиг и общий бандл — это более лучшая практика.

[magary4]

а смысл этого отдельного бандла?
один конфиг файл + один класс

[voidMan]

magary4: Бандл для переиспользования в других проектах. Подключил бандл, конфиг определил и готово. В дальнейшем, если понадобится эту функциональность доработать для всех проектов, нужно будет изменить код только в одном месте и подтянуть обновленный бандл в проектах через composer update.

[magary4]

ну так оно сейчас и так в одельном бандле
то что я выше написал не в AppBundle
в нем же дефолтная реализация контроллера которая в дальнейшем может быть переопределена