Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770
Форумы портала PHP.SU :: Версия для печати :: что то типа CI
Форумы портала PHP.SU » » Объектно-ориентированное программирование » что то типа CI

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

1. and_07 - 25 Сентября, 2012 - 09:25:56 - перейти к сообщению
всем доброго времени суток
в общем не претендую на оригинальность просто вдруг кому будет интересно
данный код взял с какогото урока или книги и чуть переделал
что бы работал так controller/metod/param1/...../pa ramN
PHP:
скопировать код в буфер обмена
  1.  
  2. <?PHP
  3. class Router{
  4.   protected $_controller;
  5.   protected $_action;
  6.   protected $_params;
  7.   static $_instance;
  8.  
  9.   public static function getInstance(){
  10.         if(!(self::$_instance instanceOf self ))
  11.                 self::$_instance = new self();
  12.         return self::$_instance;
  13.   }
  14.  
  15.   private function __construct(){
  16.                 $request = $_SERVER['REQUEST_URI'];
  17.                 $splits = explode('/',trim($request,'/'));
  18.       //conrtoller
  19.                 //ucfirst($splits[1])
  20.                 $this->_controller = !empty($splits[0])? $splits[0].'Controller':'indexController';
  21.                
  22.       //method 
  23.                 $this->_action = !empty($splits[1])?$splits[1].'Action':'indexAction';
  24.                  
  25.       //params
  26.                 if (!empty($splits[2])){
  27.                         $value = array();
  28.                         for($i=2,$cnt=count($splits);$i<$cnt;$i++)
  29.                                 $value[] = $splits[$i];
  30.                         $this->_params = $value;                     
  31.                 }    
  32.   }
  33.        
  34.   public function route(){
  35.         if(class_exists($this->getController())){
  36.                 $rc = new ReflectionClass($this->getController());
  37.                 $rm = new ReflectionMethod($this->getController(), $this->getAction());
  38.                 $class =  get_parent_class($this);
  39.                 var_dump($rc->name); //просто так
  40.                         if(get_parent_class($rc->name) == 'BaseController'){
  41.                                 if($rc->hasMethod($this->getAction())){
  42.                                 $controller = $rc->newInstance();
  43.                                 $method = $rc->getMethod($this->getAction());
  44.                                  /***********************************
  45.                                  if($method->isStatic())
  46.                                     $method->invoke(null);     
  47.                                  else
  48.                                 // *********************************/
  49.               $par = $rm->getParameters();
  50.               if($par && $this->getParams()!=0){  
  51.                  $rm->invokeArgs($controller, $this->getParams());
  52.                }//else { throw new Exception("Params"); }  */                              
  53.                                     $method->invoke($controller);                              
  54.                                 } else { throw new Exception("Action"); }
  55.                         } else { throw new Exception("Interface");}
  56.         } else { throw new Exception("Controller");}
  57.   }
  58.  
  59.   function getController(){
  60.         return $this->_controller;
  61.   }
  62.  
  63.   function getAction(){
  64.         return $this->_action;
  65.   }
  66.  
  67.   function getParams(){
  68.         return $this->_params;
  69.   }
  70. }
  71. ?>
  72.  


index.php
PHP:
скопировать код в буфер обмена
  1.  
  2. <?PHP
  3.  
  4.  require 'inc/init.php';
  5.  
  6. $front = Router::getInstance();
  7. $front->route();
  8. ?>
  9.  


на недочеты прошу указывать
2. DeepVarvar - 25 Сентября, 2012 - 15:11:32 - перейти к сообщению
and_07 пишет:
$front = Router::getInstance();

0) Не люблю инстации, они плодят рекурсивные ссылки на объекты, лучше сделать абстрактный класс роутера и обращаться к нему откуда угодно: Router::<method>();
1) getParams() возвращает сразу все параметры, которые придется потом перебирать либо в экшне контроллера, либо передавать модели, которая так же будет вынуждена уметь их перебирать, так теряется инкапсуляция.
2) Нет поддержки модулей, стоящих выше контроллеров, которых в одном модуле может быть несколько.
3) В синглтоне, если уж на то пошло, нужно еще повесить заглушку __clone, иначе всетаки сделать еще одну копию объекта получится )))
3. DelphinPRO - 25 Сентября, 2012 - 15:57:20 - перейти к сообщению
DeepVarvar пишет:
В синглтоне, если уж на то пошло, нужно еще повесить заглушку __clone

и на __wakeup
Улыбка

 

Powered by ExBB FM 1.0 RC1