Forum PHP.pl

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ć.

Ten post edytował PiotrekM 23.03.2010, 19:32:36

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ć.

Ten post edytował fifi209 23.03.2010, 20:32:01

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.

Ten post edytował Zyx 24.03.2010, 07:41:27

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?

Ten post edytował PiotrekM 24.03.2010, 17:30:49

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