Всем привет. Вот сделал типа форум, тут код его. покритикуйте. зы форум это конечно громко сказано, но разделы, темы и сообщения создаются ))
открыл каталог бин, не увидел там исполняемых файлов. закрыл. просмотрел глазами остальные каталоги - где-то класс содержит верхний регистр, где-то не содержит, где-то ундерскор встречается. каша. посмотрел пару файлов. зачем юзать неймспейс инпут для того чтоб потом постоянно относительно него использовать сешн? почему не заюзать сразу класс сешн? и такая хрень много где встречается. зачем тогда вообще юзать какой-то неймспейс? пиши всегда fqcn и не заморачивайся.
про бин, он уже очень давно так называется, я уже сам хз почему. а про неймспейсы, автолоад по другому не работает ))
это каждый раз каунт вызывает заново. Не надо. Добавлено спустя 1 минуту 41 секунду: theme же https://slovari.yandex.ru/theme Добавлено спустя 1 минуту 55 секунд: Код (Text): $temes = $temes_mapper->getCountTemesBySectionId($sections[$j]->getProperty('id')); $temes_count[$j] = $temes->getProperty('count'); $last_temes[$j] = $temes_mapper->getLastTemeBySectonid($sections[$j]->getProperty('id')); $t = $temes_mapper->getTemesBySectionId($sections[$j]->getProperty('id')); if(!is_array($t)) { $t = [$t]; } $messages_count[$j] = 0; for ($i = 0; $i < count($t); $i++) { $messages = $messages_mapper->getCountMessagesByTemeId($t[$i]->getProperty('id')); если тебе так часто нужно $t[$i]->getProperty('id') то не проще ли вынести его в паблик переменную $t[$i]->id просто. Будет и быстрее и удобнее. А то ведь вызов функции в пхп это далеко не бесплатное удовольствие. Даже пустой функции. Добавлено спустя 2 минуты 7 секунд: Итог: ты каждый раз заново пересчитываешь число сообщений =) Добавлено спустя 37 минут 58 секунд: А права у тебя как реализованы? Кто что может делать и не может.
про temes знаю, но когда вспомнил как правильно уже переделывать много было )) $t[$i]->getProperty('id') - ок, можно и вынести. права пока авторизован - не авторизован Код (PHP): if(!auth\auth::isAuth()) { $message = messages_block\messages_block::create_block('app_config', 'messages_block'); $message->set_mess('alert-danger', '5'); render\render::setVar('message', $message->get('messages')); self::callStatic(); die; } с правами то же самое только не isAuth будет, а isAdmin например Добавлено спустя 48 минут 49 секунд: в нижнем регистре старые классы, которые я переделаю обязательно... когда нибудь )) сейчас это не главная проблема
ну да. Там просто переименовать надо через рефакторинг. Он сам прошарится по проекту, и переименует всё.