Блог

У меня разные классы фильтров, и каждый класс имеет несколько методов фильтрации. Есть (пока) один случай, когда мне нужно применить фильтры из 2 классов. Что я сделал, так это то, что я создал другой класс с именем FilterManager, который хранит список всех классов фильтров, которые вам нужны, а затем выполняет итерацию по списку и применяет фильтры.

 class ThreadFilters 
{
    protected $builder;
    protected $supportedFilters = ['newThreads'];
    public function newThreads($userId)
    {
        $this->builder->orderBy('created_at', 'DESC');
    }
}
  

 class FilterManager
{
    protected $filters = [];
    
    public function addFilter($filter)
    {
        $this->filters[] = $filter;
    }

    public function apply($builder)
    {
        foreach($this->filters as $filter)
        {
                $filter->builder = $builder;
                foreach($filter->supportedFilters as $filterMethod)
                {   
                    $filter->$filterMethod()
                }
                $builder = $filter->builder;
        }
    
    }
}
  

Теперь в контроллере я могу сделать что-то вроде

Первый подход

 class ThreadController(FilterManager $filterManager)
{
    $filters = $filterManager->addFilter(new ThreadFilter());
    $filters->apply($someBuilder);
}
  

С другой стороны, я мог бы использовать функцию приложения

 app(ThreadFilters::class)
  

и в контейнере сервиса в принципе я мог бы сделать

 $this->app->bind(ThreadFilters::class, function($app){
    $filterManager = new FilterManager();
    $filterManager->addFilter(new ThreadFilters());
    return $filterManager;
});
  

и поэтому мой контроллер будет выглядеть так

Второй подход

 class ThreadController()
{
    $filters = app(ThreadFilters::class);
    $filter->apply($someBuilder);
}
  

Мой вопрос в том, какое решение является хорошим / приемлемым.

При первом подходе вводится зависимость FilterManager, и контроллер должен добавить фильтры потоков

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