Klasa do walidacji formularza - opinia Opcje

[moto0095]

Witam. Napisałem ostatnio klasę do walidacji formularza.
I prosiłbym o opinię oraz porady np: co trzeba/można zmienić.

[PHP] pobierz, plaintext 
<? 
 class FormValidator {
 	private $error = array();
  private $elem = array();
 
	public function validate($name, $error = '', $filter = array()) {
		$this->error = $error;
		foreach($filter as $key => $value) {
		  $this->filter($name, $key, $value);
		}	
	}
 
	private function filter($name, $filter, $par = FALSE) {
	  switch($filter) {
		  case 'req' : 
			  if(!isset($name) || empty($name) || $par == TRUE) {
				  $this->set_error('Pole jest wymagane i nie może pozostać puste.');
				}
			break;
 
		  case 'email' :
			  if(!filter_var($name, FILTER_VALIDATE_EMAIL) || $par == TRUE) {
				  $this->set_error('Podany adres e-mail jest niepoprawny');  
				}
			break;
 
			case 'url' :
			  if(!filter_var($name, FILTER_VALIDATE_URL) || $par == TRUE) {
				  $this->set_error('Podany adres URL jest niepoprawny');
				}
	    break;
 
		  case 'num' :
			  if(!preg_match('/^[0-9 ]+$/', $name) || $par == TRUE) {
				  $this->set_error('Ple musi zawierać wyłącznie znaki numeryczne');  
				}
			break;			
 
		  case 'alfa' :
			  if(!preg_match('/^[a-zA-Z ]+$/', $name) || $par == TRUE) {
				  $this->set_error('Ple musi zawierać wyłącznie litery');  
				}
			break;
 
		  case 'alnum' :
			  if(!preg_match('/^[a-zA-Z0-9 ]+$/', $name) || $par == TRUE) {
				  $this->set_error('Ple musi zawierać wyłącznie znaki alfanumeryczne');  
				}
			break;			
 
		  case 'max' :
			  if(strlen(trim($name)) > $par) {
				  $this->set_error('Pole musi mieć maxymalną długość '.$par.'.');
				}
			break;			
 
		  case 'min' :
			  if(strlen(trim($name)) < $par) {
				  $this->set_error('Pole musi mieć minimalną długość '.$par.'.');  
				}
			break;			
 
		  case 'length' :
			  if(strlen(trim($name)) == $par) {
				  $this->set_error('Pole musi mieć mieć długość '.$par.'.');  
				}
			break;	
 
		/* ORAZ INNE FILTRY */		
		}
	}
 
	private function set_error($str) {
	  $this->errors[] = (empty($this->error) ? $str : '<b>'.$this->error.'</b> : '.$str);
	}
 
  public function get_error($str) {
	  return $this->errors[$str];
	}	
 
  public function get_errors() {
	  foreach($this->errors as $err) { 
		  echo $err.'<br />';
		}	
	}
 
	public function is_valid() {
	   return (count($this->errors) > 0) ? FALSE : TRUE;
	}
 
 }
?>
[PHP] pobierz, plaintext 

Przykład użycia:

[PHP] pobierz, plaintext 
include_once('class.form_validator.php');
 
$val = new FormValidator;
 
$val->validate($_POST['email'], 'Email', array('req' => TRUE, 'min' => 10, 'email' => TRUE));
 
if($val->is_valid()) {
  echo 'OK';
}else {
  $val->get_errors();
}
[PHP] pobierz, plaintext 

Ten post edytował moto0095 26.12.2010, 23:13:51

[ShadowD]

Klasa nie powinna mieć na sztywno odpowiedzi - to na pewno, może jakieś kody błędów?

Ponadto nie podoba mi się zwrot - 'email' => TRUE
Szczerze pomyślał bym nad możliwością wstawienia metody, mail, nr itd niż przesyłania w array - ale to moje widzimisie i nie wszystkim pewnie by odpowiadało. :-)

[moto0095]

Czyli lepiej by było :

[PHP] pobierz, plaintext 
include_once('class.form_validator.php');
 
$val = new FormValidator;
 
$val->validate($_POST['email'], 'Pole email jest wymagane i nie może pozostać puste', 'req');
$val->validate($_POST['email'], 'Pole email musi mieć minimum 10 znaków długości', 'min', 10);
$val->validate($_POST['email'], 'Pole email zawiera błędny email', 'email');
 
if($val->is_valid()) {
  echo 'OK';
}else {
  $val->get_errors();
}
[PHP] pobierz, plaintext 

(IMG:style_emoticons/default/questionmark.gif)

Ok. Poprawiłem trochę klasę. Teraz wygląda tak:
(Proszę o opinię i porady)

[PHP] pobierz, plaintext 
<?
 class FormValidator {
   private $errors = array();
	 private $field;
	 private $label;
 
	 ### FILTRY ###
 
	 public function email($message = '') {
     $message = (empty($message)) ? 'Pole %s nie zawiera adresu email.' : $message;
     if(!filter_var($this->field, FILTER_VALIDATE_EMAIL)) {
		   $this->register_error($message);
		 }
		 return $this;
	 }
 
   public function required($message = '') {
	   $message = (empty($message)) ? 'Pole %s jest wymagane i nie może pozostać puste.' : $message;
     if(!isset($this->field) || empty($this->field)) {
		   $this->register_error($message);
		 } 
     return $this;
   }
 
	 public function numeric($message = '') {
     $message = (empty($message)) ? 'Pole %s może zawierać wyłącznie znaki numeryczne.' : $message;
     if(!preg_match('/^[0-9 ]+$/', $this->field)) {
		   $this->register_error($message);
		 }
		 return $this;
   }
 
	 public function minlength($minlen, $message = '') {
     $message = (empty($message)) ? 'Pole %s może mieć minimalną długość '.$minlen.'.' : $message;
     if(strlen(trim($this->field)) > $minlen) {
		   $this->register_error($message);
		 }
     return $this;
   }
 
   public function maxlength($maxlen, $message = '') {
     $message = (empty($message)) ? 'Pole %s może mieć maxymalną długość '.$maxlen.'.' : $message;
		 if(strlen(trim($this->field)) < $maxlen) {
		   $this->register_error($message);
		 }
		 return $this;
	 }
 
   public function length($len, $message = '') {
     $message = (empty($message)) ? 'Pole %s musi mieć wartość '.$len.'.' : $message;
     if(strlen(trim($string)) !== $len) {
		   $this->register_error($message);
		 }
     return $this;
	 }
 
	 public function matches($field, $label, $message='') {
	   $message = (empty($message)) ? 'Pole %s nie zgadza się z polem '.$label.'.' : $message; 
     if((string)$field !== (string)$this->field ) {
		     $this->register_error($message);
		 }
		 return $this;
   }
 
	 ### /FILTRY ###
 
	 public function validate($key, $label = '') {
	   $this->field = $key;
		 $this->label = $label;
		 return $this;
	 }
 
	 private function register_error($message) {
	   $message = (empty($message) ? '%s zawiera błędy.' : $message);
		 $label = (empty($this->label) ? 'Formularz' : $this->label);
		 $this->errors[] = sprintf($message, $label);
   }
 
	 public function get_error($key) {
	 	 echo $this->errors[(int)$key];
	 }
 
	 public function get_all_errors() {
     foreach($this->errors as $err) {
		   echo $err.'<br />';
		 }
	 }
 
	 public function is_valid() {
        return (count($this->errors) < 0) ? TRUE : FALSE;
	 }
 
 }
?>
[PHP] pobierz, plaintext 

Przykład użycia:

[PHP] pobierz, plaintext 
$f = new FormValidator;
 
$f->validate($_POST['email'], 'Email')->email('ERROR')->maxlength(1)->required()->numeric();
 
 
if($f->is_valid()) {
  echo 'OK';
}else {
  $f->get_all_errors();
}
[PHP] pobierz, plaintext 

Ten post edytował moto0095 27.12.2010, 23:12:51