Własny validator Opcje

[aras785]

Witam. Zacząłem pisać sobie własny validator danych oto co mam póki co:

[PHP] pobierz, plaintext 
<?php
class Validator {
	public function __construct($data,$validators) {
		foreach($data as $data_key=>$data_value) {
			if(array_key_exists($data_key, $validators)) {
				if(method_exists($this, $validators[$data_key])) {
					if($this->{$validators[$data_key]}($data_value)) {
						return true;
					}else return false;
				}
			}
		}
	}
	public function isInt($var) {
		if(is_int($var)) {
			return true;
		}else return false;
	}
}
 
$dane = array('wiek'=>2);
$validacja = array('wiek'=>'isInt');
 
if(new Validator($dane,$validacja)) echo 'ok';else echo 'Blad';
[PHP] pobierz, plaintext 

Póki co jest prosta zasada ale nie wiem dlaczego w ostatniej linijce gdy sprawdzam if(new Valid...) to zawsze pokazuje 'ok'... Szukam błędu ale niestety nie widzę. Proszę o podpowiedzi. Pozdrawiam

[nospor]

podstawy..... konstruktor zwraca obiekt, a nie żadne RETURN.... a skoro to jest obiekt, to zawsze w IF da ci logiczne TRUE
Musisz napisać oddzielną metodę, do walidacji i ją wywoływać. Konstruktor to konstruktor - nic więcej

[aras785]

Dziękuje mistrzu. Zapamiętam na przyszłość. Teraz chodzi:

[PHP] pobierz, plaintext 
<?php
class Validator {
	public function isValid($data,$validators) {
		foreach($data as $data_key=>$data_value) {
			if(array_key_exists($data_key, $validators)) {
				if(method_exists($this, $validators[$data_key])) {
					if(!$this->{$validators[$data_key]}($data_value)) {
						return false;
					}
				}
			}
		}
	return true;
	}
	public function isInt($var) {
		if(is_int($var)) {
			return true;
		}else return false;
	}
}
 
$dane = array('wiek'=>2,'wiek2'=>'asd');
$validacja = array('wiek'=>'isInt','wiek2'=>'isInt');
 
$valid = new Validator;
if($valid->isValid($dane,$validacja)) echo 'ok';else echo 'blad';
[PHP] pobierz, plaintext 

@nospor możesz mi coś doradzić ws. walidatora? Co może mi się przydać?

Ten post edytował aras785 2.03.2013, 19:26:40

[nospor]

if(is_int($var)) {
To nie ma sensu. Z forma zawsze dostajesz tekst, nawet jeśli ten tekst jest liczbą. zamiast is_int musisz użyć is_numeric

poza tym Twój walidator zwraca jedynie true lub false dla całości. W jego rezultacie nie wiesz, które pole byłe złe a to niedobrze (IMG:style_emoticons/default/smile.gif)

[Crozin]

@nospor możesz mi coś doradzić ws. walidatora? Co może mi się przydać?

Przede wszystkim przydałoby Ci się przeglądniecie istniejących rozwiązań, np. z Zenda czy Symfony. Następnie możesz się nimi pobawić, wypróbować któryś z nich w jakimś projekcie. Gdy uznasz, że z jakiegoś powodu istniejące systemy nie sprawdzają się, będziesz już wiedział co było z nimi nie tak i będziesz miał możliwość ich rozbudowy o brakujące funkcje. Jeżeli jednak uznasz, że istniejące rozwiązania w ogóle nie sprawdzają się będzie to pierwszy argument za tym by w ogóle brać się za pisanie własnej biblioteki.

[aras785]

Macie rację lecz chciałem po prostu nauczyć się czegoś. Zend ma fajne rozwiązanie lecz nie potrafię wydobyć je z samego frameworka.

Szybka modyfikacja (brzydki kod - a wszczegolnosci umiejscowienie komunikatow)

[PHP] pobierz, plaintext 
<?php
class Validator {
	public $errors=array();
	public function isValid($data,$validators) {
		foreach($data as $data_key=>$data_value) {
			if(array_key_exists($data_key, $validators)) {
				if(method_exists($this, $validators[$data_key])) {
					if(!$this->{$validators[$data_key]}($data_value)) {
						$this->errors[$data_key] = $this->komunikat($validators[$data_key]);
					}
				}
			}
		}
	if(count($this->errors)==0) return true;
	return false;
	}
	public function isInt($var) {
		if(is_numeric($var)) {
			return true;
		}else return false;
	}
	public function komunikat($valide) {
	$lang = array(
		'isInt'=>'To musi byc liczba'
		);
	return $lang[$valide];
	}
	public function getErrors() {
		return $this->errors;
	}
}
$dane = array('wiek'=>'asd');
$validacja = array('wiek'=>'isInt');
 
$valid = new Validator;
if($valid->isValid($dane,$validacja)) echo 'ok';else print_r($valid->getErrors());
[PHP] pobierz, plaintext 

Pozdrawiam:)

[Crozin]

Macie rację lecz chciałem po prostu nauczyć się czegoś.

Nie mając minimalnej wiedzy/doświadczenia z danym tematem ciężko jest stworzyć cokolwiek sensownego zaczynając od podstaw. Szczególnie, gdy nawet nie potrafisz jeszcze posługiwać się OOP. Czy się czegoś nauczysz? Pewnie tak: marnowania czasu oraz masy złych nawyków.

Nie odbieraj tego personalnie, ale to co tutaj przedstawiłeś jest pod chyba każdym możliwym względem złe. Na pewnego gorsze od serii IF-ów z echo do wyświetlania błędów.

Zend ma fajne rozwiązanie lecz nie potrafię wydobyć je z samego frameworka.

https://github.com/zendframework/Component_ZendValidator

[aras785]

Wporządku. Dziękuje za wszystkie odpowiedzi. Pozdrawiam (IMG:style_emoticons/default/smile.gif)