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ę (IMG:style_emoticons/default/smile.gif)

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 (IMG:style_emoticons/default/snitch.gif)

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

[Crozin]

1. Zadanie routera to zamiana ciągu wejściowego (najczęściej adresu URL) na parametry potrzebne do wywołania odpowiedniej akcji w aplikacji. Samo wywołanie akcji nie należy już do jego zadań. Dodatkowo router może robić jeszcze odwrotną operację, tj. na podstawie parametrów akcji generować URL.
2. Twój kod łamie podstawowe zasady OOP (patrz: trzy pierwsze punkty), zasadę pojedynczej odpowiedzialności, IoC czy ukierunkowania na interfejsy. Nieprzestrzeganie tych standardów właściwie niweluje cały pożytek z obiektówki.
3. Do obsługi błędów służą wyjątki, a od ładowania klas są autoloadery.