[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.