Ocena klasy router Opcje

[Wazniak96]

Witam,

od pewnego czasu kojarzę, że tak powiem OOP(znam mniej więcej zasady). Ostatnio postanowiłem napisać coś od podstaw, wybór padł na prostego frameworka.
I tutaj pojawia się ten problem, że tworzę go raczej w formie edukacyjnych i potrzebuję opinii kogoś znającego się na rzeczy, czy idę w dobrą stronę

Poniżej podaję kod klasy router, mającej zająć się otworzeniem odpowiedniej klasy oraz wywołania podanej metody.

[PHP] pobierz, plaintext 
<?php
 
class Router{
 
	private $cotroller;
	private $method = 'index';
	private $directory;
 
	public function __construct()
	{
		$this->directory = APPPATH . '/controllers/';
		require(APPPATH.'/config/config.php');
		$this->setController($config['base_controller']);
		$this->readData();
		$this->run();
	}
 
	public function setController($controller)
	{
		$this->controller = ucfirst(str_replace(array('/', '.'), '', $controller));
	}
 
	public function setMethod($method)
	{
		$this->method = str_replace(array('/', '.'), '', $method);
	}
 
	public function readData()
	{
		$uri = explode('/', $_GET['uri']);
		if(isset($uri[0]) && !empty($uri[0]))
			$this->setController($uri[0]);
 
		if(isset($uri[1]) && !empty($uri[1]))
			$this->setMethod($uri[1]);
 
		for($i=2; $i<12; $i++)
		{
			if(isset($uri[$i]) && !empty($uri[$i]))
				$data[] = $uri[$i];
		}
 
		return $data;
	}
 
	private function run()
	{
		if(!file_exists($this->directory . strtolower($this->controller) . '.php'))
			exit('Środowisko aplikacji nie zostało zainstalowane poprawnie.');
		else
		{
			require($this->directory . strtolower($this->controller) . '.php');
 
			$method = $this->method;
			$class = new $this->controller;
			$class->$method();
		}
	}
}
[PHP] pobierz, plaintext 

Wszelka krytyka mile widziana

PS: Brakuje jeszcze sprawdzenia czy podana metoda istniej, a w innym wypadku przekierowanie, ale postanowiłem zrobić to trochę później, kiedy ogólnie zajmę się błędami.

Ten post edytował Wazniak96 21.06.2013, 20:22:41