"или модели пустые, или контроллеры толстые" - это тавтология :)
MVC тебе еще рано, если для тебя получить комментарии под статью - и то проблема. Не занимайся пока, это реально сложно дял тебя будет, все равно накосячишь. Разделяй как у меня написано - хотя бы на уровне отдельных файлов. Этого пока будет достаточно.
ну не совсем.
Сам по себе код нормальный. но showarticle.php тоже должен быть разделен на две части - получение данных и их ввод. Для небольших файлов можно совмещать в одном файле.
Главное чтобы никакой вывод не начинался, пока все данные не будут получены. ВКЛЮЧАЯ шапку сайта
Выше писали что не надо путать логику приложения с логикой отображения. Получение данных это логика приложения. Отображение данных это логика отображения.
Так писать ненормально. Правильно будет так <p><?= $comments->content ?></p>
ну так если проверка if(empty($data)) не нужна, то и возвращать FALSE нет смысла :)
Текст передается в искючении. Полчается, что возвращать вообще ничего не нужно
ThunderCat, это какая-то ерунда. открытие соединения это затратная операция. Не говоря уже о том что случаются параллельные запросы, которые в таком случае тупо увеличат количество одновременных соединений, а "потоки не бесконечны".
Этот ответ неполон. Если предполагается в catch делать то что хочет автор (return false), то за такой код надо отрывать руки и другие выступающие части тела.
а какой смысл несут все эти 'Can not save order. Aborted', 'Can not save address'? Какую пользу из них должен извлечь посетитель сайта? Что он будет делать если увидит одно из этих сообщений?
The Dragger, костылей тут так много, что если их на тебя все поставить, ты будешь на ежика похож.
Лишний НТТР запрос, клиенты которые не поддерживают яваскрипт, не гворя уже об общей бессмысленности такого действия.
Редиректить яваскриптом это все равно что как если бы ты выходил из дома, доходил до школы, потом возвращался домой, брал рюкзак и снова шел в школу, уже окончательно.
Не нужно оправдываться, надо признать что написал ерунду про сессии.
Плюс рекомендую изучить более интеллектуальные способы улучшения производительности, кроме тыканья пальцам в небо.
То есть не сначала мечемся по "типичным местам" и "далее как советовал Yan-s ", а с точностью до наоборот - сначала делаем замеры, а потом исправляем не "типичные места", а только то что на самом деле тормозит. Поверь, такой подход гораздо эффективнее.
ну кстати да, ошибка-то другая совсем. конструктор этой статике не нужен, а, сдается мне, проблема в том что ПДО не может вызвать конструктор юзера, поскольку в него идут два параметра, а ПДО такого режима не имеет. он пишет прямо в проперти класса, а не в параметры конструктора
sir_genry, то есть у тебя не только с логикой, но и с чтением проблемы. Ну ничего, в программистах ты надолго не задержишься, а дворникам умение читать ни к чему
Перед "проверьте сами" должен идти полнофункциональный пример, демонстрирующий данную проблему.
Без такого примера вся история изложенная выше будет восприниматься как занимательная беллетристика, и не больше.
Ссылки для самообразования: idownvotedbecau.se/nomcve https://ru.stackoverflow.com/help/mcve
MVC тебе еще рано, если для тебя получить комментарии под статью - и то проблема. Не занимайся пока, это реально сложно дял тебя будет, все равно накосячишь. Разделяй как у меня написано - хотя бы на уровне отдельных файлов. Этого пока будет достаточно.