Помогите сократить Экшен чайнику

[alexa777]

Привет. я подправил под себя Экшен, все работает, но поскольку я чайник) то подозреваю что это можно сделать гораздо проще, подскажите как плиз?
Суть в том, чтобы принимать нужный параметр $sort_job, определять по нему, что надо добавить Вакансию или Резюме и добавлять нужную инфу в $model->sort

Код: Выделить всё

   public function actionCreate($sort_job)
    {

        if ($sort_job === 'vac'){
        
            $model = new Job();

        if ($model->load(Yii::$app->request->post())) {
             $model->user_id = Yii::$app->user->id;
             $model->status = 'on';
             $model->sort = 'vacancies';
             $model->save();
            return $this->redirect(['view', 'id' => $model->id]);
        } else {
            return $this->render('create', [
                'model' => $model,
            ]);
        }    
            
        } elseif ($sort_job === 'res') {
         
            $model = new Job();

        if ($model->load(Yii::$app->request->post())) {
             $model->user_id = Yii::$app->user->id;
             $model->status = 'on';
             $model->sort = 'resume';
             $model->save();
            return $this->redirect(['view', 'id' => $model->id]);
        } else {
            return $this->render('create', [
                'model' => $model,
            ]);
        }    
        } else {
           return $this->render('404');
        } 
       
        
    }

[SiZE]

Сделать отдельные экшены для создания сущностей. Создать собственные модели для каждой формы.

Код: Выделить всё

public function actionCreateVacancy()
{
  $model = new \forms\VacancyCreate();
  if ($model->load(Yii::$app->request->post())) {
    try {
      $service = new \services\VacancyCreate(Yii::$app->user);
      $service->createFromVacancyCreateForm($model);
      return $this->redirect(['view', 'id' => $service->getVacacncy()->id]);
    } catch (\Exception $e) {
       Yii::app()->session->setFlash('vacancyCreateError', 'Error on vacancy create');
    }
  }

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

[proctoleha]

Код: Выделить всё

    public function actionCreate($sort_job)
    {
        $model = new Job();

        if ($sort_job !== 'vac' && $sort_job !== 'res') {
            throw new \DomainException('$sort_job must be vac or res');
        }
        
        if ($model->load(Yii::$app->request->post())) {
            $model->user_id = Yii::$app->user->id;
            $model->status = 'on';
            $model->sort = $sort_job == 'vac' ? 'vacancies' : 'resume';
            $model->save();

            return $this->redirect(['view', 'id' => $model->id]);
        }
        
        return $this->render('create', [
            'model' => $model,
        ]);
    }

Выучите что делает ключевое слово return

[alexa777]

Выучите что делает ключевое слово return

Что не так? return тоже выводит как echo или возвращает данные. Объясните лучше поподробнее что имели ввиду

[alexa777]

Сделать отдельные экшены для создания сущностей. Создать собственные модели для каждой формы.

Код: Выделить всё

public function actionCreateVacancy()
{
  $model = new \forms\VacancyCreate();
  if ($model->load(Yii::$app->request->post())) {
    try {
      $service = new \services\VacancyCreate(Yii::$app->user);
      $service->createFromVacancyCreateForm($model);
      return $this->redirect(['view', 'id' => $service->getVacacncy()->id]);
    } catch (\Exception $e) {
       Yii::app()->session->setFlash('vacancyCreateError', 'Error on vacancy create');
    }
  }

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

В принципе вариант, но это еще больше кода и файлов, больше таблиц в БД, при таком раскладе я лучше при своем варианте останусь

[SiZE]

В принципе вариант, но это еще больше кода и файлов, больше таблиц в БД, при таком раскладе я лучше при своем варианте останусь

Хранение данных в БД и разделение форм никак не связано.

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

[alexa777]

В принципе вариант, но это еще больше кода и файлов, больше таблиц в БД, при таком раскладе я лучше при своем варианте останусь

Хранение данных в БД и разделение форм никак не связано.

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

Впринципе ты прав, я просто не сразу понял фразу Создать новые модели(я понял что надо создавать новые таблицы и новые модели к ним) а ты про $model = new \forms\VacancyCreate();

[proctoleha]

Выучите что делает ключевое слово return

Что не так? return тоже выводит как echo или возвращает данные. Объясните лучше поподробнее что имели ввиду

В числе прочего ключевое слово return прекращает работу функции, или метода класса. Т.е. можно просто написать return; , без всего, и функция прекратит работу. И городить ифы после него совсем не нужно.

[alexa777]

Выучите что делает ключевое слово return

Что не так? return тоже выводит как echo или возвращает данные. Объясните лучше поподробнее что имели ввиду

В числе прочего ключевое слово return прекращает работу функции, или метода класса. Т.е. можно просто написать return; , без всего, и функция прекратит работу. И городить ифы после него совсем не нужно.

спасибо, теперь буду знать