Как организовать контроллеры в Silex?

Question

[Mr Freeman]

Я делал сначала через сервисы их

class StatsController{

    protected $app;

    function __construct(Application $app){

        $this->model = new Stats($this->app);

        if(UserHelper::isOnlySalesManager($this->app)){
            $this->model->isSalesManager  = true;
        }
    }
}

Тогда приходилось каждый контроллер вызывать отдельно в init.php, отдавая в конструктор экземпляр приложения

Это нужно исправить, чтобы они сами загружались.

В роутах тогда прописываю роут с указанием экшена

$admin->get('/stat', 'Controllers\\StatsController::dailyAction')

Тогда все бы ничего, но вконструкторе контроллера теперь не будет доступна $app - что приведет к реорганизации структуры. Может есть способы, как можно не используя контроллеры как сервисы, в конструкторе контроллера получать экземпляр приложения? Чтобы там уже вытаскивать общие куски кода. Спасибо.

Answer 1

[Сергей Протько]

silex.sensiolabs.org/doc/providers/service_control...

Comments

[Mr Freeman]

> Тогда приходилось каждый контроллер вызывать отдельно в init.php, отдавая в конструктор экземпляр приложения

> Это нужно исправить, чтобы они сами загружались.

Это вы сервисную архитектуру предлагаете. Я как раз хотел от нее избавиться. То есть чтобы в роутах указал контроллер, и он сам уже определился, и соотвественно еще в конструктор передать экземпляр приложения. Есть ли такая возможность.

[Сергей Протько]

vladamir smertniy: я не сервисную архитектуру предлагаю, я предлагаю регистрировать контроллеры как сервисы. Если они зарегистрированы как сервисы вы сами можете спокойно разруливать что куда там передается в конструктор, хоть App хоть только то что нужно. Ну и опять же вы можете сделать одну фабрику для всех контроллеров если хотите.

Далее все происходит ровно по вашему сценарию. Вы прописываете какой метод какого контроллера-сервиса дернуть, далее уже на основе рефлексии будет произведен подбор того что вам там нужно, App, Request или еще чего...

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

Answer 2

[miy]

В Slim framework использовал так:

$app->get('/main/:param', function ($param) use($app) {
     (new \Controllers\Main($app))->index($param);
});

Comments

[Сергей Протько]

печалька... вы в контроллер весь контейнер пихали?

[miy]

Можно подробнее в чем заключается "печалька"?

[Сергей Протько]

miy: ну на самом деле в контексте контроллеров печальки нет, печалька тут в другом - инстанцирование контроллера по сути у вас происходит в контроллере. Что не правильно с идеологической точки зрения.

Если уж хочется делать контроллеры тонкими, можно делать так (в silex)

$app->get('/main/{param}', function (Request $request, Application $app) {
    return $app['main_provider']->getMain($request->get('param'));    
});

А в сервисах мы уже делаем конкретные задачи и реализуем операционный слой (или application layer)

[miy]

Спасибо за разъяснение!

[Mr Freeman]

Сергей Протько:

> Если уж хочется делать контроллеры тонкими, можно делать так

И где же у вас в примере есть контроллер?

[Сергей Протько]

vladamir smertniy: ну вот это замыкание и есть контроллер. И по сути больше он ничего делать не должен, ну он может еще попросить какой другой сервис сформировать нужное представление данных на выход, еще поюс строчка две.

Контроллер это медиатор между слоями и только, его обязанность преобразовать данные из формата интерфейса (http, cli, etc) в формат приложения (то что кушают сервисы). Не больше. Никакой логики там быть не должно, только то что относится к слою интерфейса.