Witam,


Chciałby prosić o ocenę oraz konstruktywną krytykę mojej klasy Route. Domyślam się że jest w niej dużo rzeczy do poprawny, udoskonalenia. Dopiero zaczynam na "poważnie" swoją przygode z OOP (w całości w php jestem samoukiem). Także prosze o wytknięci błędów.

Plik Route.php

[PHP] pobierz, plaintext 
/**
 * @autor: Rafał Kowalski
 * @version: 1.0
 * @date release: 2.09.2013r
 */
class Route{
	private $_url;
	private $_directory;
	private $_controller;
	private $_action;
	private $_param;
	public function __construct( $directory )
	{
		$this->_url = $_SERVER['REQUEST_URI'];
		$this->_directory = $directory;
		$this->getParam();
	}
	private function explodeURL( )
	{
		return explode('/',$this->_url);
	}
	private function isAdmin( )
	{
		if( strpos( $this->_url, 'admin' ) )
		{
			return true;
		}
		else
		{
			return false;
		}
	}
	private function getParam( )
	{
		$array_MVC = $this->explodeURL();
		if ( $this->isAdmin( ) )
		{
			$this->_controller	= $array_MVC[2];
			$this->_action		= $array_MVC[3];
			$this->_param		= (int)$array_MVC[4];
 
		}
		else
		{ 
			$this->_controller	= $array_MVC[1];
			$this->_action		= $array_MVC[2];
			$this->_param		= (int)$array_MVC[3];
		}
	}
	public function isController( )
	{
		if( empty( $this->_controller ) )
		{
			if( $this->isAdmin() )
			{
				$this->_controller = 'Login';
			}
			else
			{
				$this->_controller = 'Default';
			}
		}
		else
		{ 
			foreach( $this->_directory as $path )
			{
				if( is_file( $path.ucfirst( $this->_controller ).'_Controller.php' ) )
				{
					return true;
				}
				else
				{
					return false;
				}
			}
		}
 
	}
	public function defaultMethod( )
	{
		if( empty( $this->_action ) )
		{
			$this->_action = 'index()';
		}
	}
	public function isParam( )
	{
		if( empty( $this->_param ) )
		{
			return false;
		}
		else
		{
			return true;
		}
	}
	public function run( )
	{
		try
		{
			if( $this->isController( ) )
			{
				require_once( $path.ucfirst( $this->_controller ).'_Controller.php' );
				$controller = new $this->_controller;
				$action = $this->_action;
				if( $this->isParam() )
				{
					$controller->$action();
				}
				else
				{
					$controller->$action( $this->_param);
				}
			}
			else
			{
				throw new RuntimeException("Błędne wywołanie skryptu");
			}
		}
		catch( RuntimeException $e )
		{
			echo $e->getMessage().'<br />
				Plik: '.$e->getFile().'<br />
				Numer lini: '.$e->getLine().'<br />
				Znak: '.$e->getTraceAsString();
			exit;
		}
	}
}
[PHP] pobierz, plaintext 

Oraz jego użycie index.php

[PHP] pobierz, plaintext 
$directory = array('controller/','controller/admin/','core/');
include 'core/Route.php';
$route = new Route( $directory );
$route->run();
[PHP] pobierz, plaintext 

1) Łamiesz zasadę Single Responsibility Principle
2) Uzależniasz klasę od swojej struktury katalogów (vide admin), poza tym źle używasz funkcji strpos - 0 a false to różnica, poczytaj dokumentację.
3) Poczytaj o PSR-0, PSR-1
4) Czemu łapiesz wyjątki i wyświetlasz błędy? Co jeżeli ktoś chciałby sam błędy obsługiwać?
5) Gdzie testy jednostkowe?

Przeanalizuj i oceń jak takie rzeczy wyglądają w profesjonalnych rozwiązaniach:
http://api.symfony.com/2.0/Symfony/Compone...ing/Router.html
http://framework.zend.com/manual/2.0/en/mo...vc.routing.html