[PHP][OOP]Walidacja, Klasa walidujaca, Opcje

[MateuszS]

Witam, zaczynam z OOP i napisałem prostą klasę walidującą dla małych formularzy, napiszcie co można poprawić, miałem ją zaopatrzyć w MVC i prezentować błędy widokiem ale to już na kiedy indziej.

[PHP] pobierz, plaintext 
//wlasciwa klasa w moim nastepnym poscie w tym temacie ;P
[PHP] pobierz, plaintext 

Ten post edytował MateuszS 21.04.2010, 17:23:14

[MateuszS]

To tak:
1. Miałem tragiczny błąd w metodzie, nie pozamieniałem z powrotem $this->zmienna na $zmienna, już to poprawiłem.
2. Dodałem metodę za pomocą której możesz bez przerabiania klasy dodawać nowe typy pól, np. ów nip.
3. Mógłbyś wytłumaczyć o co chodzi z

walidatory powinny być osobnymi obiektami implementującymi jakiś interfejs

[PHP] pobierz, plaintext 
<?
 
class Walidacja 
{
 
	/*	 Dozwolone opcje:
    * 	min_znakow - minimalna ilosc znakow
	* 	max_znakow - maksymalna ilosc znakow
	*	 wzor       - wzor wg ktorego ma byc walidowana zmienna
 
		Wywolanie obiektu
		$ob = new Walidacja;
		$ob->waliduj(zmienna, kryterium);
 
		Mozna swobodnie dodawac nowe pola/typy pol do ustawien, np
		kod pocztowy i zaopatrzyc go w atrybut wzor w ktorym bylby
		wzor w postaci wyrazen regularnych, ktory mialby za zadanie
		sprawdzac zmienna czy ma poprawny format.
	*/
 
	public $ustawienia = array(
		"login" 			=> array(
			"min_znakow"		=> 3,
			"max_znakow" 		=> 25	
		),
		"haslo" 			=> array(	
			"min_znakow"		=> 5,
			"max_znakow"		=> 15		
		),	
		"email"				=> array(
			"wzor"				=> "^[_.0-9a-z-]+@([0-9a-z][0-9a-z-]+.)+[a-z]{2,4}$",
		),
		"zabronione"	=> array()
 
	);
 
	private $bledy 			= array();
 
	public function dodaj($typ, $min = null, $max = null, $wzor = null)
	{
		if(!isset($min) and !isset($max) and !isset($wzor)) {
			throw new Exception("Musisz podac dodatkowy atrybut(maksymalna ilosc znakow, minimalna lub wzor");
		} else {
			$typ = (string)$typ;
			if(isset($min)) $this->ustawienia[$typ]["min_znakow"] = (int)$min;
			if(isset($max)) $this->ustawienia[$typ]["max_znakow"] = (int)$max;
			if(isset($wzor)) $this->ustawienia[$typ]["wzor"] = $wzor;
		}
 
	}
 
	public function zabron($zabronione)
	{
		if(!is_array($zabronione)) {
			array_push($this->ustawienia["zabronione"], $zabronione);
		} else {
			foreach($zabronione as $wartosc) 
				array_push($this->ustawienia["zabronione"], $wartosc);
		}	
	}
 
	public function waliduj($zmienna, $typ = null)
	{
 
		if(is_array($zmienna)) {
			throw new Exception("Weryfikowana zmienna nie moze byc tablica. Bo tak.");
		}
 
		$zmienna = trim(strip_tags($zmienna));
 
		foreach($this->ustawienia["zabronione"] as $zabronione) {
			if(preg_match("#".$zabronione."#", $zmienna)) {
				$this->bledy[] = "Formularz zawiera niedozwolone znaki/slowa: ".$zabronione."."; 
				break;
			}	
		}
 
		if(isset($typ)) {
			if(isset($this->ustawienia[$typ])) {
				$ustawienia = $this->ustawienia;
 
				foreach($this->ustawienia[$typ] as $klucz => $wartosc) {
					if($klucz == "min_znakow") { 
						if(strlen($zmienna) < $this->ustawienia[$typ]["min_znakow"])
					$this->bledy[] = ucfirst($typ)." musi miec conajmniej ".$this->ustawienia[$typ]["min_znakow"]." znakow.";
					}
					if($klucz == "max_znakow"){
						if(strlen($zmienna) > $this->ustawienia[$typ]["max_znakow"])
							$this->bledy[] = ucfirst($typ)." moze miec conajwyzej ".$this->ustawienia[$typ]["max_znakow"]." znakow.";		
					}
					if($klucz == "wzor"){
						if(!preg_match("#".$this->ustawienia[$typ]["wzor"]."#", $zmienna))
							$this->bledy[] = ucfirst($typ)." ma niepoprawny format.";		
					}			
 
				}
			}
		}
 
 
		return $zmienna;
	}
 
	public function pokazbledy()
	{
		return $this->bledy;
	}
}
 
try {
 
 
	$obWaliduj = new Walidacja();
	$obWaliduj->dodaj("tresc", 1, 10);
	$obWaliduj->waliduj("hesdsadsaddads", "tresc");
	$obWaliduj->dodaj("tresc", 4, 7);
	$obWaliduj->waliduj("aa", "tresc");
	$bledy = $obWaliduj->pokazbledy();
		foreach($bledy as $blad) {
			echo $blad."<br />";
		}
 
} catch(Exception $e) {
	echo "Blad: ".$e->getMessage();
}	
[PHP] pobierz, plaintext 

Ten post edytował MateuszS 21.04.2010, 18:36:42