[Klasa] Router, Moje pierwsze praktyczne kroki w OOP i prośba o ocenę :) Opcje

[mathev19]

Na wstępie chciałem się ze wszystkimi przywitać, ponieważ jest to mój pierwszy post.
Dobrze przejdźmy więc do rzeczy napisałem sobie klasę Routera, której kod możecie zobaczyć poniżej. Przede wszystkim proszę o obiektywną krytykę bo jest to moja pierwsza klasa. (IMG:style_emoticons/default/snitch.gif)

[PHP] pobierz, plaintext 
<?php
 
class Router {
	private $sPath = null;
	private $sController = null;
	private $sAction = null;
	private $aUrlSegments = array();
	private $aParams = array();
 
	public function __construct() {
		$this->sPath = $_SERVER['PATH_INFO'];
		$this->setUrlSegments();
		$this->setController();
		$this->setAction();
		$this->setParams();
	}
 
	/**
	 * pobiera nazwę serwera
	 * @return string
	 */
	public function getServerName() {
		return $_SERVER['HTTP_HOST'];
	}
 
	/**
	 * pobiera ścieżkę i dzieli ją na segmenty
	 * następnie umieszcza wszystko w tablicy aUrlSegments
	 */
	private function setUrlSegments() {
		$this->aUrlSegments = explode('/', $this->sPath);
	}
 
	/**
	 * ustawia nazwę controllera
	 */
	private function setController() {
		 $this->sController = $this->getUrlSegment(1);
	}
 
	/**
	 * pobiera nazwę controllera
	 */
	public function getController() {
		 return $this->sController;
	}
 
	/**
	 * ustawia nazwę akcji
	 */
	private function setAction() {
		$this->sAction = $this->getUrlSegment(2);
	}
 
	/**
	 * pobiera nazwę akcji
	 */
	public function getAction() {
		return $this->sAction;
	}
 
	/**
	 * pobiera segment URL'a o wyznaczonym id
	 * @param int $id
	 * @return string
	 */
	public function getUrlSegment($id) {
		return (isset($this->aUrlSegments[$id])) ? $this->aUrlSegments[$id] : false;
	}
 
	/**
	 * ustawia parametry z URL'a
	 */
	private function setParams(){
		$this->aParams = array_slice($this->aUrlSegments,3);
	}
 
	/**
	 * pobiera parametry z URL'a
	 */
	public function getParams(){
		return $this->aParams;
	}
 
	/**
	 * tworzy adres URL
	 * @param string $sController
	 * @param string $sAction
	 * @param array $aParams
	 * @return string
	 */
	public function createUrl($sController,$sAction,$aParams) {
		return $this->getServerName().'/index.php/'.$sController.'/'.$sAction.'/'.$aParams[0].'/'.$aParams[1].'/'.$aParams[2];
	}
}
 
?>
[PHP] pobierz, plaintext 

Teraz moje pytania dotyczące ów klasy:
1. Czy ogólny sposób napisania Routera jest zgodny z filozofią OOP?
2. Co mogę napisać inaczej (czyt. lepiej)?
3. W jaki sposób prawidłowo tworzyć URL'e (metoda createUrl, na którą jak widać w kodzie nie mam zupełnie pomysłu) (IMG:style_emoticons/default/sadsmiley02.gif)

[Crozin]

singleton nie,rejestr nie, jak to rozwiążesz? wszystko przez parametr ciągle?

Przekazywanie jako argument jest bardzo dobrym sposobem. Jednak z czasem może nie wystarczyć - wtedy warto skorzystać np. z Dependency Injection czy Service Locator.

Jest to tzw. notacja węgierska

Ahhh... dzięki za przypomnienie. (IMG:style_emoticons/default/winksmiley.jpg)

Mógłbyś rozwinąć twoją myśl? Bo nie za bardzo "łapię".

W skrócie: jeżeli dana metoda ma być niedostępna z zewnątrz i nie masz jakiegoś konkretnego powodu by blokować jej widoczność w klasach potomnych - powinna być prywatna. Co do tego, że te metody set*() wepchałeś na siłę:

[PHP] pobierz, plaintext 
public function __construct() {
    $this->aUrlSegments = explode('/', $this->sPath);
    $this->sController =$this->aUrlSegments[1];
    $this->sAction = $this->aUrlSegments[2];
}
 
// tutaj same gettery
[PHP] pobierz, plaintext 

Nie wykonujesz tam żadnych skomplikowanych operacji - dlatego możesz bez problemu wrzucić całość do kontrolera.

Jeżeli chodzi o Kontroler to powiedz mi jakiego typu informację może on wysyłać do Modelu?

Może przekazać mu takie dane do jakich widok nie ma dostępu lub nie leżą one do zakresu działania logiki widoku.

Kontroler ma być pośrednikiem informacji pomiędzy Modelem a Widokiem?

W sieci znajdziesz w zdecydowanej większości przypadków (dla PHP) kontrolera działającego właśnie jako pośrednik (kontroler pobiera dane z modelu i przekazuje je do widoku). Natomiast poprawną implementacją MVC jest zrobienie z kontrolera takiej "swatki". Określa on, żę ten a ten widok ma użyć tego, a tego modelu.