[PHP][OOP]Walidacja, Klasa walidujaca, Opcje

[MateuszS]

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

[Crozin]

Twój walidator zwraca błędy (do tego poprzez wyjątek który jest zupełnie nie potrzebny w tym miejscu)

1) Wyjątków tam nie użyłem - ale w sumie lepiej by było z nich skorzystać przy obecnej wersji.
2) No tak... nazwy z serii "isXXX" same w sobie sugerują, że metoda będzie zwracać true/false. I tu też mogłoby tak być. Wtedy tylko zmieniło by się na:

[PHP] pobierz, plaintext 
$result = $....->isValid($subject);
 
if (true == $result) {
  $this->data[..] = $...->getData();
} else {
  $this->errors[...] = $...->getError();
}
[PHP] pobierz, plaintext 

Kod pisałem z miejsca - nie jest on w żaden dogłęby sposób przemyślany, więc takich kruczków spodziewałbym się więcej. W każdym bądź razie dzięki za zwrócenie uwagi.

Tablice mają to do siebie, że przechowują dane różnych typów w tym wypadku obiekty walidatorów tak jak w kodzie Crozina.

Tutaj tak tylko od siebie dodam, że to zdanie tyczy się tablic w PHP. Tablice w swojej znanej z "normalnych" języków formie przechowują dokładnie X elementów danego typu.

[smentek]

1) Wyjątków tam nie użyłem - ale w sumie lepiej by było z nich skorzystać przy obecnej wersji.

Wyjątki służą do obsługi sytuacji nadzwyczajnych. W systemie walidacji 99% przypadków są zbędne czyli złe.

Scenariusz użycia wyjątku w walidatorze:
Twój walidator ma za zadnie sprawdzać czy wpisany numer jest jednym z numerów katalogowych produktu a owe produkty (a więc i ich numery) są przechowywane w zewnętrznej bazie danych, do której czasem może nie być dostępu, pomimo że baza naszej aplikacji działa. Obsługę takiej (NADZWYCZAJNEJ, NIETPYOWEJ, NIESTANDARDOWEJ, ŻADKIEJ, SPECJALNEJ) sytuacji załatawia nam wyrzucany wyjątek który przekieruje wykonanie naszego kodu do innego miejsca w programie, i tam to sobie obsłużymy, nie zaśmiecając walidatora kodem związanym z połączeniami z bazą danych czy czymkolwiek...

Do tego służy wyjątek.

I jeszcze się przyczepię:

if (true == $result) {

Piszmy:
(true === $result) nawet jeśli (true == $result) przechodzi.