Witam,
napisałem dla siebie klasę walidacji. Nigdy nie korzystałem z walidacji, tak więc nie wiem czy jest dobrze napisane.
kod się nie zmieścił, więc wrzuciłem na stronę typu no-paste
na nopaste.pl
na wklej.org
Użycie:

[PHP] pobierz, plaintext 
	validation::checkString(array('test' => 'lorem ipsum dolor sit amet'));
 
	validation::setValidations(array('test' => array(
		'maxlenght' 	=> 3,
		'minlenght' 	=> 100,
		'between'		=> array(10,11),
		'type'			=> 'bool',
		'maxtimechar'	=> 'a[spilit]0',
		'required' 		=> true,
		'regex'			=> '/([0-9]+)/'
	)));
 
	echo (validation::isValid('test')) ? 'true' : 'false';
 
	if( count( @validation::$error['test']) > 0) {
 
		echo '<ul>';
 
		foreach( validation::$error['test'] as $error) {
 
			echo '<li>' . $error . '</li>';
		}
 
		echo '</ul>';
	}
[PHP] pobierz, plaintext 

Opcje walidacji

pola z błędami są publicznie dostępne, także można jest dowolnie zmieniać.

Moim zdaniem klasa niepotrzebnie jest statyczna. Dorób metodę która sprawdzi wszystkie elementy po kolei czy są dobre, przydaje się taka funkcjonalność

Jest niepotrzebnie statyczna, w dodatku nie wiem czemu dziedziczysz po exception, te wyjątki robią tam zbędny chaos, ich użycie jest źle przemyślane.

Ja bym zrobił coś na ten wzór:

[PHP] pobierz, plaintext 
<?php
 
	interface i_validation {
		public function isValid($name);
		public function setValidation($options);
	}
 
	class validation {
 
		public function check($name, i_validation $obj) {
			return $obj->isValid($name);
		}
 
	}
 
	class validation_string implements i_validation {
 
		private $status=false;
		private $options;
 
		public function isValid($name) {
			if ($this->status != false) {
				if (preg_match($this->options['regex'], $name) && strlen($name) >= $this->options['minlenght'] && strlen($name) <= $this->options['maxlenght']) {
					return true;
				}else{
					return false;
				}
			}
		}
 
		public function setValidation($options) {
			$keys = array('minlenght', 'maxlenght', 'regex');
			if (count(array_diff($keys, array_keys($options))) == 0) {
				$this->options = $options;
			}else{
				return false;
			}
			$this->status = true;
			return true;
		}
 
	}
 
	$obj = new validation();
 
	$string = new validation_string();
	$string->setValidation(array('regex' => '#^[\w]+$#i', 'minlenght' => 4, 'maxlenght' => 6));
 
	var_dump($obj->check('jakis tekst', $string)); // bool(false)
	var_dump($obj->check('jakisd', $string)); // bool(true)
 
?>
[PHP] pobierz, plaintext 

Dla każdego typu możesz sobie odpowiedni obiekt przekazać w zależności od tego co potrzebujesz sprawdzać.

jest statyczna, bo w moim frameworku chce móc się odwoływać statycznie do klas

Ale tu nie chodzi o to, czy chcesz się tak odwoływać, tylko czy daje Ci to coś konkretnego. Jak ja mam ochotę dziedziczyć klasę do wysyłania e-maili po PDO, to tego mimo wszystko nie robię, bo nie będzie to miało żadnego sensu . Klasy stosuje się w konkretnym celu do uzyskania konkretnego efektu, podobnie jak elementy statyczne i wiele innych rzeczy. Jeśli to ma zostać w takim kształcie, to za bardzo nie widzę potrzeby uzasadniającej istnienie tego kodu. To samo masz we wbudowanym rozszerzeniu filter, tyle że na funkcjach - wprowadzasz wartość, podajesz reguły i dostajesz odpowiedź czy jest dobrze czy źle. Dobra, filter nie generuje komunikatów błędów, ale to nie jest funkcjonalność tłumacząca np. podwójnie zagnieżdżony foreach.

Druga rzecz to błąd merytoryczny:



Poczytaj sobie, co robi funkcja is_integer(), bo na pewno nie to, co myślałeś, pisząc ten kod . Polecam przeanalizować ten krótki skrypt i wynik jego działania:

[PHP] pobierz, plaintext 
<?php
var_dump(is_integer('15'));
[PHP] pobierz, plaintext 

hmm ,tworząc klasę do vaidacji formularza podeszłem do tego inaczej : http://wklej.org/id/302832/ ogólnie spisuję sie ok ale musze znaleść czas aby dopracować ją ,wszelkie uwagi milę widziane

I robisz dokładnie ten sam błąd, co kolega wyżej:



is_float() nie robi tego, co myślałeś, pisząc ten kod, że robi. Ponadto stosujesz jakąś archaiczną składnię obiektową .

W jakim sensie archaiczną ?

Dzięki serdeczne za odpowiedzi

Nie rozumiem co masz na myśli, is_integer() zwraca wartość bool - wiem o tym. Jeżeli chodzi o jakiegoś typu serializowanie tego ifem, to dałem tak, gdyż wraz ze zwrotem false zgłasza błąd

is_integer() na stringu zawsze zwróci bool i zawsze zwróci false:)

gothye -> archaiczną, bo a'la PHP4. Dobry zwyczaj nakazuje, aby nie stosować słowa var na rzecz modyfikatorów dostępu i Tobie by się to przydało o tyle, że masz parę pól prywatnych zadeklarowanych przez to jako publiczne. Dobry zwyczaj nakazuje też umieszczać modyfikatory dostępu przy funkcjach.

PiotrekM -> a sprawdziłeś, JAKĄ wartość logiczną zwraca? Dla Twojej informacji, jeśli będziesz próbował w ten sposób sprawdzać np. $_GET, to tam początkowo wszystkie pola mają typ string (z wyjątkiem tablic). Typów PHP się nie wykorzystuje w sprawdzaniu, ponieważ nie są one miarodajne i tak naprawdę niewiele mówią o tym, co się w danym polu zawiera.

no dlatego do sprawdzania $_GET jest numeric.

Ja u mnie rozwiazalem to tak: http://3paste.com/s/1601 mam do poprawy jeszcze tylko jedna rzecz a dokladniej jak na jedno pole jest kilka warunkow a takich pol jest wiecej zwraca tylko pierwsze zle pola czy jakos tak musze do konca przetestowac.
A tak wyglada akcja komponentu Auth metoda Auth razem z walidacja:

[PHP] pobierz, plaintext 
  public function Auth() { 
 
   if($this -> validation -> Submitted('log')) {
 
         $this -> inputfilter -> Filter();
 
	 $this -> validation -> AddRules('login', 'Login', 'required');
	 $this -> validation -> AddRules('password', 'Haslo', 'required');
	 $this -> validation -> AddRules('login', 'Login', 'AlphaDigit');
	 $this -> validation -> AddRules('password', 'Haslo', 'AlphaDigit');
 
	 $this -> validation -> Validation();
 
	 $errors = $this -> validation -> getValidationErrors();
 
      if(empty($errors)) {
 
        if($this -> AuthModel -> Login($_POST['login'], $_POST['password'])) {
 
	        $this -> Redirect('/index.php');
 
       }
 
	 }
 
     else {
 
		   $this -> view -> AddVar('error', $errors);
 
     }
 
   }  
 
   $user = $this -> auth -> GetUserType();
   return $this -> view -> Layout('Auth', 'component', $user);
 
  }
[PHP] pobierz, plaintext 

PiotrekM -> ech, przeczytaj wszystko jeszcze raz, ale ze zrozumieniem... mówimy właśnie o tym Twoim "numeric", które nie zadziała, bo do sprawdzenia czy pole zawiera liczbę użyłeś funkcji is_int(). Nie rozumiesz w ogóle, jak ta funkcja działa - ona nie sprawdza, czy wartość jest liczbą, tylko czy wartość zmiennej PHP ma aktualnie typ liczbowy, a to dwie różne rzeczy:

[PHP] pobierz, plaintext 
<?php
var_dump(is_integer(15));
var_dump(is_integer('15'));  // TAK WŁAŚNIE WYGLĄDA LICZBA W $_GET!
[PHP] pobierz, plaintext 

Piszemy przecież cały czas, że w tablicach $_GET, $_POST wszystkie wartości są ciągami tekstowymi, więc bez względu na to, jaką wartość będą zawierać, is_int() zwróci Ci dla nich false i możesz pożegnać się np. ze sprawdzeniem czy przekazany do skryptu ID jest poprawny... do takiego sprawdzania się używa wyrażeń regularnych i funkcji z rodziny ctype_xxx().

przepraszam, rozumiem.
można by użyć is_numeric, ale gdy liczba nie jest liczbą całkowitą, to dupa.
zaraz to poprawie

edit:
próbowałem dać settype dla stringu, lecz nawet gdy liczba nie jest liczbą całkowitą, zwraca true - jakieś sugestie?

Ogólnie to marne szanse, php to język w którym nie musisz dbać o typy i jak widać to się mści.