[klasa] kontroler żądań Opcje

[dyktek]

Witam, napisałem ostatnio kontroler żądań, na razie wyłapuje dane GET i POST oraz filtruje dowolnie wybrane klucze lub wszystkie klucze. Zalezy mi raczej na tym zeby ktos spojrzał na to okiem fachowca wiec wklejam to do tego działu

[PHP] pobierz, plaintext 
<?php
	/* 
	@ KONTROLER
	@dyktek/hadzia...
	-------------------------------------------------INFO------------------------------------------------------------
	@ $oKon = new kontroler;
	@ $oKon->exclute('pole', 'pole2') - pola, ktore maja byc pominiete przy filtrowaniu danych
	@ $oKon->prepare();
	@ ?index.php?module=artykuly&pole=dyktek&id=23
	@ ### wartosc module ### - okresla modul, ktory bedzie zaladowany
	@ pozostale klucze moga byc dowolne
	--------------------------------------------------------------------------------------------------------------------
	@@@ CHANGE LOG @@@
	- start 27.05.06
	- ukonczona wersja 0.1.9
	*/
 
	class kontroler
	{
		protected $aGet = array();
		protected $aPost = array();
 
		/* tablica z polami wykluczonymi 
		przy sprawdzaniu */
		protected $aExcluteItem = array();
 
		/* tablica z nazwami pol 
		ktore maja zostac sprawdzone
		pod katem nieprawidlowych znakow*/
		protected $aClean = array();
 
		/* tablica zawierajaca pola do ktorych 
		wprowadzono niedozwolone znaki */
		protected $aIncorrectRequest = array();
 
		/* nazwa katalogu z modelami */
		protected $sDir = 'model';
 
		/* wyrazenie z niechcianymi (niedozwolonymi) znakami  */
		protected $regExp = '/('|")/';
 
 
		public function __construct(){
			$this->aGet = $_GET;
			$this->aPost = $_POST;
		}
 
		/* czy klucz istnieje w tablicy */
		private function key_exists($sKey, $aTab){
			if(array_key_exists($sKey, $aTab)) 
				return true; 
			else 
				return false;
		}
 
		/* czy klucz module przenosi prawidlowa nazwe */
		private function valid_module($aTab){
			if($this->key_exists('module', $aTab)){
				if(preg_match('/^(w+)$/', $aTab['module']))
					return true;
				else
					return false;	
			}
		}
 
		/* czy istnieje model */
		private function is_module($aTab){
			$file_dir = ''.$this->sDir.'/'.$aTab['module'].'.php';
			if(file_exists($file_dir))
				return true;
			else
				return false;
		}
 
		/* @ prepare_array($aTab)
		@ usuwa z tablicy ktora jest transportowane zadanie
		@ pola, ktore maja byc pominete przy sprawdzaniu
		@ pola te transportowane sa w tablicy $this->aExcluteItem
		@ w/w tablica jest tworzona przez metode exclute
		*/
		private function prepare_array($aTab){
			foreach($this->aExcluteItem as $sValue){
				foreach($aTab as $key => $value){
					if($sValue != $key)
					$aNewArray[$key] = $value;	
				}
				unset($aTab);
				$aTab = $aNewArray;
				unset($aNewArray);
			}		
			$this->aClean = $aTab;
			$iStart = 0;
			   foreach($this->aClean as $key => $value){
					$aNewArray[$iStart++] = $key;
			   }
			   unset($this->aClean);
			   return $this->aClean = $aNewArray;
		}
 
		/* @ check_values($aArray, $sMethod)
		@ metoda sprawdza elementy tablicy pod katem
		@ niedozwolonych znakow, ktore sa definiowane w $this->regExp
		*/
		private function check_values($aArray, $sMethod){
			$aSource = $this->prepare_array($aArray);
			$iA = 0;
				for($i = 0; $i < count($aSource); $i++){
					if(preg_match($this->regExp, $aArray[$aSource[$i]])){
						$this->aIncorrectRequest[$aSource[$i]] = $aArray[$aSource[$i]];
						$this->aIncorrectRequest[$iA++] = $aArray[$aSource[$i]];	
					}				
				}						
			return $this->aIncorrectRequest;
		}
 
		/* @ exclute()
		@ metoda tworzy tablice z polami
		@ ktore maja zostac pominiete przy 
		@ sprawdzaniu pod katem niedozwolonych zankow
		*/
		public function exclute(){
			$iArgs = func_num_args();
			$this->aExcluteItem = func_get_args();
			return $this->aExcluteItem;
		}
 
		/* @ prepare()
		@ metoda zwraca jako tablice
		@ informacje o wszystkich zadaniach
		@ oraz informacje o zadaniach, ktore
		@ w zawieraja niedozwolone znaki
		*/
		public function prepare(){
			if(count($this->aGet) > 0){
				if($this->valid_module($this->aGet)){
					if($this->is_module($this->aGet))
						return array($this->aGet['module'] ,$this->aGet ,$this->check_values($this->aGet, '$_GET'));
					else
						throw new Exception('nie ma takiego modulu');		
				} else {
					throw new Exception('nieprawidlowa nazwa modulu');
				}		
			} 
			elseif(count($this->aPost) > 0){			
					if($this->valid_module($this->aPost)){
						if($this->is_module($this->aPost))
							return array($this->aPost['module'], $this->aPost ,$this->check_values($this->aPost, '$_POST'));
						else
							throw new Exception('nie ma takiego modulu');		
					} else {
						throw new Exception('nieprawidlowa nazwa modulu');
					}		
			} 
		}	
	}	
 
	$oKon = new kontroler();
 
	$oKon->exclute('module', 'id');
 
	try {
		$dane = $oKon->prepare();
		echo'<pre>';
		print_r($dane);
		echo'</pre>';
	}
	catch(Exception $e) {
		echo $e->getMessage();
	}
?>
[PHP] pobierz, plaintext 

Ten post edytował dyktek 1.06.2006, 14:55:57

[bigZbig]

Dobrze kombinujesz, ale popełniłeś błędy już na etapie założeń teoretycznych. Pomijam zbyt ogólnikową nazwę klasy - w końcu kontrolerów możesz napisać całą masę, ale przydałoby się w nazwie zazanaczyć przynajmniej co to za kontroler. Wracając do błędów w założeniach. Napisałeś coś co jest mixem klasy odpowiedzialnej za odbiór danych od uzytkownika i wstępną ich walidacją zwykle nazywanej Requestem i dyspozytora - Dispatcher - zajmującego się obsługą żadań, wydarzeń lub akcji (zależy jak się to nazwie).

Dyspozytor, który jak rozumiem był Twoim głównym celem powinien otrzymywać jedynie zmienną która zawiera lub może zawierać żadanie. Następnie powinien sprawdzić czy potrafi obsłużyć to żądanie (zdarzenie lub akcję) i czy użytkownik, który dane żądanie przesłał ma do tego prawo. Jeśli dyspozytor nie znajdzie klasy lub metody odpowiedzialnej za obsługę danego zdarzenia lub stwierdzi tę akcję za nieuprawnioną musi jakoś zareagować np. zgłaszając wyjątek lub wykonująć akcję domyślną i dokonując odpowiedniego wpisu w logu. Warto też się zastanowić czy nie zastosować w tym przypadku wzorca singelton.

W zadnym wypadku dyspozytor nie powinien pobierać całej tablicy $_POST i $_GET i szukać w nich zmiennej zawierającej żadanie. Wstępną filtracją danych powinna zajmować się, jak już o tym wspomniałem inna klasa, która oprócz tablic $_POST i $_GET powinna też brać pod uwagę $_COOKIE. Niekiedy do wstępnej walidacji przydają się też informacje z tablicy superglobalnej $_SERVER lub z sesji. Przeczytaj np. Aplikacja php – system wejścia.

Inne uwagi.
Widze, że stosujesz notację węgierską, ale zdarza Ci się o tym zapominać

[PHP] pobierz, plaintext 
<?php
$file_dir = ''.$this->sDir.'/'.$aTab['module'].'.php';
?>
[PHP] pobierz, plaintext 

Poważniejszą przypadłością jest sposób komentowania. Polecam przyjąć jeden ze standardów np. rodem z phpDocumentatora.