Где лучше прописывать условия в YII2?

Question

[Space]

Есть несколько вопросов по yii2, а именно оформление кода.

1. Красиво ли и целесообразно писать условия и циклы в модели, а не в контроллере, если это удобнее?
2. Мне нужно выполнять ajax запросы из БД. Стоит это оформить прям в контроллере или равносильно вынести ajax во вьюшку и передавать через POST уже контроллеру?
3. Как лучше - делать все циклы с массивами в контроллере и переменные уже передавать во вьюшку или равносильно, что и разбирать массивы во вью?

Есть ли вообще какие-то правила по этому поводу - оформление кода?

Answer 1

[Dave]

1. Красиво ли и целесообразно писать условия и циклы в модели, а не в контроллере, если это удобнее?

Да желательно писать всё что связано с обработкой бизнес-логики (там парсинги всякие) в моделе. В контроллере же, нужно это вызывать и просто передавать переменные ($_POST, $_GET)

2. Мне нужно выполнять ajax запросы из БД. Стоит это оформить прям в контроллере или равносильно вынести ajax во вьюшку и передавать через POST уже контроллеру?

Но ведь УРЛ от Ajax-запроса все равно вызывает экшен контроллера. POST'ом следует выполнять только то, что изменяет что-то - пишет в базу например. А для выборки нужно использовать GET. Во вью задаётся клинтская логика, которая вызывает урл и в ответ делается что-то - обычно это в 99% делается именно так.

3. Как лучше - делать все циклы с массивами в контроллере и переменные уже передавать во вьюшку или равносильно, что и разбирать массивы во вью?

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

Есть ли вообще какие-то правила по этому поводу - оформление кода?

Почитай что такое SOLID, Separation of Concerns

А что касается MVC:

- то модели должны быть толстыми (вся обработка должна быть только там)
- конроллеры тонкими (должны уметь только получать и передавать)
- а вью - тупыми (должны знать только как отображать)

Как-то так

Comments

[Space]

Спасибо. А можно увидеть пример на второй пункт?

Насчет 3:
т.е. мне нужно все разобрать в модели? А как же в экшине принять эти данные из модели (я обычно делаю запрос к базе в экшене, разбираю их и отправляю простые массивы уже во вью, которые одним циклом выводятся во вью, например, список комментариев).

[Space]

И чем вообще чревато делать наоборот - тонкие модели и толстые вью?

[Dave]

Rusnire: Если это хоть как-то касается обработки данных, то да разбираешь в модели. Нет - просто вызываешь в экшене.

[Dave]

Rusnire: Поддерживаемость, читабельность, и это впринципе правило MVC

[Space]

Давид: вот это находится в контроллере, т.е. правильно будет вынести это в отдельную функцию в модели и уже в экшене вызывать эту функцию и получить нужные переменные?

public function actionIndex()
    {
        $group_id = '-10639516';
        $answer = new Comments;
        if($answer->load(Yii::$app->request->post()) && $answer->validate()) {
           $infoAnswer =  Yii::$app->vk->api('wall.addComment', array('owner_id' => $group_id,'post_id'=>'3', 'from_group'=>'1','text'=>$answer->comment,'reply_to_comment'=>$answer->cid));

        }

        $comMes =  Comments::find()->select('DISTINCT `commes`')->orderBy(['id'=>SORT_DESC])->all();

        return $this->render('index', [
            'answer' => $answer,
            'comMes' => $comMes,
        ]);
    }

[Dave]

Rusnire: Открой модель Comments и допиши один метод - назови его getAll(). Внутрь засунь `return self::find()->select('DISTINCT `commes`')->orderBy(['id'=>SORT_DESC])->all();` И потом в экшене вызывай так: 'comMes' => Comments::getAll() вместо того что у тебя там есть сейчас. А в остальном все нормально.

[Space]

Давид: спасибо, Давид, начинает проясняться.
А если таких выборок несколько, то на каждую переменную создавать свой метод?

И еще, вот добавление коммента правильно сделано:
// Заносим коммент в базу
Comments::addTableRec($group_id,$cid, $uid, $reply_to_cid, $comment, $date, $newCom, $oldCom, $answer, $comMes, $post_id, $can_edit);

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

[Dave]

Rusnire: Когда аргументов слишком много, лучше всего передавать массивом, то есть Comments::addTableRec($data), где $data - это массив из всех $group_id, $cid, .... А если выборок несколько, то метод создавать да - желательно, так как всю SQL (т.е все кишки) логику ты держишь в моделе, и эта SQL логика даже не светится в контроллере. Так правильнее по MVC.

[Space]

Давид: спасибо большое за все подробности.
Если можно, то ответьте еще на вопрос в комментарии ниже про ajax.

Answer 2

[amstr1k]

всё завит от того какой подход выберите именно вы. Можно вообще выделить слой сервисов которые будут делать ту или иную работу. Но я бы вьюхи использовал только для отображения данных