[PHP]Klasa do sprawdzania formularzy Opcje

[cer98]

Ogólnie dopiero zaczynam programować w OOP, dlatego postanowiłem rozdzielić klasę do walidacji na części.
W ogólnym skrócie przejrzałem sporo materiału, znalazłem, iż klasa abstrakcyjna to taka klasa, której obiektu nie można utworzyć, więc można tylko dziedziczyć.
Proszę o nakierowanie mnie, jakie metody powinny znaleźć się w interfejsie, które w klasie, która dziedziczy. Czy użyć klasy abstrakcyjnej ?
Tutaj wykorzystałem informacje, że przy walidacji, która jest wszędzie, używam Validator validator, gdzie Validator to nazwa interfejsu z metoda doValidate().

Jakie elementy powinny znaleźć się w interfejsie czy są to np. filtry, sposób walidacji, sposób odebrania danych, a których mi brakuje ?
Jak powinna wyglądać taka zdatna klasa?

Ten post edytował cer98 6.08.2011, 21:31:59

[IceManSpy]

Straszne błędy popełniasz.

1. Wiesz do czego służy interfejs? Ale nie wal regułki z wikipedii, tylko wg Ciebie, do czego głównie służy interfejs?
2. Tutaj masz klasę do walidacji tylko stringów, a co z liczbami, datami itd? Nazywaj klasy tak, aby było jednoznaczne co robią?

Na Twoim miejscu olałbym ten interfejs, bo jest tutaj nie potrzebny. Zrób sobie klasę Validator. W niej pola z nazwą obiektu do walidacji i parametrami. Potem dopisz metody set, get i dla każdego typu walidacji dopisz metode do walidacji na zasadzie:

[PHP] pobierz, plaintext 
$obj = new Validator();
$obj->doValidateString($_POST['nick']);
$obj->doValidateEmail($_POST['mail']);
$obj->doValidateDate($_POST['birth']);
[PHP] pobierz, plaintext 

itd.

Ten post edytował IceManSpy 6.08.2011, 19:43:33

[cer98]

1. Żeby obiekt klasy, która dziedziczy mógł nabrać jakieś właściwości i wiele innych
2. One są w private $parameters

Więc ani jednej klasy abstrakcyjnej, dziedziczącej i interfejsu ? Nie wierzę, że to dobre rozwiązanie, ponieważ walidacja jest wszędzie.
Nie wiem jak taka metoda mogłaby wyglądać, jeżeli walidację przeprowadza już jedna metoda, która wczytuje parametry z tablicy.

Ten post edytował cer98 6.08.2011, 19:49:55

[IceManSpy]

1. A po co Ci to? Ok, możesz napisać interfejs,a potem go implementować dla każdego walidatora (każda osobna klasa dla każdego typu walidacji), ale po co? Ktoś, kto by używał Twojego kodu musi pamiętać nazwy klas. A moim przypadku zrobi sobie obiekt klasy walidator, napisze -> i wyskoczy mu podpowiedź. I przy dobrym nazewnictwie połapie się do czego dana metoda służy.
Zobacz jak to w Zendzie robią:
http://framework.zend.com/manual/en/zend.validate.set.html
A tutaj masz krótki kod:

[PHP] pobierz, plaintext 
new Zend_Validate_NotEmpty(),
new Zend_Validate_StringLength(2,30)
[PHP] pobierz, plaintext 

I od razu wiesz, co dana metoda robi.

2. A w tym przypadku inny programista musi wiedzieć jakie parametry ma podać. No chyba, że napiszesz do tego wielką dokumentację ;>

Strasznie utrudniasz sobie życie. To co chcesz zrobić to albo klasa zbiorcza, albo interfejs i miniklasy. Nie myśl, że wielki profesjonalizmem jest zrobieniem skomplikowanego dziedziczenia czy interfejsów. PO CO?!

Ten post edytował IceManSpy 6.08.2011, 20:07:23

[cer98]

Chodzi mi o to, żeby klasa była uniwersalna.
Podając po kolei metody na login, hasło, aktualnie muszę dodać również walidator, napisać co i jak ma być sprawdzane, dodaję za pomocą addValidator, następnie doValidate z nazwą stworzonego filtra w tej samej tablicy.
doValidateEmail po prostu wywoływałby funkcję doValidate z wnętrza, dobrze rozumiem ? nie chciałbym używać w kontrolerze $_POST, tylko dane odebrać wcześniej.

[PHP] pobierz, plaintext 
$validate->addValidator('login', array('min_length' => 3, 
				       'max_length' => 35, 
			       	       'pattern' => 'numericString',
				       'label' => 'Login' 
				       ));
 
$validate->doValidate(@$_POST['login'] ,'login', true);
[PHP] pobierz, plaintext 

Ten post edytował cer98 6.08.2011, 20:15:57

[IceManSpy]

Skąd Ci się jakiś kontroler tutaj bierze?! Zostaw na razie MVC.

Na prawdę poczytaj na temat obiektówki, bo strasznie mieszasz.
Zrób sobie klasę walidator, w niej metody doValidateString, doValidateMail itd które zwracaja true/false, a parametry możesz przekazać przez te metody (np długość itd).
Wtedy będziesz mógł wykorzystać to w przyszłości.

Jak się nie zgadzasz z tym to napisz jeszcze raz jak Ty myślisz. Tylko krótsze zadania pisz, a najlepiej to w punktach.

P.S. A podanie w tablicy: 'pattern' => 'numericString' to jak odbierzesz i przerobisz? Ile będzie tych patternów?

Ten post edytował IceManSpy 6.08.2011, 20:22:50

[cer98]

obecny sposób:

[PHP] pobierz, plaintext 
 
class Validate {
 
private $parameters = array(
 
		'numericString' => array(
			'pattern' => '^[a-zA-Z0-9]+$',
		),
 
		'www' => array(
			'pattern' => '^(http://|https://)(([a-z0-9]([-a-z0-9]*[a-z0-9]+)?){1,63}\.)+[a-z]{2,6}',
		));
 
 
 
public function addValidator(){ 
// dodaje po prostu do tablicy wyżej, jeśli trzeba ustalić np. długość znaków i patter numericString
}
 
 
public function doValidate(){
// sprawdzanie czy sa bledy (znaki lub pattern)
}
 
 
public function allClear(){
// jesli wszystko gra
}
 
}
 
//wywolanie
$obj->addValidator('opis', array('min_length' => 3, 
				       'max_length' => 35, 
			       	       'pattern' => 'numericString',
				       'label' => 'Opis' 
				       ));
 
$obj->doValidate(@$_POST['opis'] ,'opis', true);
 
[PHP] pobierz, plaintext 

Teraz możesz mi powiedzieć, jak mam wykonać twoją metodą, jeśli nie wiem, czy np. pole jest wymagane ? Jaki ma label ? Przecież jest mi to potrzebne, co widzisz powyżej, ale zwracam honor mogłeś nie wiedzieć mojej konstrukcji.

[Spawnm]

Cer98 - to co próbujesz zrobić strasznie przypomina mi: http://spawnframework.com/doc/25/sf_valid (IMG:style_emoticons/default/smile.gif)

[cer98]

Po prostu jeśli dodam do

[PHP] pobierz, plaintext 
$obj->doValidateString($_POST['nick']);
[PHP] pobierz, plaintext 

to, czy ma być wypełnione, label dla błędu, będzie takie same.

To nie mój pomysł, ja zaczerpnąłem sposób, w jaki wypełnić tablicę.

Może tak być ?

1. Jak napisać do tego kontroler wiedząc, że allClear zwraca true jeśli gra, jeśli nie serializuję tablicę błędów ?

2. Mam drugie pytanie, co do wysyłania formularza, bo musi być określone. Jeśli mam wzorzec router, do tego index.php korzysta z szablonów i w jednym szablonie mam plik z takim formularzem - wysyłam go gdzie (pole action="") ?

[IceManSpy]

Teraz możesz mi powiedzieć, jak mam wykonać twoją metodą, jeśli nie wiem, czy np. pole jest wymagane ? Jaki ma label ? Przecież jest mi to potrzebne, co widzisz powyżej, ale zwracam honor mogłeś nie wiedzieć mojej konstrukcji.

Do czego Ci label? Label służy tylko do opisu w formularzu - nie jest przekazywany. Tworzysz obiekt i dla każdej wartości z formularza wywołujesz odpowiednią metodę. I za pomocą niej sprawdzasz to co chcesz (np isRequired , isValid itd).
No ale zostańmy, że przekazujesz jako tablica parametry do sprawdzenia. Pomysł nie najgorszy o ile nie będzie kilkudziesięciu parametrów do przekazania.

Co do pytań.
1. Jak zwraca true to chyba Ty powinieneś wiedzieć, co chcesz zrobić z tymi danymi. Czyli:

[PHP] pobierz, plaintext 
if($obj->allClear){
przetwarzam dane
} else {
odrzucam dane
}
[PHP] pobierz, plaintext 

Swoją drogą dla mnie allClear oznacza - wyczyść wszystko, a nie, że wszystko ok. Powinno być coś a'la isValid.

2. Tam go wysyłasz, gdzie go chcesz odebrać ;>

[cer98]

Czy powinienem użyć tutaj redirect z serializacją błędów ?

Label używam po to, aby przy określonym polu wyświetlić komunikat błędu.

[IceManSpy]

Czy powinienem użyć tutaj redirect z serializacją błędów ?

Używasz tam, gdzie są błędy i gdzie chcesz, aby ona się pokazały.

[cer98]

Mam już model do formularzy, jak napisać kontroler do obsługi formularzy, który korzystałby z modelu walidacja ?

p.s. czy te błędy można przesłać sesją ?

Ten post edytował cer98 7.08.2011, 13:06:57

[ano]

Co do walidacji:
najbardziej uniwersalne byłoby, jakbyś napisał sobie interfejs Validator (np tylko z metodą isValid(string) )
potem do każdego typu walidacji osobną klasę implementującą Twój interfejs. (validator od emaila, adresu strony i co tam sobie wymyślisz).
W Twoim przypadku byłoby to korzystniejsze przy tworzenie formularzy i ich walidacji.
W zendzie element formularza to obiekt

[PHP] pobierz, plaintext 
$title = new Zend_Form_Element_Text('title');
		$title->setLabel('Tytuł:')
			->addValidator('NotEmpty')    //metoda addvalidator tak naprawdę tworzy obiekt Validate_NotEmpty
			->setRequired(true);
[PHP] pobierz, plaintext 

Dzięki czemuś takiemu, w przyszłości będziesz mógł łatwiej wykorzystywać swoje validatory do różnych celów.
Do prostych zastosowań możesz zrobić jedną klasę validator z metodami isEmailValid i milionem podobnych, ale na dłuższą metę to się nie sprawdzi.

[IceManSpy]

Skoro piszesz o kontrolerach i modelach, to jakiego używasz frameworka? Czy coś własnego masz?

[lukaskolista]

Polecam modul validacji z frameworka kohana, bez problemu mozna go "wyciagnac" z framweorka. Masz tam wszystkie popularne metody walidacji. Walidator: http://kohanaframework.org/3.1/guide/api/Validation, metody walidacji: http://kohanaframework.org/3.1/guide/api/Valid
Wykorzystanie jest banalnie proste:

[PHP] pobierz, plaintext 
$validation = Validation::factory($_POST)
				->rules('username', array(
					array('not_empty'),
					array(array('klasa', 'metoda'), array('parametr1', 'parametr2'))
				))
				->rules('password', array(
					array('not_empty'),
					array(array('klasa', 'metoda'), array('parametr1', 'parametr2'))
				));
 
if ($validation->check())
{
	echo 'Dane poprawne.';
}
else
{
	echo 'Dane niepoprawne.';
}
[PHP] pobierz, plaintext 

Zaleta jest mozliwosc definiowania wlasnych metod walidacji z ktorejkolwiek innej klasy

[cer98]

Ice piszę framework, mam już wszystko, jestem przy FrontControllerze, dispatcherze. W końcu muszę jakoś dobrać się do moich metod. Dzięki za przykłady.