Зачем strtolower?
Написал: Артур
20.11.2014 / 09:20
Новые комментарии
Простой роутер (33)
Простой роутер (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)
В предыдущем варианте $application_name и $action_name задавались дефолтные значения но если передавались другие то заменялись новыми т.е. если передали "domaine.ru/book" то $application_name с "main" заменялся на "book" а $action_name так и оставался "index". Поэтому эти переменные нельзя было вписывать в пути приложения по умолчанию и получалось их значения нужно было прописывать в двух местах.
Написал: Александр 20.11.2014 / 02:12
Написал: Александр 20.11.2014 / 02:12
Простой роутер (33)
iNeeXT, тогда уже думаю лучше немного переписать. Задавать дефолтные значения приложения и экшена если что то из них не было передано то присваивать дефолтное. И так же их использовать в пути если заданное приложения не существует. Короче изменил код. Если есть еще предложения/замечания буду рад выслушать.
Написал: Александр 20.11.2014 / 02:06
Написал: Александр 20.11.2014 / 02:06
Простой роутер (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
Из-за этого минус? Пишите свои статьи если такие грамотные. .
Написал: Dmitry Kokorin 19.11.2014 / 20:34
Написал: Dmitry Kokorin 19.11.2014 / 20:34
Простой роутер (33)
Простой роутер (33)
Определяешь переменные $application_name и $action_name, но не используешь их. Исправь.
Написал: iNeeXT 19.11.2014 / 18:18
Написал: iNeeXT 19.11.2014 / 18:18
Много лишнего в регулярке, к примеру цифры 0-9 можно сократить так \d
А буквы \w
Написал: Антон 18.11.2014 / 23:07
А буквы \w
Написал: Антон 18.11.2014 / 23:07