Почему foreach не выводит данные?

Question

[Евгений Питаев]

Столкнулся с тем что foreach ни чего не выводит pastebin.com/EJXCpYpF
сам класс pastebin.com/E3kQHxmW

но print_r($comments);
выводит

Comment Object
(
    [data:Comment:private] => Array
        (
            [id] => 1
            [article_id] => 1
            [name] => Евгений
            [email] => [e-mail]
            [body] => Проверка вывода комментариев
            [comm_date] => 2015-01-22 17:19:27
        )

)

Answer 1

[zugo]

У вас ошибка в первом цикле:
$comments = new Comment($row);
должно быть:
$comments[] = new Comment($row);

А про то, что так писать вообще не стоит, тактично умолчу.

Comments

[Евгений Питаев]

может не стоит умалчивать?
Поделитесь опытом

[zugo]

Евгений Питаев: Во-первых, самое главное: нельзя смешивать логику, данные и представление, это очень плохая практика. Почитатайте про паттерны проектирования, в частности MVC. Во-вторых, SQL-запросы в цикле - это кошмар. Пока у вас 1,5 пользователя и база с десятком записей - кажется, что ничего страшного, а на любом реальном проекте это приведет к огромным нагрузкам на базу и тормозам. В третьих, mysql_* - это deprecated функции, их уже нельзя использовать. Используйте PDO или, в крайнем случае, mysqli_*

Answer 2

[Евгений]

как банально, только $comments тоже надо массивом собирать, а в фориче выводить через echo
echo $comment_item->render_comment();

Answer 3

[Макс]

$comments = array();
$comments_query = mysql_query("SELECT * FROM comments ORDER BY id ASC");
 
while ($row = mysql_fetch_assoc($comments_query)) {
    $comments[] = new Comment($row);
}
foreach ($comments as $comment_item) {
    $comment_item->render_comment();
}