PHP.SU

Программирование на PHP, MySQL и другие веб-технологии

PHP.SU Портал

Главная

Помощь

Поиск

Поиск Яндекс Вакансии

Пользователи

Здравствуйте, Гость

( Вход · Регистрация · Правила форума )

Забыли пароль?

Зацените мой роутер

Форумы портала PHP.SU » PHP » Программирование на PHP » Объектно-ориентированное программирование (Модераторы: OrmaJever, Саныч)

Страниц (2): [1] 2 »

Без описания

Поиск в теме | Версия для печати

esterio	Отправлено: 08 Декабря, 2012 - 19:43:44
Активный участник Покинул форум Сообщений всего: 5025 Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов Помог: 127 раз(а)	Наконец то решил выслушать критику. поэтому прошу посмотреть на реализацию моего роутера. Хотелось бы узнать: 1. Чтобы бы вы сделали иначе 2. Какие ошибки 3. Что доработать 4. Удобство в пользовании основной класс neusRouter Спойлер (Отобразить) PHP: скопировать код в буфер обмена class neusRouter { protected $_config = NULL; protected $_route = NULL; protected $_vars = array(); protected $_params = array(); public function __construct() { // } public function route($url) { if($this->_config === null) throw new Exception('config no found'); if($url[0] === 'index' \|\| $url[0] === '') { $this->_route = $this->_config->getConfig(); $this->_vars = array(); $this->_params = $this->_config->getParams(); return array( 'route' => $this->_route, 'vars' => $this->_vars, 'params' => $this->_params ); } $config = $this->_config; $childs = $config->getChilds(); $vars = array(); foreach($url as $u) { $found = false; if(isset($childs[$u])) { $found = true; $config = $childs[$u]; $childs = $config->getChilds(); } else { foreach($childs as $key=>$value) { $preg = $value->getPreg($key); if(preg_match_all($preg['preg'], $u, $matches)) { foreach($preg['vars'] as $key=>$var) $vars[$var] = $matches[$key+1][0]; $config = $value; $childs = $config->getChilds(); $found = true; break; } } } if($found === false) throw new Exception('Not found'); } $this->_route = $config->getConfig(); $this->_vars = $vars; $this->_params = $config->getParams(); return array( 'route' => $this->_route, 'vars' => $this->_vars, 'params' => $this->_params ); } public function getRoutes() { $this->_route; } public function getRoute($key) { if(isset($this->_route[$key])) return $this->_route[$key]; else return false; } public function getVars() { $this->_vars; } public function getVar($key) { if(isset($this->_vars[$key])) return $this->_vars[$key]; else return false; } public function getParams() { $this->_params; } public function getParam($key) { if(isset($this->_params[$key])) return $this->_params[$key]; else return false; } public function setConfig(neusRouterConfigInterface $config) { $this->_config = $config; } public function createConfig($route) { return ($this->_config = new neusRouterConfig($route)); } } Интерфейс для конфигурации neusRouterConfigInterface Спойлер (Отобразить) PHP: скопировать код в буфер обмена interface neusRouterConfigInterface { public function __construct($route, $parrent = NULL, $vars = array()); public function add($url); public function assert($params); public function getConfig(); public function getParams(); public function getChilds($name = NULL); public function getPreg($name); } Класс конфигурации neusRouterConfig Спойлер (Отобразить) PHP: скопировать код в буфер обмена class neusRouterConfig implements neusRouterConfigInterface { protected $_parrent = NULL; protected $_sheme = array(); protected $_route = array(); protected $_vars = array(); protected $_params = array(); protected $_childs = array(); protected $_preg = NULL; public function __construct($route, $parrent = NULL, $vars = array(), $params = array()) { $this->_sheme = array_keys($route); $this->_route = $route; $this->_parrent = $parrent; $this->_vars = $vars; $this->_params = $params; } public function __destruct() { $this->_parrent = NULL; } public function add($url) { $route = array_combine($this->_sheme, array_slice(func_get_args(), 1)); $url = trim($url,'/'); if(strpos($url, '/') === false) { $vars = array(); if((preg_match_all('#\{([\w_-]+)\}#', $url, $matches))) { foreach($matches[1] as $match) $vars[$match] = '.+?'; } $url = strtolower($url); return ($this->_childs[$url] = new self($route, $this, $vars, $this->_params)); } else { $url = explode('/', $url); $config = $this; foreach($url as $u) { if(isset($this->_childs[$u])) $config = $this->_childs[$u]; else $config = $config->add($u, $route); } return $config; } } public function set($name, $value) { $this->_params[$name] = $value; return $this; } public function assert($params) { if(is_array($params) && count($params) > 0) { $parrent_params = array(); foreach($params as $key=>$value) { if(isset($this->_vars[$key])) $this->_vars[$key] = $value; else $parrent_params[$key] = $value; } if($this->_parrent !== null) $this->_parrent->assert($parrent_params); } return $this; } public function getConfig() { return $this->_route; } public function getParams() { return $this->_params; } public function getChilds($name = NULL) { if($name === null) return $this->_childs; elseif(isset($this->_childs[$name])) return $this->_childs[$name]; else return null; } public function getPreg($name) { if($this->_preg === null) $this->_preg ='#'.preg_replace_callback('#\{([\w_-]+)\}#', array($this, 'pregCallback'), $name).'#'; return array( 'preg' => $this->_preg, 'vars' => array_keys($this->_vars) ); } protected function pregCallback($match) { return '('.$this->_vars[$match[1]].')'; } } Пример использования: PHP: скопировать код в буфер обмена $router = new neusRouter(); $neusRoute = $router->createConfig(array('controller' => 'index', 'action' => 'index')); На вход функции подаем массив, который будет взят за основу (index) Далее сама конфигурация: PHP: скопировать код в буфер обмена $neusRoute->add('category_{id}', 'category', 'index')-> add('{m_id}', 'category', 'show')-> assert(array( 'id' => '\d+', 'm_id' => '\d+' )); $neusRoute->add('information', 'information', 'index')-> add('{id}', 'information', 'index')-> assert(array( 'id' => '\d+', )); $neusRoute->add('about_us', 'index', 'about_us'); $neusRoute->add('search', 'search', 'index')->set('access_level', 100); Здесь используется метод add, где 1 параметр - часть URL, а все остальное это элементы массива которые мы задали ранее. Например $ neusRoute-> add ('category_ {id}', 'category', 'index') означает, что: для части URL category_ {id} результатом будет array ('controller' => 'category', 'action' => 'index') В параметре URL можно использовать переменные с помощью "{var}". Метод assert задает правила для переменных в URL, и представляет собой регулярное выражение.Наприклад: assert(array( 'id' => '\d+', 'm_id' => '\d+' )); Указывает, что часть URL category_{id} и {m_id} может содержать переменные только числа. Метод set задает дополнительные параметры. Например set ('access_level', 100); - задает параметры access_level , который равен 100. Теперь например наш URL следующий: http://example.com/category_18/123 Тогда после разбора URL получается массив: PHP: скопировать код в буфер обмена $url = array('category_18', '123'); выполняем роутинг PHP: скопировать код в буфер обмена $routing = $router->route($url); var_dump($routing); Результат выполнения: array 'route' => array 'controller' => string 'category' (length=8) 'action' => string 'show' (length=4) 'vars' => array 'id' => string '18' (length=2) 'm_id' => string '123' (length=3) 'params' => array empty Индексная страница будет следующей: PHP: скопировать код в буфер обмена $url = array('index'); Чуть не забыл преобразованием из http://example.com/category_18/123 в array('category_18', '123') у меня занимается совсем другой класс. (Отредактировано автором: 08 Декабря, 2012 - 19:46:16)

caballero	Отправлено: 08 Декабря, 2012 - 20:28:36
Активный участник Покинул форум Сообщений всего: 5998 Дата рег-ции: Сент. 2011 Откуда: Харьков Помог: 126 раз(а)	ты что у индусов учился? Цитата: преобразованием из http://example[dot]com/category_18/123 в array('category_18', '123') у меня занимается совсем другой класс. это одной строкой можно сделать. ----- Бесплатная система складского учета с открытым кодом https://zippy[dot]com[dot]ua/zstore

esterio	Отправлено: 08 Декабря, 2012 - 20:34:46
Активный участник Покинул форум Сообщений всего: 5025 Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов Помог: 127 раз(а)	caballero пишет: ты что у индусов учился? Цитата: преобразованием из http://example[dot]com/category_18/123 в array('category_18', '123') у меня занимается совсем другой класс. это одной строкой можно сделать. Да там кроме преобразования, есть еще определения браузера, предпочитаемого языка, является ли запрос ПОСТ-ом или АЯКС и т.д. (Добавление) И еще ето сделано ен спроста, У меня УРЛ может быть например http://example.com/ru/category_18. Таким образом я делаю с строкой то чно мне нужно, например - узнаю что язык русский и забираю елмент 'ru'.

caballero	Отправлено: 08 Декабря, 2012 - 20:48:16
Активный участник Покинул форум Сообщений всего: 5998 Дата рег-ции: Сент. 2011 Откуда: Харьков Помог: 126 раз(а)	Цитата: Да там кроме преобразования, есть еще определения браузера, предпочитаемого языка, является ли запрос ПОСТ-ом или АЯКС и т.д. а зачем это роутеру? ----- Бесплатная система складского учета с открытым кодом https://zippy[dot]com[dot]ua/zstore

esterio	Отправлено: 08 Декабря, 2012 - 20:49:50
Активный участник Покинул форум Сообщений всего: 5025 Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов Помог: 127 раз(а)	caballero пишет: а зачем это роутеру? Так и я о том-же, есть отдельной компонент request который етим занимаеться, но если чесно я уже думаю что все-таки стоит добавить преоброзование в роутер. П.С. спасибо за отзыв

caballero	Отправлено: 08 Декабря, 2012 - 20:53:06
Активный участник Покинул форум Сообщений всего: 5998 Дата рег-ции: Сент. 2011 Откуда: Харьков Помог: 126 раз(а)	Цитата: Так и я о том-же, есть отдельной компонент request который етим занимаеться, но если чесно я уже думаю что все-таки стоит добавить преоброзование в роутер. значит ты не о том же. Я о том что это вообще не функции роутера должны быть, а ты еще туда хочешь накидать функционала ----- Бесплатная система складского учета с открытым кодом https://zippy[dot]com[dot]ua/zstore

esterio	Отправлено: 08 Декабря, 2012 - 20:56:28
Активный участник Покинул форум Сообщений всего: 5025 Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов Помог: 127 раз(а)	Значит не правильно обьяснил. Уже есть другой класс который етим занимаеться. А именно Цитата: Да там кроме преобразования, есть еще определения браузера, предпочитаемого языка, является ли запрос ПОСТ-ом или АЯКС и т.д. И я не собираюсь все ето в роутер сувать. Думаю лиш что преобразоване из http://example[dot]com/category_18/123 в array('category_18', '123') стоит все-таки засунуть в роутер и не более.

caballero	Отправлено: 08 Декабря, 2012 - 20:57:14
Активный участник Покинул форум Сообщений всего: 5998 Дата рег-ции: Сент. 2011 Откуда: Харьков Помог: 126 раз(а)	а что тогда делают это стопицот строк кода? ----- Бесплатная система складского учета с открытым кодом https://zippy[dot]com[dot]ua/zstore

esterio	Отправлено: 08 Декабря, 2012 - 21:01:17
Активный участник Покинул форум Сообщений всего: 5025 Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов Помог: 127 раз(а)	Узнают например какой контроллер и метот вызивать для даного УРЛя, и витаскивають переменные например категория с ИД 18.

caballero	Отправлено: 08 Декабря, 2012 - 21:04:53
Активный участник Покинул форум Сообщений всего: 5998 Дата рег-ции: Сент. 2011 Откуда: Харьков Помог: 126 раз(а)	чтобы распознать category_ в урле вынуть id и перенаправить на нужный контроллер я думаю достаточно полдесятка строк ----- Бесплатная система складского учета с открытым кодом https://zippy[dot]com[dot]ua/zstore

esterio	Отправлено: 08 Декабря, 2012 - 21:08:05
Активный участник Покинул форум Сообщений всего: 5025 Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов Помог: 127 раз(а)	Тут столько строк для того чтоб класс был менее зависим, конифиг например можно сделать чтоб тянул с БД или с файла какого-нить. А что Вы предлагаете использовать switch..case

avtor.fox	Отправлено: 08 Декабря, 2012 - 21:13:51
Постоянный участник Покинул форум Сообщений всего: 2083 Дата рег-ции: Март 2012 Откуда: Воронеж Помог: 50 раз(а)	esterio ну и жесть Вы понаписали. Запомните, выучив несколько строк кода из книги и сразу их запостив в свой "проект", лучше от этого не будет. Слишком всё сложно. Там натыкано, здесь накидано, ничего непонятно, а на выходе получаем массив с определением "контроллера" и скармливаемая "гет" строка. Ну и намудрили. Но это Ваше видение. Всё равно ж Вам переписывать, не нам

caballero	Отправлено: 08 Декабря, 2012 - 21:14:24
Активный участник Покинул форум Сообщений всего: 5998 Дата рег-ции: Сент. 2011 Откуда: Харьков Помог: 126 раз(а)	чем писать такое PHP: скопировать код в буфер обмена $neusRoute->add('information', 'information', 'index')-> add('{id}', 'information', 'index')-> assert(array( 'id' => '\d+', )); уж лучше switch..case роутер судя по всему для новостей. что такого сложного там надо роутить чтобы писать такие конструкции. а если там еще картинки блоги товары. как ты себе представляешь чтобы другой разработчик в этом разбирался (Добавление) вообще, роутеры как правило нужны для ЧПУ. адресоватся к контроллеру можно просто задав имя контроллера и экшена в урле, что многое фреймворки и делают ----- Бесплатная система складского учета с открытым кодом https://zippy[dot]com[dot]ua/zstore

esterio	Отправлено: 08 Декабря, 2012 - 22:03:29
Активный участник Покинул форум Сообщений всего: 5025 Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов Помог: 127 раз(а)	avtor.fox пишет: Запомните, выучив несколько строк кода из книги и сразу их запостив в свой "проект" Все лично мой код, никакого копипаста. caballero пишет: роутер судя по всему для новостей. что такого сложного там надо роутить чтобы писать такие конструкции Ето пример, а служет он далеко не для таких целей. caballero пишет: вообще, роутеры как правило нужны для ЧПУ. адресоватся к контроллеру можно просто задав имя контроллера и экшена в урле, что многое фреймворки и делают Он и предназначен для ЧПУ. И являеться частю моего фреймворка(велосипеда). От даной затее отказался в связи з задачами для которих такой подход не подошел бы.

avtor.fox	Отправлено: 09 Декабря, 2012 - 05:44:23
Постоянный участник Покинул форум Сообщений всего: 2083 Дата рег-ции: Март 2012 Откуда: Воронеж Помог: 50 раз(а)	esterio пишет: такой подход не подошел бы Ды я боюсь представить куда бы он подошёл. esterio пишет: Все лично мой код, никакого копипаста. Я не говорил что это копипаст чужого кода. И caballero прав. Роутер нужен в основном для ЧПУ и вызова нужного контроллера. А Вам с таким подходом к задаче нужно идти в 1С работать, там любят таких

Поиск в теме | Версия для печати

Страниц (2): [1] 2 »

Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)

« Объектно-ориентированное программирование »

Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.