Walidacja formularzy ., Czyli Ja kontra klasy , pierwsze starcie . Opcje

[nekomata]

Witam , dzisiaj rano postanowiłem zagłębić się troszeczkę w klasy php , wcześniej wiedziałem tylko jak używać gotowych więc trochę poczytałem i spróbowałem własnych sił z klasą która przejrzy za mnie dane z formularza. W sumie jest to bardziej funkcja w formie klasy niż profesjonalna klasa , ale jak pisałem dopiero zacząłem pisanie klas . Dobra przejdźmy do kodu :

[PHP] pobierz, plaintext 
<?php
class form_validator{
	private $form;
	private $boundaries;
	public $errors = array();
	public function load_form($form){
		if(!is_array($form))die('Form to validate is not an array.');
		$this->form = $form;
	}
	public function boundaries($boundaries){
		if(!is_array($boundaries))die('Validator data is not an array.');
		$this->boundaries = $boundaries;
	}
	public function validate(){
		foreach($this->boundaries as $k => $v){
			if(isset($this->form[$k])){
				foreach($v as $key=>$value){
					switch($key){
						case 'length':
							$lengths = (explode('-',$value));
							if(strlen($this->form[$k])<$lengths[0]||strlen($this->form[$k])>$lengths[1]){
								$this->errors[$k][$key]=true;
							}
						break;
						case 'allowed_characters':
							$value = str_replace(array('\\','[',']','^','$','.','|','?','*','+','(',')'),array('\\\\','\[','\]','\^','\$','\.','\|','\?','\*','\+','\(','\)'),$value);
							if(!preg_match('#^['.$value.']{1,}$#',$this->form[$k])){
								$this->errors[$k][$key]=true;
							}
						break;
						case 'type':
							switch($value){
								case 'numbers':
									if(!is_numeric($this->form[$k])){
										$this->errors[$k][$key]=true;
									}
								break;
								case 'e-mail':
									if(!filter_var($this->form[$k],FILTER_VALIDATE_EMAIL)){
										$this->errors[$k][$key]=true;
									}
								break;
								case 'ip':
									if(!filter_var($this->form[$k],FILTER_VALIDATE_IP)){
										$this->errors[$k][$key]=true;
									}
								break;
								case 'ipv4':
									if(!filter_var($this->form[$k],FILTER_VALIDATE_IP,FILTER_FLAG_IPV4)){
											$this->errors[$k][$key]=true;
										}
								break;
								case 'ipv6':
									if(!filter_var($this->form[$k],FILTER_VALIDATE_IP,FILTER_FLAG_IPV6)){
											$this->errors[$k][$key]=true;
										}
								break;
							}
						break;
						case 'regex':
						if($value[0]!==$value[strlen($value)-1]){$value = '#'.$value.'#';}
						if(!preg_match($value,$this->form[$k])){
								$this->errors[$k][$key]=true;
						}
						break;
 
					}
				}
			}
			else{
					$this->errors[$k]['isset']=true;
			}
		}
		if(empty($this->errors)){
			return true;
		}
		else{
			return false;
		}
	}
}
?>
[PHP] pobierz, plaintext 

Do użycia klasy potrzebne będą nam dwie tablice z danymi , pierwsza która otrzymujemy z formularza i druga w której określamy właściwości naszych wartości z formularza . Ja w przykładzie posłużę się tymi dwoma

[PHP] pobierz, plaintext 
$form = array(
	'login'=>'zdzisiek',
	'haslo'=>'zdzisiek1234',
	'email'=>'zdzsiek@email.com',
	'test'=>'lolek');
$boundaries = array(
	'login'=>array(
		'length'=>'3-5',
		'allowed_characters'=>'abcdefghijklmnoprstuwxy'
	),
	'haslo'=>array(
		'length'=>'6-16'
	),
	'email'=>array(
		'length'=>'5-40',
		'type'=>'e-mail'
	),
	'test'=>array(
		'regex'=>'^[abcdes]{1,}$'
	),
	'pusty'=>array(
		'type'=>'ip'
	)
);
[PHP] pobierz, plaintext 

Pierwsza tablica to mój sfabrykowany formularz a druga to właściwości dla niego . Oczywiście dla przykładu wprowadziłem błąd w każdym z nich , teraz przejdę do budowy tablicy boundaries(ograniczenia) , tablica ta składa się z pomniejszych tablic które noszą nazwy komórek z formularza , dla każdej z pomniejszych tablic dostępne są dosyć oczywiste opcje a mianowicie

Kod

lenght - długość wartości , podana jako  min-max czyli np "3-15"
allowed_characters - dozwolone znaki , po prostu wypisujemy znaki po kolei .
type - typ wartości , dostępne wartości to : numbers,e-mail ,ip,ipv4,ipv6
regex - tutaj podajemy wyrażenie regularne

Przykładowe użycie klasy :

[PHP] pobierz, plaintext 
<?php // Po załadowaniu klasy i powyższych ograniczeń :
$validator = new form_validator;
$validator->load_form($form);
$validator->boundaries($boundaries);
if($validator->validate()){
	echo 'WIN ! ';
	//Tutaj dodajemy do bazy czy co tam chcemy , operujemy na normalnych zmiennych
	//Klasa zwraca true tylko w wypadku bezbłędnego formularzu 
}
else{
	ob_start();
foreach($validator->errors as $key=>$value){
	foreach($value as $k=>$v){
		switch($k){
			case 'isset':
				echo 'Zapomniałeś wypełnić '.$key;
			break;
			case 'length':
				echo 'Niepoprawna ilość znaków w polu '.$key.' , zalecane '.$boundaries[$key][$k].' znaków.<br />';
			break;
			case 'allowed_characters':
				echo 'Niedozwolone znaki w polu '.$key.'<br />';
			break;
			case 'type' :
				echo 'Niepoprawna wartość w polu '.$key.'<br />';
				// można zrobić switch na $boundaries[$key][$k] dla róznych typów.
			break;
			case 'regex' :
				echo 'Niewłaściwe dane w komórce '.$key.'<br />';
			break;
		}
	}
}
echo str_replace(array('login','haslo','email','test'),array('nazwa użytkownika',' hasło ',' adres internetowej skrzynki pocztowej ',' komorka testowa'),ob_get_clean());
}
?>
[PHP] pobierz, plaintext 

Jak już pisałem dwa razy jest to moja pierwsza klasa więc :
A ) Krytykujcie
B ) Sugerujcie
C ) Doradzajcie

(Klasa nie jest żadnym arcydziełem , ba pewno znajdziecie masę błedów , ale wolałbym jakbyście nie wrzucali tego na inne fora albo używali gdziekolwiek podając za swoje dzieło ... a poza tym to róbcie z tym co chcecie )

Ten post edytował nekomata 2.06.2011, 15:48:54

[greycoffey]

Tutaj Crozin zamieścił swojego walidatora, popatrz i wyciągnij wnioski ;-)

[nekomata]

Mhmmm , ogólnie to napisałem ten temat właśnie po takie porady , chce napisać klasę która jest najbardziej poprawna ze standardami (jeśli gdzieś by była lista takowych to byłbym wdzięczny za link) .
Obejrzałem sobie tą klasę Crozin'a jedyna rzecz która mi się "nie spodobała" to to że obiekt klasy był wrzucany za każdym razem , moim celem było 'uprościć' formularze a to trochę mija się z celem. Dlatego przepisałem te klasę (użycie nadal jest takie samo) jednak za każdym razem klasa jest wywoływana automatycznie , napiszcie co myślicie o takowym rozwiązaniu i czy takie rozwiązanie jest w ogóle poprawne , Thx.

[PHP] pobierz, plaintext 
<?php
class form_validator{
	private $form;
	private $boundaries;
	public $errors = array();
	public function start(Array $form = array(), Array $boundaries = array()){
		$this->form = $form;
		$this->boundaries = $boundaries;
	}
	public function validate(){
		foreach($this->boundaries as $k => $v){
			if(isset($this->form[$k])){
				foreach($v as $key=>$value){
					$test = new $key($this->form[$k],$value);
					if($test->isValid()){
						$this->errors[$k][$key]=true;
					}
				}
			}
			else{
					$this->errors[$k]['isset']=true;
			}
		}
		if(empty($this->errors)){
			return true;
		}
		else{
			return false;
		}
	}
}
class length{
	private $value;
	private $validators;
	public function __construct($value,$validators){
		$this->value = $value;
		$this->validators = $validators;
	}
	public function isValid(){
		$lengths = (explode('-',$this->validators));
		if(strlen($this->value)<$lengths[0]||strlen($this->value)>$lengths[1]){
			return true;
		}
	}
}
class allowed_characters{
	private $value;
	private $validators;
	public function __construct($value,$validators){
		$this->value = $value;
		$this->validators  = str_replace(array('\\','[',']','^','$','.','|','?','*','+','(',')'),array('\\\\','\[','\]','\^','\$','\.','\|','\?','\*','\+','\(','\)'),$validators);
	}
	public function isValid(){
		if(!preg_match('#^['.$this->validators.']{1,}$#',$this->value)){
			return true;
		}
	}
}
class type{
	private $value;
	private $validators;
	public function __construct($value,$validators){
		$this->value = $value;
		$this->validators = $validators;
	}
	public function isValid(){
		switch($this->validators){
			case 'numbers':
			if(!is_numeric($this->value)){
				return true;
			}
			break;
			case 'e-mail':
			if(!filter_var($this->value,FILTER_VALIDATE_EMAIL)){
				return true;
			}
			break;
			case 'ip':
			if(!filter_var($this->value,FILTER_VALIDATE_IP)){
				return true;
			}
			break;
			case 'ipv4':
				if(!filter_var($this->value,FILTER_VALIDATE_IP,FILTER_FLAG_IPV4)){
				return true;
			}
			break;
			case 'ipv6':
				if(!filter_var($this->value,FILTER_VALIDATE_IP,FILTER_FLAG_IPV6)){
				return true;
			}
			break;
		}
	}
}
class regex{
	private $value;
	private $validators;
	public function __construct($value,$validators){
		$this->value = $value;
		$this->validators = $validators;
	}
	public function isValid(){
		if(!preg_match($this->validators,$this->value)){
			$this->errors[$k][$key]=true;
		}
	}
}
 
			$form = array(
	'login'=>'zdzisiek',
	'haslo'=>'zdzisiek1234',
	'email'=>'zdzsiek@email.com',
	'test'=>'lolek','pusty'=>'not_anymore');
$boundaries = array(
	'login'=>array(
		'length'=>'5-15',
		'allowed_characters'=>'abcdefghijklmnoprstuwxyz'
	),
	'haslo'=>array(
		'length'=>'6-16'
	),
	'email'=>array(
		'length'=>'5-40',
		'type'=>'e-mail'
	),
	'test'=>array(
	'length'=>'5-50'
	),
	'pusty'=>array(
		'length'=>'5-50'
	)
);
 
 
$validator = new form_validator;
$validator->start($form,$boundaries);
if($validator->validate()){
	echo 'WIN ! ';
}
else{
	ob_start();
foreach($validator->errors as $key=>$value){
	foreach($value as $k=>$v){
		//echo 'KEY : '.$key.' .  k : '.$k.' value: '.$v;
		switch($k){
			case 'isset':
				echo 'Zapomniałeś wypełnić '.$key;
			break;
			case 'length':
				echo 'Niepoprawna ilość znaków w polu '.$key.' , zalecane '.$boundaries[$key][$k].' znaków.<br />';
			break;
			case 'allowed_characters':
				echo 'Niedozwolone znaki w polu '.$key.'<br />';
			break;
			case 'type' :
				echo 'Niepoprawna wartość w polu '.$key.'<br />';
				// można zrobić switch na $boundaries[$key][$k] dla róznych typów.
			break;
			case 'regex' :
				echo 'Niewłaściwe dane w komórce '.$key.'<br />';
			break;
		}
	}
}
echo str_replace(array('login','haslo','email','test'),array('nazwa użytkownika',' hasło ',' adres internetowej skrzynki pocztowej ',' komorka testowa'),ob_get_clean());
}
?>
[PHP] pobierz, plaintext 

(zostawiłem przykładowe dane itd do testowania.)

Ten post edytował nekomata 4.06.2011, 07:28:38

[mat-bi]

[PHP] pobierz, plaintext 
if(!is_array($form))die('Form to validate is not an array.');
$this->form = $form;
if(!is_array($boundaries))die('Validator data is not an array.');
[PHP] pobierz, plaintext 

Jak dobrze wiem, klasy powinny wyrzucać wyjątkami, nie die(). Potem, jak wejdę, dokładnie to przejrzę, to mi się tylko rzuciło.

Aha i coś - zrób

[PHP] pobierz, plaintext 
public function start(Array $form = array(), Array $boundaries = array())
[PHP] pobierz, plaintext 

nie będziesz musiał wyrzucać wyjatków, php się tym zajmie

Ten post edytował mat-bi 4.06.2011, 07:07:12

[nekomata]

@mat-bi Dzięki za spostrzeżenia , poprawione i zapamiętane liczę na więcej takich "wytyczeń".

[cer98]

[PHP] pobierz, plaintext 
function foo()
{
     $numargs = func_num_args();
     echo "Number of arguments: $numargs<br />\n";
     if ($numargs >= 2) {
         echo "Second argument is: " . func_get_arg(1) . "<br />\n";
     }
}
 
foo (1, 2, 3);
[PHP] pobierz, plaintext 

p.s. a do obsługi błędów nie mógłbyś użyć specjalnej klasy ?

Ten post edytował cer98 14.08.2011, 12:59:04