MVC | Где получать имя категории, пользователей?

Question

[Влад]

Делаю mvc oop блог, и столкнулся с вопросом
Есть пост, в котором отображется катерия и автор. В базе данных лежат category_id и author_id
Вопрос в том, как правильно организовать получение имени категории и автора, имея их айди?
Изначально была идея передать объект модели(category, user) в вид, и уже там выполнять нужные методы(getCategoryName или getUserName). Но потом узнал, что это плохая практика, и надо делать вид максимально простым, подавая туда уже готовые данные.
Сейчас пришел к такому решению.
В контроллере для каждого поста добавить элементы:

$postList = Post::getPostList(); // модель возвращает список постов
foreach($postList as $post) {
    $post['category_name'] = Category::getCategoryName($post['category_id']);
    $post['author_name'] = User::getCategoryName($post['author_id']);
}
$view->render($postList); // передаю посты в вид

Можно ли поступить так?

Comments

[N]

Ещё плохая идея в цикле запросы делать...судя по этим строкам, там идут запросы в БД:

// ...
$post['category_name'] = Category::getCategoryName($post['category_id']);
$post['author_name'] = User::getCategoryName($post['author_id']);
// ...

[Влад]

N, да, запросы в базу идут. А как тогда лучше поступить?

[N]

Влад,
Можно например одним запросом внутри getPostList получать данные из "связных" таблиц...гуглите что такое "JOIN в MySQL" например...если же не всегда нужны эти категории и авторы, то сделать например параметры в функции на основе которых делать или не делать JOIN'ы...
Ну вариаций много ещё на самом деле...

[Влад]

N, спасибо!!

[FanatPHP]

Но потом узнал, что это плохая практика,

Это не плохая практика, а ад кромешный.

А по вопросу, для начала надо понять, что модель в МВЦ - это НЕ класс для получения данных из таблицы в БД.

Answer 1

[Илья]

Для начала если хотите использовать ООП перестаньте использовать ассоциативные массивы.
У вас данные из базы забирает маппер/ar/запрос соответственно он и должен заботиться об остальных данных.
Пример (псевдокод):

Вариант 1

$postList = Post::getPostList([
    'join' => [
        'category',
        'author',
    ],
]);
foreach($postList as $post) {
    // то есть маппер у вас уже эти данные получил и перед отдачей объектов инициализировал их
    $post->getCategoryName();
    $post->getAuthorName();
}
$view->render($postList);

Или так:

Вариант 2

$postList = Post::getPostList();

$postIds = array_column($postList, 'id');
$categoryNames = Category::getNamesWherePostId($postIds);
$authorNames = User::getNamesWherePostId($postIds);

foreach($postList as $post) {
    $post['category_name'] = $categoryNames[$post->categoryId] ?? null;
    $post['author_name'] = $authorNames[$post->authorId] ?? null;
}
$view->render($postList);