Валидация в контроллере и распределение ролей в MVC?

Question

[kot-samolet]

Почитал несколько похожих вопросов, например Как и где правильно осуществлять валидацию данных в MVC? пишут что валидацией должен заниматься контроллер. Это понятно если валидировать нужно относительную простую информацию типа URL или IP и то если таких проверок много не множить же их в каждом контроллере... Напрашивается вынос в отдельный класс, назовём его Validator и просто подключение его нужных методов в контроллере.

1) К какому типу тогда будет относиться сам класс Validator? Model, Controller или что-то типа Хелпера или Middleware?

2) В системе запрашивают страницу редактирования сотрудника вида domain.com/employer/edit/123 срабатывает класс employer метод edit. Где надо проверить само существование сотрудника № 123? Я сейчас проверяю в модели, т.е. в классе employer методе edit в самом начале. Вроде бы надо вынести в контроллер, но для такой проверки нужна работа с БД, а работа контроллера чисто разруливать запросы. На ум идёт так же только создание класса Validator который с БД будет работать но он сам по себе уже полноценной моделью по сути будет.

Answer 1

[Максим Федоров]

В сервисе.

1. Контроллер получает запрос
2. Контроллер создает форму
3. Если форма валидируется -- вызывается метод сервиса
Пример хорошего ООП на Yii2 от Елисеева:
экшн контроллера: https://github.com/ElisDN/yii2-demo-shop/blob/mast...
метод сервиса запрашивает юзера: https://goo.gl/g9Vf9G

4. Рендерится вьюшка

Comments

[kot-samolet]

В сервисе

В имеете ввиду что Validator будет сервисным классом? Т.е. Хелпером?

Контроллер создает форму

Погодите, какая форма, в вопросе нет никакой формы которая оптправляется или которую надо сгенирировать

Yii2

Это вопрос не по Yii2

[Максим Федоров]

kot-samolet,

Это вопрос не по Yii2

я понял, что общий вопрос, пример вам привел просто, в Yii2 так тоже кстати не делают, приведенные примеры -- выход из стандартной практики работы с этим фреймворком в сторону хорошего ООП

EmployerValidator будет уже моделью, только будет уже моделью валидации, а не какой-то "полноценной" моделью

[kot-samolet]

Максим Федоров, Я Yii2 не знаю, поэтому по возможности прошу пояснить ссылку

public function actionUpdate($id)
    {
        $user = $this->findModel($id);
        $form = new UserEditForm($user);
        if ($form->load(Yii::$app->request->post()) && $form->validate()) {
            try {
                $this->service->edit($user->id, $form);
                return $this->redirect(['view', 'id' => $user->id]);
            } catch (\DomainException $e) {
                Yii::$app->errorHandler->logException($e);
                Yii::$app->session->setFlash('error', $e->getMessage());
            }
        }
        return $this->render('update', [
            'model' => $form,
            'user' => $user,
        ]);
    }

Сначала они ищут этого юзера с помощью модели поиска передавая в неё $id
$user = $this->findModel($id);

Далее всё это переходит сразу к форме
$form = new UserEditForm($user);

Но непонятно что происходит если findModel($id) вернула false надо бы остановить всё безобразие в этом месте.

Вот тут тоже не понятно. Контроллер что сам используя объект UserManageService пытается отредактировать форму у этого юзера?
$this->service->edit($user->id, $form);
С чего бы это? Если это работа чисто модели уже...

[Максим Федоров]

kot-samolet,

Но непонятно что происходит если findModel($id) вернула false надо бы остановить всё безобразие в этом месте.

исключение же, вот экшн findModel
https://github.com/ElisDN/yii2-demo-shop/blob/mast...

[Максим Федоров]

kot-samolet,

С чего бы это? Если это работа чисто модели уже..

вызывается сервис, сам контроллер не делает. логика инкапсулирвоана в сервисе, контроллер передал сервису только id и форму