Как отобразить комментарий под статьёй, под которой его написали, через PDO PHP?

Question

[kafkiansky]

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

У меня есть две таблицы:

articles(id, title, slug, content, author_id, date)

и comments(id, author_id, article_id, content, date). Я хочу отображать комментарии под постом, под которым его задали, то есть сравнивать, соответствует ли id из таблицы articles с article_id из таблицы comments. Это самописный блог, никаких шаблонизаторов или паттернов, использую PDO PHP. Примерно делаю следующее:

$stmt = $db->query('SELECT * FROM articles');   
$comments = $db->prepare('SELECT * FROM comments WHERE article_id = ?');

Дальше не совсем понятно, что делать. Как поступить?

Comments

[Yan-s]

Читать про SQL и PDO. Налицо отсутствие базовых знаний.

Алгоритм действий такой:
- получить по урл статью
- получить комменты по id полученной статьи
- вывести данные в разметке

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

[kafkiansky]

Yan-s, уже сделал. по урлу получил через $_SERVER['QUERY_STRING'] значение параметров, обрезал лишнее, получил чистый id статьи, сохранил в переменной, а дальше так:

<?php while ($comment = $comments->fetch(PDO::FETCH_OBJ)): ?>
            <?php if ($comment->article_id === $url): ?>
                <p><?php echo $comment->content; ?></p>
                <span><?php echo $comment->date; ?></span>
                <hr>
            <?php endif;?>
        <?php endwhile; ?>

Возможно, это ужасный код, но я только месяц с этим знаком, извините.

[FanatPHP]

Отркактируй свой пост, добавив в него текущий код который у тебя есть. из тех двух огрызков что ты написал в посте и в комментарии ничего не понятно.

Answer 1

[Yan-s]

Возможно, это ужасный код, но я только месяц с этим знаком, извините.

Да это ужасный код. Тем более для месяца изучения, за который документацию можно было прочитать раз 5 целиком.
$_SERVER['QUERY_STRING']
Вместо этого:
php.net/manual/ru/reserved.variables.get.php
php.net/manual/ru/language.variables.external.php

while ($comment = $comments->fetch(PDO::FETCH_OBJ))

Крайне не рекомендуется мешать HTML разметку с логикой к ней не относящейся. Тем более с логикой работы с БД. Разметка должна быть в отдельном файле.

Хорошие практики для начинающих:
getjump.github.io/ru-php-the-right-way

Comments

[kafkiansky]

Правильно ли я понимаю, что всю логику делаем в отдельном файле, а в разметке используем только переменные, содержащие нужные данные?

[Yan-s]

Kafkiansky, логику приложения в одних файлах, а логика отображения в других. Читаем про MVC.

[kafkiansky]

Yan-s, читал про MVC, написал файл главного приложения, который определяет по урлу контроллер и action и подключает нужные, иначе выбрасывает исключения. этот блог старый, не MVC, и я хотел сделать комментарии, чтобы на собеседовании хоть что-то показать.

Answer 2

[FanatPHP]

$stmt = $db->prepare('SELECT * FROM comments WHERE article_id = ?');
$stmt->execute([$_GET['post_id']]);
$comments = $stmt->fetchAll(PDO::FETCH_OBJ)

И дальше уже выводишь из массива $comments в разметке. Самый простой пример: phpfaq.ru/tpl

Comments

[kafkiansky]

Спасибо. А вот выше писали про то, что не надо путать разметку с логикой, но как иначе мне отобразить комментарии, не трогая разметку? Разве ненормально писать так:
<p><?php echo $comments->content; ?></p> ?

[FanatPHP]

Выше писали что не надо путать логику приложения с логикой отображения. Получение данных это логика приложения. Отображение данных это логика отображения.

Так писать ненормально. Правильно будет так <p><?= $comments->content ?></p>

[kafkiansky]

FanatPHP, а нормально ли делать так:

<h1><a href="articles/showarticle.php?post_id=<?=$row->id ?>"></a></h1>

А потом уже файл showarticle.php отвечает за вывод статьи по id.

[FanatPHP]

ну не совсем.
Сам по себе код нормальный. но showarticle.php тоже должен быть разделен на две части - получение данных и их ввод. Для небольших файлов можно совмещать в одном файле.

Главное чтобы никакой вывод не начинался, пока все данные не будут получены. ВКЛЮЧАЯ шапку сайта

[kafkiansky]

FanatPHP, Да, разумеется. Тяжеловато становится следить за архитектурой приложения, с вашей помощью оно стало понятнее, но всё равно не то, чувствую. Надо смотреть в сторону MVC. Только вот нормального объяснения пока не нашёл, везде или модели пустые, или контроллеры толстые, а читать код фреймворка, чтобы написать своё MVC приложение, наверно, не очень поможет.

[FanatPHP]

"или модели пустые, или контроллеры толстые" - это тавтология :)

MVC тебе еще рано, если для тебя получить комментарии под статью - и то проблема. Не занимайся пока, это реально сложно дял тебя будет, все равно накосячишь. Разделяй как у меня написано - хотя бы на уровне отдельных файлов. Этого пока будет достаточно.

[kafkiansky]

FanatPHP, не могли бы поделиться своим репозиторием или ссылкой на другие интересные репы? интересно читать чужой профессиональный код.

[FanatPHP]

У меня нет серьезных публичных репозиториев. Я уже давно не писал ничего с нуля, только на основе фреймворка. И то что я пишу сейчас на работе - это код на основе Симфони, то есть без понимания фреймворка его тоже не поймешь.

Я всем рекомендую вот этот текст https://symfony.com.ua/doc/current/introduction/fr...

он помогает с переходом к фреймворку.

[kafkiansky]

FanatPHP, Спасибо большое за помощь и советы, очень много растолковали новичку.

[FanatPHP]

На самом деле вся помощь будет в этой статье. Я ее сейчас просмотрел по-быстрому, похоже что это как раз ответы на все твои вопросы, прямо в точку. Тут и МВЦ, и разделение кода и данных, и фреймворк и даже пример как раз блога! только что комментариев нет, но с ними ты уже разобрался.

[kafkiansky]

FanatPHP, да, я уже читаю и вижу, что до некоторых моментов я додумался сам (что подняло настроение), а некоторые моменты можно ещё доработать. Крутая статья, спасибо.