Как разделить модели в Yii2?

Question

[vitovt]

Наткнулся на мысль, что при сохранении данных из форм стоит валидировать данные сначала валидирующей моделью, а затем сохранять все через модель, наследуемую от ActiveRecord.

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

QuestionForm:

namespace backend\models;

use yii\base\Model;

class QuestionForm extends Model {

    public $active;
    public $question;
    public $answer;

    public function rules()
    {
        return[
            ['question','required'],
            ['answer','required'],
            ['active','boolean'],
        ];
    }

    public function create()
    {
        if(!$this->validate()) {
            return null;
        }

        $model = new Question();        
         ......
        return $model->save() ? true : false;
    }
}

В контроллере actionCreate():

public function actionCreate() {

        $model = new QuestionForm();

        if( \Yii::$app->request->isPost) {
            if( $model->load(\Yii::$app->request->post()) && $model->create(  ) ) {
             ...........
            }
        }
        
        return $this->render('create', ['model' => $model]);
    }

Тут все понятно. В view будет форма, с полями, названия которых будут что-то вроде QuestionForm[question] или QuestionForm[active]

Вот теперь самый главный вопрос, как сделать actionUpdate используя валидируюущую модель?

Если так

public function actionUpdate($id) {

         $model = $this->findModel($id);

        if( \Yii::$app->request->isPost) {

            if($model->load( \Yii::$app->request->post()) && $model->save() ) {

                $this->setSuccessAlert();
                $this->redirect(['update', 'id' => $model->id ]);

            }
        }

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

То валиадция будет уже в основной модели, которая наследована от ActiveRecord и получается что валидацию нужно дублировать и в Model и здесь.

Answer 1

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

Если у Вас есть модель представления с правилами валидации (а Ваша QuestionForm это по сути дела и есть модель представления) и Вы полностью хотите избавиться от валидаций в ActiveRecord тогда Вам в actionUpdate стоит использовать именно ее, а не ActiveRecord. Т.е. загружать данные из поста в QuestionForm, валидировать и уже после этого загонять их в AR и сохранять ...
Например, в самом простом виде это могло бы выглядеть так:

class QuestionForm extends \yii\base\Model {

	public function findModel($id){
		$model = MyActiveRecord::findOne($id)
		return $model;
	}
		
	public function update($id)
	{
		$model = $this->findModel($id);
		if (is_null($model)) {
			return false;
		}
		$model->load($this->attributes, '');
		return $model->save();		
	}
}

Comments

[vitovt]

Как? Я имею в виду в теории я понимаю, что надо сделать $model = $this->findModel($id); потом $validateForm = new QuestionForm(); и что потом?

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

vitovt: дополнил свой ответ примером

[Максим Тимофеев]

Максим Федоров: А зачем? Что бы с сценариями не возиться?

[vitovt]

Максим Федоров: спасибо, кажется именно эти строки не давали пазлу сложитья

public function findModel($id){
$model = MyActiveRecord::findOne($id)
return $model;
}

так как я не понимал как вытянуть данные в валидирующей модели из невалидирующе.

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

vitovt: ну это все грубый пример, что б показать суть ... но я бы рекомендовал вообще для связи между моделью представления и ActiveRecord реализовать некий сервис....

[vitovt]

Максим Федоров: Этот совет я тоже слышу часто и понимаю, что это вероятно правильно но пока без примеров хорошего кода не понимаю как это работает, не видел best practics на эту тему.

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

Максим Тимофеев: объяснить в двух словах сложно, рекомендую почитать вот это https://smarly.net/asp-net-mvc-4-in-action/working... , хоть и примеры на asp но саму суть статья передает

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

vitovt: к сожалению ссылок привести не могу, но могу посоветовать поискать на форуме yii эта тема там поднималась и активно обсуждалась (с примерами)

[vitovt]

Максим Федоров: попробую поискать. Последний вопрос для закрепления чтобы я понимал туда ли копать. В общем Ваш пример кое что прояснил, но остался вопрос передачи $model в view где генерируется форма. Ведь если я делаю $model = (new QuestionForm())->find($id); то в $model уже объект Question и соответсвенно форма генерирует input с именами Question[active] Question[answer] и так далее. С большего такой код работает - за исключением вывода error которые, естесвенно в $model отсуствуют

$model = (new QuestionForm())->find($id);

$validation = new QuestionForm();
$validation->find( $id );

if( \Yii::$app->request->isPost) {

if($validation->load( \Yii::$app->request->post(), 'Question') && $validation->save( $model ) ) {

$this->setSuccessAlert('', true);
return $this->redirect(['update', 'id' => $model->id ]);
}
}

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

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

vitovt: опять таки для вывода формы с данными которые нужно редактировать нужно сначала загружать данные из ActiveRecord в форму представления, и рендерить уже ее.
P.S. а зачем Вы в данном случае вообще сделали форму представления? судя по коду это обычный круд, на котором она не особо имеет смысл....

[vitovt]

Максим Федоров: все верно, CRUD, тогда возникает вопрос при каких задачах имеет смысл делать такое разделение? Может я вообще задачу не верно понимаю и реализую ее не так как надо, может в данном случае для обычного CRUD стоит отказаться от этой модели и делать все через ActiveRecord

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

vitovt: для crud-ов вполне подойдет ActiveRecord, используя модели представлений для простых крудов, Вы только увеличиваете объем кода проекта (а соответственно и увеличиваете сложность его понимания и поддержки). Модели представлений лучше использовать в более сложных ситуациях, например: когда у Вас есть возможность редактирования сущности в разных местах, и в каждом месте необходим набор своих полей и т.п., или допустим когда у Вас должна быть какая-та специфическая логика для определенного представления.

[vitovt]

Максим Федоров: вот оно что, теперь немного начинаю понимать, что зря поддался мысли всегда и везде перед созданием\сохранением прогонять данные через модель extends Model