Верна ли такая реализация?

Question

[monday_suicide]

public function actionUpdate($id)
    {
        $product = $this->findModel($id);
        
        if (!$updateProduct = Yii::createObject(ProductUpdate::class, [$product, Yii::$app->request->post()])->execute()) {
            return $this->redirect(['/product/view', 'id' => $model->getId()]);
        }
        return $this->render('update', [
            'product' => $product,
            'model' => $updateProduct,
        ]);
    }

//

class ProductUpdate
{
    protected $product;
    protected $post;
    
    
    public function __construct(\app\models\Product $product, array $post)
    {
        $this->product = $product;
        $this->post = $post;
    }
    
    protected function createUpdateForm()
    {
        $productForm = new ProductForm($this->product);
        $productForm->attributes = $this->product->attributes;
        
        if (empty($this->post)) {
            return $productForm;
        }
        $productForm->load($this->post);
        Yii::$app->session->setFlash('success', 'Вы успешно изменили товар!');
        $productForm->save();
        return $productForm;
    }
    
    protected function checkStatus()
    {
        if ($this->product->status == $this->product::STATUS_SOLD_OUT || $this->product->status == $this->product::STATUS_PARTIAL_SALE) {
            Yii::$app->session->setFlash('error', 'Вы не можете изменить этот товар, т.к. он уже продан, либо частично продан!');
            return false;
        }
        return true;
    }
    
    public function execute()
    {
        if ($this->checkStatus()) {
            return $this->createUpdateForm ();
        }
        return false;
    }
    
}

Comments

[Максим]

Какая такая? Суть вопроса опишите

[monday_suicide]

Максим, привет.
Вынос логики из экшона в отдельный класс, и передача $_Post данных через конструктор контейнера?
Не поймите не правильно, я пытаюсь разобраться с DI , и я не понимаю, делаю я это правильно или нет, в правильном ли направлении иду. Доку читал, но хочется чуть больше подробностей.
Спасибо

Answer 1

[Максим]

Выносить из контроллера - правильная вещь. Но не правильно что вы туда затащили всю логику. Флешь сообщения. Обработка формы. Можете посмотреть пример как это делается.

Пример 1
Пример 2

Comments

[monday_suicide]

А куда тогда выносить всю другую логику вроде вывода флеш сообщений и присвоений атрибутов в форме? И как к ним получать потом доступ? Что то пока что не очень укладывается в голове)
С примерами вашими обязательно ознакомлюсь!

[Максим]

Артур, в контроллер. Это никакая не логика. Форма для получения данных от пользователя через HTML. Флеш сообщения - отображение красивых уведомлений. Флеш сообщения и форма есть только в web. В косоли и API их нет.

[monday_suicide]

Максим, хорошо, спасибо большое за консультацию!
Появляется ясность такого подхода!