Замечания:
1. Использовать global в фукнциях - плохой тон.
2. Фильтрировать $_SESSION не нужно, так как вы сами его заполняете в коде.
Да и пора уже знакомиться с такими понятиями как SQL Builder и ORM.
Написал: iNeeXT (14.08.2015 / 11:29)
Список комментариев iNeeXT
Работа с MySQLi (11)
Простой роутер (33)
Башка (Сегодня / 10:53)Я тоже имею привычку использовать предопределённую константу для root-категории. Все из-за того, что не у всех стоит актуальная версия PHP, и бывают случаи, когда нужно её менять на dirname(__FILE__) или использовать $_SERVER['DOCUMENT_ROOT'].
klesh, а чем плоха константа __DIR__?
Написал: iNeeXT (20.11.2014 / 12:57)
Простой роутер (33)
Простой роутер (33)
Я говорю о том, что нужно переводить в нижний регистр это:
$application_name . '/' . $action_name
а не отдельно, ведь зачем пихать в функцию перевода __DIR__ ?
Написал: iNeeXT (20.11.2014 / 12:36)
$application_name . '/' . $action_name
а не отдельно, ведь зачем пихать в функцию перевода __DIR__ ?
Написал: iNeeXT (20.11.2014 / 12:36)
Простой роутер (33)
Башка, чтобы при запросе /Blog/SomeAction он искал файл app/blog/someaction.php.
Написал: iNeeXT (20.11.2014 / 12:18)
Написал: iNeeXT (20.11.2014 / 12:18)
Простой роутер (33)
Думаю, так будет лучше:
В переменной $application_path функцию strtolower нужно применять для $application_name и $action_name.
Написал: iNeeXT (20.11.2014 / 04:40)
// Получаем имя приложения $application_name = !empty($routes[1]) ? $routes[1] : $application_name_default; // Получаем имя экшена $action_name = !empty($routes[2]) ? $routes[2] : $action_name_default;
В переменной $application_path функцию strtolower нужно применять для $application_name и $action_name.
Написал: iNeeXT (20.11.2014 / 04:40)
Простой роутер (33)
klesh, здесь нужно изменить:
Написал: iNeeXT (19.11.2014 / 23:42)
else { // Если неверное приложение то на главную $application_path = strtolower(CORE_DIRECTORY . '/application/main/index.php'); require $application_path; }
Написал: iNeeXT (19.11.2014 / 23:42)
Простой роутер (33)
Определяешь переменные $application_name и $action_name, но не используешь их. Исправь.
Написал: iNeeXT (19.11.2014 / 18:18)
Написал: iNeeXT (19.11.2014 / 18:18)
Мне тоже не нравятся стандартные шрифты в Linux. Нужно не только функциональная машина, но и удобная.
Написал: iNeeXT (11.09.2014 / 01:11)
Написал: iNeeXT (11.09.2014 / 01:11)