Validująca klasa Opcje

[szmerak]

[PHP] pobierz, plaintext 
<?php
class Validate {
 
	private $text = null;
 
	private $methods = array();
 
	private $result = null;
 
	public function do_validate($text, $methods)
	{
		if(sizeof($methods) < 1)
		{ return false;	}
		$this->methods = (array) $methods;
		$this->text = $text;
		if(array_key_exists("MAX_LENGHT", $this->methods) and array_key_exists("MIN_LENGHT", $this->methods))
		{
			$max_lenght = intval($this->methods["MAX_LENGHT"][0]);
			$min_lenght = intval($this->methods["MIN_LENGHT"][0]);
			if($max_lenght <= $min_lenght)
			{
				exit("You have screwed up something with setting. Check them again");
			}
		}
		foreach($this->methods as $k => $v)
		{
			switch(strtolower($k))
			{
				case 'empty':
					if(empty($text) or $text == "" or !isset($text))
					{
						$this->result[] = $v;
					}
				break;
				case 'max_lenght':
					if(is_array($v))
					{
						if(is_numeric($v[0]))
						{
							$param["lenght"] = intval($v[0]);
						}else{
							exit("First param of array must be numeric");
						}
						$param["alert"] = $v[1];
					} else {
						exit("Second param must be an array");
					}
					if(strlen($this->text) > $param["lenght"])
					{
						$this->result[] = $param["alert"];
					}
				break;
				case 'min_lenght':
					if(is_array($v))
					{
						if(is_numeric($v[0]))
						{
							$param["lenght"] = intval($v[0]);
						}else{
							exit("First param of array must be numeric");
						}
						$param["alert"] = $v[1];
					} else {
						exit("Second param must be an array");
					}
					if(strlen($this->text) < $param["lenght"])
					{
						$this->result[] = $param["alert"];
					}
				break;
			}
		}
	}
	public function get_results()
	{
		return $this->result;
	}
 
}
?>
[PHP] pobierz, plaintext 

Przedstawiam wam moją walidującą klase i chciał bym ją użyć do mojego przyszłego CMS.
z czasem dodam validating emaila itp... ale jak narazie tyle zostanie co powiecie na jej temat?
co byscie zmienili dodali itp?
Sposob uzycia

[PHP] pobierz, plaintext 
include 'class.validate.php';
$val = new Validate;
$validate = array(
'EMPTY' => "field can not be empty",
'MAX_LENGHT' => array('30', "This string is too big"),
'MIN_LENGHT' => array('29', "This string is too short"),
);
$val->do_validate("JAKIS TEKST NIE SPELNIAJACY WARUNKU", $validate);
var_dump($val->get_results());
[PHP] pobierz, plaintext 

Ten post edytował Daiquiri 5.11.2011, 09:52:10

Powód edycji: [Daiquiri]: Przenoszę do odpowiedniego działu.

[cudny]

Nie chce mi się całości patrzeć, wybacz
Rażące uwagi:
1. Brak obiektowości - po co ci klasa z jedną metodą robiącą "wszystko" ? Przez to już więcej jej nie wykorzystasz. Powinny być metody powiedzmy takie jak $validate->between($min,$max,$res); która po daniu argumentów: $min = 3, $max = 6, $res = ? zwróci true jeśli $res zawiera się pomiędzy $min i $max czyli pomiędzy 3 a 6 i metody takie wykorzystywane dalej.
32 if(empty($text) or $text == "" or !isset($text)) - jeśli na serwerze masz włączone error_reporting to wywali ci wyjątek, że $text is not exist poza tym po co sprawdzać czy coś jest puste i lub "" ? KOLEJNOŚĆ WARUNKÓW SIĘ LICZY ! powinno być tak: if(!isset($text) || empty($text)) return false; wtedy nawet jeśli $text nie istnieje to warunek już nie przejdzie dalej i nie sprawdzi czy $text is empty - także nie wywali ci wyjątku

Idę spać nie jestem od sprawdzania poprawności kodu musisz poczytać o obiektowości (najlepiej naucz się frameworka, polecam Zend'a, one wymuszają obiektowość) i popracuj na serwerze, który wywala każdy błąd - będziesz wiedział jak wyłapywać wyjątki i kiedy warunki się spełniają
Kliknij pomógł, dobranoc


Jezu...
A po co ci sprawdzenie po sobie czy coś istnieje ?
if(array_key_exists("MAX_LENGHT", $this->methods) and array_key_exists("MIN_LENGHT", $this->methods))

Ogólnie $methods moim zdaniem nie powinna być array'em
Użyj pól:

[PHP] pobierz, plaintext 
protected $_min_len = 0;
protected $_max_len = 0;
[PHP] pobierz, plaintext 

I daj do nich metody:

[PHP] pobierz, plaintext 
public function setMin($min) {
$min = intval($min);
if($min < 1) return false;
$this->_min_len = $min;
}
 
public function setMax($max) {
$max = intval($max);
if($max < 1) return false;
$this->_max_len = $max;
}
[PHP] pobierz, plaintext 

I pamiętaj, DRY - Dont Repeat Yourself

Ten post edytował cudny 4.11.2011, 23:19:34

[szmerak]

pomyśle nad tym.
Dzięki za odpowiedź o coś takiego mi chodziło

Jezu...
A po co ci sprawdzenie po sobie czy coś istnieje ?
if(array_key_exists("MAX_LENGHT", $this->methods) and array_key_exists("MIN_LENGHT", $this->methods))

Tutaj nie sprawdzam czy istnieje. Tylko czy w 1 walidacji 2 metody są ustawione i działają z sensem. Bo trochę głupio by było gdybym ustawił: max_len = 30 min_len = 40 , to nie ma sensu; linijka wyzej poinformuje mnie o tym ze cos spiepszylem...

I pamiętaj, DRY - Dont Repeat Yourself

Jesli chodzi o to, to nie zaczailem ?
to że kod wygląda podobnie w max_len / min_len ?

Ten post edytował szmerak 5.11.2011, 02:53:55

[skowron-line]

Ja Ci proponuje napisać klasę która będzie odwoływała się do zewnętrznych Helperów a sama zawierała by tylko metody typu add i check

[PHP] pobierz, plaintext 
class validate {
    /*
        validator - nazwa helpera lub funkcji natywnej
        field - pole lub string do sprawdzenia
    */
    public function add($validator, $field){}
    /*
        sprawdza wszystkie warunki
        zwraca true/false
    */
    public function check(){}
}
[PHP] pobierz, plaintext 

Zobacz sobie jak to jest w FW rozwiązane
http://kohanaframework.org/3.0/guide/api/Validate
http://framework.zend.com/manual/en/zend.validate.html

przykłady znajdziesz w sieci.