Forum PHP.pl

Witam, zaczynam z OOP i napisałem prostą klasę walidującą dla małych formularzy, napiszcie co można poprawić, miałem ją zaopatrzyć w MVC i prezentować błędy widokiem ale to już na kiedy indziej.

[PHP] pobierz, plaintext 
//wlasciwa klasa w moim nastepnym poscie w tym temacie ;P
[PHP] pobierz, plaintext 

Ten post edytował MateuszS 21.04.2010, 17:23:14

Dlaczego ten kod się do niczego nie nadaje?

1) Powiedzmy, że mamy pole "treść" w dwóch formularzach: edycji sygnaturki oraz tworzenia nowego wpisu do np. bloga. Chcę by pole "treść" w sygnaturce miało pomiędzy 10, a 255 znaków oraz by nie można było podać w nim żadnego adresu URL. Natomiast "treść" w formularzu nowego wpisu ma mieć długość co najmniej 50 znaków.
W tym momencie to "narzędzie" poległo (czyt.: to w jaki sposób co sprawdzać nie powinno być zawarte w klasie).

2) Powiedzmy, że ponownie mam dwa pola "treść". Chcę by treść komunikatu w przypadku gdy podano za mało znaków w formularzu edycji sygnaturki była następująca: "ABC Sygnaturka musi mieć co najmniej 10 znaków", a w formularzu dodania nowego wpisu: "DEF +50 znaków proszę."
Ponownie Twój kod się nie spisuje (czyt.: treści komunikatów również poza klasą lub możliwość ich dowolnej modyfikacji)

3) Chcę dodać sobie nowy walidator - np. sprawdzający czy nr NIP jest poprawny. Muszę przebudowywać pół klasy? (czyt.: walidatory powinny być osobnymi obiektami implementującymi jakiś interfejs)

To tak na początek.

To tak:
1. Miałem tragiczny błąd w metodzie, nie pozamieniałem z powrotem $this->zmienna na $zmienna, już to poprawiłem.
2. Dodałem metodę za pomocą której możesz bez przerabiania klasy dodawać nowe typy pól, np. ów nip.
3. Mógłbyś wytłumaczyć o co chodzi z

[PHP] pobierz, plaintext 
<?
 
class Walidacja 
{
 
	/*	 Dozwolone opcje:
    * 	min_znakow - minimalna ilosc znakow
	* 	max_znakow - maksymalna ilosc znakow
	*	 wzor       - wzor wg ktorego ma byc walidowana zmienna
 
		Wywolanie obiektu
		$ob = new Walidacja;
		$ob->waliduj(zmienna, kryterium);
 
		Mozna swobodnie dodawac nowe pola/typy pol do ustawien, np
		kod pocztowy i zaopatrzyc go w atrybut wzor w ktorym bylby
		wzor w postaci wyrazen regularnych, ktory mialby za zadanie
		sprawdzac zmienna czy ma poprawny format.
	*/
 
	public $ustawienia = array(
		"login" 			=> array(
			"min_znakow"		=> 3,
			"max_znakow" 		=> 25	
		),
		"haslo" 			=> array(	
			"min_znakow"		=> 5,
			"max_znakow"		=> 15		
		),	
		"email"				=> array(
			"wzor"				=> "^[_.0-9a-z-]+@([0-9a-z][0-9a-z-]+.)+[a-z]{2,4}$",
		),
		"zabronione"	=> array()
 
	);
 
	private $bledy 			= array();
 
	public function dodaj($typ, $min = null, $max = null, $wzor = null)
	{
		if(!isset($min) and !isset($max) and !isset($wzor)) {
			throw new Exception("Musisz podac dodatkowy atrybut(maksymalna ilosc znakow, minimalna lub wzor");
		} else {
			$typ = (string)$typ;
			if(isset($min)) $this->ustawienia[$typ]["min_znakow"] = (int)$min;
			if(isset($max)) $this->ustawienia[$typ]["max_znakow"] = (int)$max;
			if(isset($wzor)) $this->ustawienia[$typ]["wzor"] = $wzor;
		}
 
	}
 
	public function zabron($zabronione)
	{
		if(!is_array($zabronione)) {
			array_push($this->ustawienia["zabronione"], $zabronione);
		} else {
			foreach($zabronione as $wartosc) 
				array_push($this->ustawienia["zabronione"], $wartosc);
		}	
	}
 
	public function waliduj($zmienna, $typ = null)
	{
 
		if(is_array($zmienna)) {
			throw new Exception("Weryfikowana zmienna nie moze byc tablica. Bo tak.");
		}
 
		$zmienna = trim(strip_tags($zmienna));
 
		foreach($this->ustawienia["zabronione"] as $zabronione) {
			if(preg_match("#".$zabronione."#", $zmienna)) {
				$this->bledy[] = "Formularz zawiera niedozwolone znaki/slowa: ".$zabronione."."; 
				break;
			}	
		}
 
		if(isset($typ)) {
			if(isset($this->ustawienia[$typ])) {
				$ustawienia = $this->ustawienia;
 
				foreach($this->ustawienia[$typ] as $klucz => $wartosc) {
					if($klucz == "min_znakow") { 
						if(strlen($zmienna) < $this->ustawienia[$typ]["min_znakow"])
					$this->bledy[] = ucfirst($typ)." musi miec conajmniej ".$this->ustawienia[$typ]["min_znakow"]." znakow.";
					}
					if($klucz == "max_znakow"){
						if(strlen($zmienna) > $this->ustawienia[$typ]["max_znakow"])
							$this->bledy[] = ucfirst($typ)." moze miec conajwyzej ".$this->ustawienia[$typ]["max_znakow"]." znakow.";		
					}
					if($klucz == "wzor"){
						if(!preg_match("#".$this->ustawienia[$typ]["wzor"]."#", $zmienna))
							$this->bledy[] = ucfirst($typ)." ma niepoprawny format.";		
					}			
 
				}
			}
		}
 
 
		return $zmienna;
	}
 
	public function pokazbledy()
	{
		return $this->bledy;
	}
}
 
try {
 
 
	$obWaliduj = new Walidacja();
	$obWaliduj->dodaj("tresc", 1, 10);
	$obWaliduj->waliduj("hesdsadsaddads", "tresc");
	$obWaliduj->dodaj("tresc", 4, 7);
	$obWaliduj->waliduj("aa", "tresc");
	$bledy = $obWaliduj->pokazbledy();
		foreach($bledy as $blad) {
			echo $blad."<br />";
		}
 
} catch(Exception $e) {
	echo "Blad: ".$e->getMessage();
}	
[PHP] pobierz, plaintext 

Ten post edytował MateuszS 21.04.2010, 18:36:42

destruct masakryczny , echo w modelu ?
Dodatkowo lepiej by wyglądała całość gdybyś dał angielskie nazwy.
rozbił bym walidację na metody np valid::mail('adf@dsaf.pl');

Walidacja NIPu nie polega tylko na policzeniu jego długości. http://pl.wikipedia.org/wiki/NIP/Implementacja

Ten post edytował Wicepsik 21.04.2010, 17:43:07

Spawnm, tak wiem, to nie jest problem zamienić na ang, echo dałem bo jak napisałem, nie mam klasy widoku, więc na szybko wyświetlam błędy,

Wicepsik, gdybyś zobaczył kod, byś wiedział że można dodać do metody 4 parametr którym jest wyrażenie regularne. W każdym razie jeżeli wyrażenia nie wystarczą to dodam jeszcze metodę sprawdzającą nip.

edit:: dodałem prostą metodę zwracania błędów.

Ten post edytował MateuszS 21.04.2010, 18:37:01

Chodzi o to, że walidatory powinny być osobnymi obiektami wyspecjalizowanymi w walidacji konkretnego rodzaju. Daje to możliwość, bym utworzył sobie swój własny walidator, niewpisany na sztywno w kod Twojej klasy - innymi słowy: zwiększamy elastyczność narzędzia.

Tutaj masz przykładowy kod, który ilustruje jak mogło by to wyglądać (oczywiście brak tutaj dziesiątek rzeczy (ustawienia czytelnej treści błędu, wielu walidatorów dla jednego elementu, przedefiniowanych opcji dla walidatorów i całej masy innych rzeczy)):

[PHP] pobierz, plaintext 
<?php
 
namespace Crozin\Validation {
	interface Validatable {
		public function configure(array $params = array());
		public function getOptions();
		public function getOption($key);
 
		public function isValid($subject);
	}
 
	class Validator {
		protected $subjects = array();
		protected $validators = array();
 
		protected $errors = array();
		protected $data = array();
 
		public function isValid() {
			$isValid = true;
 
			foreach ($this->subjects as $key => $subject) {
				$result = $this->validators[$key]->isValid($subject);
 
				if ($result instanceof Response) {
					$isValid = false;
					$this->errors[$key] = $result;
				} else {
					$this->data[$key] = $result;
				}
			}
 
			return $isValid;
		}
 
		public function addSubjects(array $subjects) {
			foreach ($subjects as $key => $subject) {
				$this->addSubject($key, $subject);
			}
		}
 
		public function addSubject($key, $subject) {
			$this->subjects[$key] = $subject; 
		}
 
		public function addValidators(array $validators) {
			foreach ($validators as $key => $validator) {
				$this->addValidator($key, $validator);
			}
		}
 
		public function addValidator($key, Validatable $validator) {
			$this->validators[$key] = $validator; 
		}
 
		public function getData() {
			return $this->data;
		}
 
		public function getErrors() {
			return $this->errors;
		}
	}
 
	class Response {
		protected $msg;
		protected $subject;
		protected $validator;
 
		public function __construct($msg, $subject, Validatable $validator) {
			$this->msg = $msg;
			$this->subject = $subject;
			$this->validator = $validator;
		}
 
		public function getMsg() {
			return $this->msg;
		}
 
		public function getSubject() {
			return $this->subject;
		}
 
		public function getValidator() {
			return $this->validator;
		}
	}
}
 
namespace Crozin\Validation\Validator {
	use \Crozin\Validation\Validatable;
	use \Crozin\Validation\Response;
 
	abstract class Base {
		protected $options = array();
 
		public function __construct(array $options = array()) {
			$this->configure($options);
		}
 
		public function configure(array $options = array()) {
			$this->options = $options;
		}
 
		public function getOptions() {
			return $this->options;
		}
 
		public function getOption($key) {
			return isset($this->options[$key]) ? $this->options[$key] : null; 
		}
	}
 
	class NIP extends Base implements Validatable {
		public function isValid($subject) {
			/* NIP ma nieprawidłowy format (np. podano litery) */
			if (1 == mt_rand(1, 2)) {
				return new Response('format', $subject, $this);
			}
 
			/* NIP ma nieprawidłową sumę kontrolną */
			if (1 == mt_rand(1, 2)) {
				return new Response('control.sum', $subject, $this);
			}
 
			return $subject;
		}
	}
 
	class String extends Base implements Validatable {	
		public function isValid($subject) {
			if ($this->getOption('trim')) {
				$subject = trim($subject);
			}
 
			if (($min = $this->getOption('min')) && strlen($subject) < $min) {
				return new Response('min', $subject, $this);
			}
 
			if (($max = $this->getOption('max')) && strlen($subject) > $max) {
				return new Response('max', $subject, $this);
			}
 
			return $subject;
		}
	}
 
	class Email extends Base implements Validatable {
		public function isValid($subject) {
			/* Nieprawidłowy format */
			if (1 == mt_rand(1, 2)) {
				return new Response('format', $subject, $this);
			}
 
			/* Jeżeli wpisy MX nie są dobre */
			if (1 == mt_rand(1, 2)) {
				return new Response('mx', $subject, $this);
			}
 
			return $subject;
		}
	}
 
	class NumberRange extends Base implements Validatable {
		public function isValid($subject) {
			if ($this->getOption('natural.only') && false == ctype_digit($subject)) {
				return new Response('not.natural', $subject, $this);
			}
 
			if ($subject < $this->getOption('min') || $subject > $this->getOption('max')) {
				return new Response('out.of.range', $subject, $this);
			}
 
			return $subject;
		}
	}
}
 
namespace {
	use \Crozin\Validation\Validator;
	use \Crozin\Validation;
 
	error_reporting(E_ALL | E_STRICT);
 
	$_POST = array(
		'firstname' => ' Krzysiek  ',
		'nickname'  => 'Crozin',
		'age'       => '7',
		'mail'      => 'k...s@gmail.com'
	);
 
	$v = new Validator();
	$v->addSubjects($_POST);
	$v->addValidators(array(
		'firstname' => new Validator\String(array('trim' => true, 'min' => 6)),
		'nickname'  => new Validator\String(),
		'age'       => new Validator\NumberRange(array('natural.only' => true, 'min' => 9, 'max' => 99)),
		'mail'      => new Validator\Email()
	));
 
	if ($v->isValid()) {
		var_dump('Poprawne dane:');
		var_dump($v->getData());
	} else {
		foreach ($v->getErrors() as $error) {
			var_dump(sprintf('%s - %s - %s - %s', get_class($error->getValidator()), $error->getSubject(), $error->getMsg(), print_r($error->getValidator()->getOptions(), 1)));
		}
	}
}
[PHP] pobierz, plaintext 

Ale dlaczego to rozwiązanie jest o niebo lepsze? Chcę utworzyć nowy walidator... wystarczy mi obiekt dowolny, który implementuje interfejs \Crozin\Validation\Validatable; Chcę spersonalizować dany walidator? Podaję mu parametr. Itp. itd.

PS. Całość pisana "z palca" bez zastanowienia (jedynie sprawdziłem czy w ogóle działa) - więc jest to na pewno kod do poprawki (przemyślenia sposobu działania, przerobienia), ale daje jako taki zarys pewnej elastyczności.

Ten post edytował Crozin 21.04.2010, 22:03:19

Niestety jest to dla mnie czarna magia co napisałeś, te interfejsy, klasy abstrakcyjne, nigdy nie mogłem załapać o co w nich chodzi, nie sądziłem że mi się prędko przydadzą.
Poza tym wywala błędy z tym namespace, nieoczekiwany string, no i czy ten system waliduje chociaż pod kątem tagów HTML w polach? Bo widzę setki dziwnych rzeczy ale nigdy bym nie zgadł że to ma coś wspólnego z walidacją.

Widocznie masz PHP w wersji poniżej 5.3 - ale przestrzenie nazw można stąd wywalić na dobrą sprawę.
Nie, dlaczego miałby to robić? A jeżeli jest Ci takie coś potrzebne wystarczy zrobić sobie walidator, który będzie sprawdzać coś takiego.

W takim razie radzę sobie na chwilę obecną darować jakiekolwiek pisanie i skupić się wyłącznie na teorii i możliwościach języka (swoją drogą bardzo ograniczonych pod względem OOP).

Ten post edytował Crozin 21.04.2010, 20:18:39

No mam PHP 5.2.6.


No ja przecież napisałem skrypt który ma za zadanie walidować dane z formularza, myślałem że twój też to robi Bo o to mi głównie chodzi, napisać dobrą klasę która sprawdza takie rzeczy


Nie no wiesz, logiczne że najpierw teoria, ale w mojej książce są zaledwie 2 strony o interfejsach i brak jakichkolwiek przykładów praktycznego zastosowania, stąd mój brak wiedzy na ten temat... a podobno najlepsza jest praktyka.

Wiele osób radzi żebym zajął się jakimś frameworkiem, ale co mi po frameworku jak wg was nie umiem napisać najprostrzej rzeczy w OOP

Jeżeli mu rozkażesz to i to będzie Ci robić tylko dlaczego miałby z automatu blokować elementy HTMLa w tych polach? Że niby nie mogę w treści wątku wpisać <p> to akapit w HTMLu? Takimi rzeczami jak wyświetlanie wprowadzonych przez użytkowników danych (by uniemożliwić wstawienie własnego kodu HTML itp.) też trzeba się zająć - ale to nie jest robota dla walidatora.
No na dobrą sprawę to o interfejsach nie ma się co wiele rozpisywać. Nie jest to jakaś skomplikowana część języka.No to to co Ci podałem możesz potraktować jako konkretny przykład zastosowania interfejsów - metoda \Crozin\Validation\Validator::addValidator() wymaga by drugi parametr był obiektem implementującym interfejs \Crozin\Validation\Validatable. Wymaga tego ponieważ później wykonywane są operacje na tych walidatorach (a dokładnie to wywoływanie metody isValid()). Dzięki temu mamy pewność, że to co przekazujemy jako walidator jest ów walidatorem, albo przynajmniej potrafi się zachowywać jak takowy.

Również uważam, że branie się za FW w momencie gdy nie zna się nawet elementów języka jest głupotą.

No nie przesadzajmy, podstawy pewne mam

Tego negować nie będę, ale jeżeli nie znasz elementów języka (interfejsy, klasy abstrakcyjne, przestrzenie nazw, Type Hinting (nie wiem jaka można by to po polsku nazwać) itp) nie wspominając o podstawowych paradygmatach obiektówki (abstrakcja bytów, polimorfizm, hermetyzacja itp.) to ciężko jest mówić o tym, że ma się opanowane podstawowe założenia programowania obiektowego.

Zacznij od manuala - przebrnij przez paskudnych kilkanaście stron teorii - potem polecałbym poczytać nieco artykułów dotyczących OOP (nie koniecznie nastawionego na PHP, gdyż te idee są wspólne dla wszystkich języków obiektowych).
Czasami dobrze też zacząć pisanie od końca czyli od interfejsu (chodzi tutaj o to w jaki sposób będziesz danego narzędzia używać (czyli to co jest w liniach 192 - 199 mojego kodu)). Tworzysz sobie jak najbardziej skomplikowane przykłady użycia danego narzędzia, a dopiero potem bierzesz się za implementację.

@Crozin mógłbyś rzucić kilka linków do artykułów które dotyczą OOP ? Chodzi mi o artykuły które są sprawdzone i dobre według Ciebie.

Crozin, to że nie użyłem w mojej klasie np. dziedziczenia to nie znaczy że nie wiem jak to używać
marcio, potiwerdzam, kursy nie mają dobrych przykładów, w książki też trochę zainwestowałem ale chyba nie bardzo się opłacało. Manual może i jest dobry, ale trzeba znać perfekcyjnie angielski ja puki co jeszcze się kształcę w tym kierunku.

No a jak wy się OOP uczyliście? Też mieliście takie problemy? ;/ Może ponaśladuję

Ja czytalem kursy,ksiazki i zadawalem glupie pytania i brali mnie za debila i tlumaczyli jak debilowi az wkoncu posciagalem pare fw, roznych klas poprobowalem sam cos pokodzic az weszlo mi w krew oop.

He czyli głupie pytania kluczem do sukcesu To kolejne:

Jaka jest praktyczna różnica pomiędzy dziedziczeniem a wywołaniem obiektu klasy w klasie, przykład:

[PHP] pobierz, plaintext 
class a
{
   public $c = 1;
}
class b extends a
{
function bla()
{
echo $this->c; 
}
 
}
[PHP] pobierz, plaintext 

a

[PHP] pobierz, plaintext 
class a
{
public $aa=1;
}
 
class b
{
function __construct()
{
$this->ob = new a;
echo $this->ob->aa;
}
}
[PHP] pobierz, plaintext 

Ten post edytował MateuszS 22.04.2010, 20:59:13