Надо ли в yii2 создавать модели для форм?

Question

[Эдуард]

Давно мучает этот вопрос, стоил ли создавать например модель Singup(как например эта), если можно использовать основную модель User:

public function actionSignup()
    {
        if (!Yii::$app->user->isGuest) {
            return $this->goHome();
        }

        $model = new User();        
        if ($model->load(Yii::$app->request->post())) {
            $model->setPassword($model->password);
            if ($model->save()) {
                if (Yii::$app->getUser()->login($model)) {
                    return $this->goHome();
                }
            }
        }

        return $this->render('signup', [
            'model' => $model,
        ]);
    }

Вместо:

public function actionSignup()
    {
        $model = new SignupForm();
        if ($model->load(Yii::$app->request->post())) {
            if ($user = $model->signup()) {
                if (Yii::$app->getUser()->login($user)) {
                    return $this->goHome();
                }
            }
        }
        return $this->render('signup', [
            'model' => $model,
        ]);
    }

Answer 1

[Елена Степанова]

- Формы более переносимы и переиспользуемы
- Формы могут быть не завязаны на аттрибуты модели и лучше выражать предметную область
- Формы проще тестировать
- С использованием форм проще вести разработку в команде
- Валидация в моделях нарушает принципы SOLID (AR их в принципе нарушает, но хотябы эту часть можно нивелировать)
- Для элементарного CRUD с минимальной бизнес-логикой можно обойтись, но если на модель завязано много логики и сценарии будут разрастаться - модель быстро превратится в божественный объект
- Лепить все валидации в одну форму-модель ничуть не лучше, чем лепить их в Ар-модель -> для каждого кейса отдельная модель-форма
-------------------
Резюме: Используйте валидацию в моделях со сценариями, пока не поймёте для чего нужны формы. Использование ради использования, без осознания - всё равно бессмысленно и беспощадно

Comments

[Дмитрий]

Добрый вечер.
Допустим на странице несколько форм, но все они используют одну и туже модель, таблицу в базе. Различие только в том, что в каждой форме свой набор полей. Решил эту проблему как раз через сценарии.
Или всё таки переделать на несколько моделей, одна модель - одна форма?

[Елена Степанова]

Во-первых, не мыслите в рамках active-record модели. Уже где только не говорилось и разработчики признают, что совершили ошибку из-за названия класса model внеся путанницу. Модель - это не один класс. а совокупность.
>Различие только в том, что в каждой форме свой набор полей
Попробуйте рассмотреть не со стороны формы и полей, а со стороны бизнес-логики - для какой цели служат эти формы, и какие бизнес-процессы должны произойти.

Модели-формы служат для обработки пользовательских данных + подготовки дополнительной информации при необходимости. Чтобы на сохранение в базу шли только валидные данные. Один запрос - одна форма-модель которая должна вернуть валидный набор данных.

> Или всё таки переделать на несколько моделей, одна модель - одна форма?
Нет смысла переделывать, пока не прийдёт понимание - для чего нужно переделывать. Если не прийдёт понимания на конкретном проекте, значит норм и не надо ничего переделывать.

[Дмитрий]

Елена Степанова, формы служат для сбора заказов на перевозку груза.
Первая - обратный звонок(имя и телефон), вторая - короткое сообщение(имя, почта и сообщение), а в третьей разные варианты параметров груза(имя, маршрут и почта всегда есть).
Проект изначально был на yii1, всё свалено в одну кучу, много if else, чтобы разрулить, куда и что подставлять. Разбил на несколько действий с разными сценариями, чтобы поменьше проверок было и не путаться. Но модель осталась одна общая. Вернее для backend и frontend две отдельные и одна родительская, где собраны общие данные для backend и frontend. Во frontend модели идёт проверки в действиях, какой сценарий подключать, и мне кажется, что это полная лажа.

[Елена Степанова]

и данные со всех этих форм у вас хранятся в одной таблице?
Какие действия происходят после заполнения каждой из форм? Просто сохранили что юзер навводил и всё, или они всё-таки порождают какие-то процессы?

Если валидацией занимаются соответствующие запросу формы и обрабатываются соответствующими сервисами нет необходимости плодить копии AR -модели. ее задача только сохранить данные, и вывести данные.

Хотя если у вас что-то типа одностраничника с заказом звонка, обратной связи и калькулятором груза, и особого развития и поддержки проекта не предвидится, можно не заморачиваться.

[yurygolikov]

Елена Степанова,

Формы могут быть не завязаны на аттрибуты модели и лучше выражать предметную область

Какие образом формы должны выражать предметную область? Формы это лишь слой отображения, там может происходить лишь простейшая валидация данных, не завязанная на бизнес-логику. Это можно сравнить с front валидацией форм перед отправкой в SPA.

Валидация в моделях нарушает принципы SOLID

Есть разные реализации, но в большинстве случаев рекомендуется(в том числе в доке yii): в модели как раз должна содержаться валидация бизнес-правил, реализация бизнес-логики предметной области.
AR это просто паттерн в котором сочетается бизнес-сущность и репозиторий(объект отвечающий за хранение данных).

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

[Елена Степанова]

yurygolikov, давно не заходила, я не говорила про хранение бизнес-логики в формах, а про то, что данные форм далеко не всегда могут соответствовать AR-модели = таблице, если это не простой crud.
yiiframework.ru/forum/viewtopic.php?p=212604#p212604
https://github.com/yiisoft/yii2/commit/35f7c61f776...
>>в модели как раз должна содержаться валидация бизнес-правил, реализация бизнес-логики предметной области.
Но бизнес-модель !== AR-модель, модель это не один класс.

Answer 2

[Alexander]

Модели только тонкие AR классы с атрибутами без валидации.
Формы это вход и валидация.
Используйте принцип SRP, формы должны валидировать данные, сущность AR хранить просто поля.

Answer 3

[Impeeeery]

не для "конкретных целей", а для форм. да, надо.
уже не помню чем именно было убого, когда при регистрации нет модели формы, но чем-то точно убого.
но если у вас все и так в порядке, то пока ограничьтесь User, потом перекинете если что.

Comments

[Эдуард]

да, точно, для форм. Вот мне поэтому и интересно какие плюсы дают отдельные модели под формы.

[Impeeeery]

Эдуард, если надо в форму поле добавить то не придется добавлять его в модель=в базу=в другие места CRUD

[Эдуард]

Impeeeery, да, но это опять таки специфичный случай(конкретно для моего примера)

Answer 4

[Boris Köln]

стоил ли создавать например модель Singup

Лучше User со scenario

$model = new User;
$model->scenario = User::SCENARIO_LOGIN;

$model = new User;
$model->scenario = User::SCENARIO_REGISTER;

www.yiiframework.com/doc-2.0/guide-structure-model...

Comments

[Эдуард]

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

[Александр Урих]

Сценарии - самое кромешное и темное зло, которое было придумано в Yii

[Эдуард]

Александр Урих, аргументируйте

[Александр Урих]

Эдуард,
basic шаблон.
Пользователи.
1. Регистрация.
2. Сброс пароля.
3. Изменение роли.
4. Изменение статуса.
5. Редактирование профиля.
6. Добавление/редактирование пользователя в админке.
...
n. ...
По хорошему на каждую операцию создавать отдельный сценарий.
Тут мы имеем уже 6 сценариев, ведь вы не будете разрешать пользователю изменять id или роль, путем помена данных в html. Так же при определенном сценарии может понадобиться совершить какую-то операцию над одним или несколькими полями. Потом все забывается, начинается путаница, горождение костылей, и следующий разраб тихо материт предыдущего