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

[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.

[Wazniak96]

Sorki, że tak trochę nie drążyłem tematu ale byłem trochę zalatany, że tak powiem, mniejsza o to.

1. Da się zrobić.
2. Okej, przeczytałem na ten temat i wiem na czym polegają moje błędy, ale teraz jak przenieść to na tą klasę? Jak abstrakcję, polimorfizm oraz dziedziczenie wykorzystać? Z tego punktu wygląda na to, że lepiej byłoby zamienić ją na zbiór funkcji..
3. Poczytam-zrobię. Tak jak pisałem, tworzę tego frameworka dla celów edukacyjnych, ale nie wszystko na raz.

Ten post edytował Wazniak96 27.06.2013, 12:57:06

[Crozin]

Pisanie w celach edukacyjnych jest jak najbardziej OK, ale trzeba mieć na start jakieś obeznanie w temacie. Wygląda na to, że go nie posiadasz dlatego powinieneś raczej skupić się na wykorzystaniu i zrozumieniu działania istniejących rozwiązań - dużo szybciej i solidniej nauczysz się masy rzeczy na tym etapie. W dodatku w nieporównywalnie krótszym czasie. Tutaj zasugerowałbym wykorzystanie Symfony, bo jako jeden z niewielu PHP-owych "fullstack webframeworków" solidnie realizuje drugi punkt z mojej listy.

Ten post edytował Crozin 27.06.2013, 13:11:12

[Wazniak96]

Szczerze mówiąc bawiłem się już trochę w CodeIgniter i całkiem dobrze się pracowało. Za Symfony wolałbym się na razie nie zabierać bo obawiam się, że go nie ogarnę.

W każdym razie dzięki za pomoc i mądrą radę

[Crozin]

CI to wyjątkowo słaby FW jeżeli chodzi o OOP. SF czy inny "rozbudowany" FW na pewno może sprawić na początku nieco problemów, ale żadne z nich nie są nie do przejścia.

[Wazniak96]

Hmm... Dobrze wiedzieć. Pisząc powyższą klasę wzorowałem się na CI, ale jak widać nie wszylo za dobrze.

Jeszcze takie jedno pytanko. Co byłoby najlepiej spróbować na początek napisać w Symfony?

[Crozin]

Co byłoby najlepiej spróbować na początek napisać w Symfony?

Obojętnie co. Może to być standardowy blog czy inna strona-wizytówka. Chociaż wg mnie najlepiej zrobić sobie klona jakiejś istniejącej "nieszablonowej" strony.

[Wazniak96]

Okej, jeszcze raz wielkie dzięki za pomoc


Ten post edytował Wazniak96 28.06.2013, 00:25:39