Обращение к методам из вью это говнокод?

Question

[Insout]

Здравствуйте.

Есть некий объект, к этому объекту прикрепляются пользователи, допустим этот объект это задача, а пользователи - исполнители.

Во вью, помимо контента самой задачи, необходимо вывести всех пользователей которые прикреплены к данной задаче.

Обращение к функции из вью это нормально или необходимо заранее подготовить массив из пользователей и просто вывести циклом его во вью ?

<?php foreach ($model->relUsers as $t) : ?>
        <span><?= User::getById($t,'name') ?></span>
<?php endforeach; ?>

Answer 1

[Кирилл Мокевнин]

Функции бывают разные.

Ключевые понятия:

1. Идемпотентность.
1. Чистая функция.
1. Побочные эффекты.
1. Детерминированность

Так вот во вью, в идеале должны вызываться только чистые функции или на худой конец функции без побочных эффектов.

В вашей запрос в базу это побочный эффект, такое лучше делать в контроллере, а во вью передавать данные.

Но на самом деле ваш код еще и антипаттерн, так называемая проблема `select N + 1`. Запросы в цикле - зло. Вы не пользуетесь ни базой данных, ни взможностями вашей ORM (если это она).

Comments

[mitaichik]

Почему запрос в базу это побочный эффект? Побочный эффект меняет внутреннее состояние объекта, системы. Запрос же в чистом виде ничего не меняет.

[Кирилл Мокевнин]

Любое IO это всегда побочный эффект. Если ваша база лежит, то функция просто упадет.

[Кирилл Мокевнин]

Кстати факт того что вы "читаете данные" из базы, всегда приводитк изменениям внутри базы данных. Меняется статистика запросов, обновляются кеши и бог знает что еще. Вы даже можете на локи нарваться и помешать другим транзакциям выполняться.

Answer 2

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

Ошибка в том, что $model->relUsers это должна быть связь возвращающая массив моделей. И тогда все будет отлично:

<?php foreach ($model->relUsers as $t) : ?>
        <span><?= $t->username ?></span>
<?php endforeach; ?>

Так что подозреваю, что имеет место ошибка архитектуры. Вы храните список id в строке, а под это должна быть отдельная таблица:
id | model_id | user_id
и связь hasMany в модели с моделями пользователей, через эту базу.

Comments

[Insout]

Нет. Храню в отдельной таблице. relUsers это массив айди пользователей, который готовится в контроллере.

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

Insout, А зачем его готовить в контролере, если это должна быть связь в моделе?

[Dmitry Bay]

Insout, тогда загружайте этот массив там же в контроллере.

К примеру, если вы готовите массив пользователей в контроллере (или только селект с именем), то вы делаете один запрос на базу.

А если вы делаете в вьюхе как у вас в примере , то на 100 пользователей будете делать 100 запросов на ровном месте.

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

Дмитрий Байчапанов, +1, но все же связь и жадная загрузка мне кажется более верным методом

Answer 3

[Александр Сисюкин]

Нормально если это необходимо, например тот же twig он разрешает добавление пользовательских функций.

В вашем примере я бы просто подготовил нужный массив и вывел его

Answer 4

[mitaichik]

Правильно будет сделать релейшен, что то типа $task->executors

Наверняка вы используете промежуточную таблицу (я так понял что relUsers - это она). В таком случае релейшн нужно делать с помощью метода via.

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

Answer 5

[ThunderCat]

Наборы элементов готовятся в контроллере и передаются во вью, где только итерируются.

<?php foreach ($model->relUsers as $t) : ?>
        <span><?= User::getById($t,'name') ?></span>
<?php endforeach; ?>

Тут, помимо того что у вас запросы идут прямо по ходу вывода контента(привет из 2000), так еще и вместо одного запроса выбора пользователей вы выбираете их по одному, КАЖДОГО, КАРЛ! А если на страничке 300 пользователей(или 3000) это вообще не гуд. Для этого есть коллекции.

Answer 6

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

View - для того и называется view (отображение), что должно только отображать.
Получать данные нужно в контроллере.
Максмсум, что можно в представлении - условия, типа если массив с данными пуст, вывести ошибку, иначе данные