![]() |
![]() |
![]() ![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 1 429 Pomógł: 195 Dołączył: 6.10.2008 Skąd: Kraków/Tomaszów Lubelski Ostrzeżenie: (0%) ![]() ![]() |
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.
Ten post edytował MateuszS 21.04.2010, 17:23:14 |
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Zarejestrowani Postów: 6 476 Pomógł: 1306 Dołączył: 6.08.2006 Skąd: Kraków Ostrzeżenie: (0%) ![]() ![]() |
Cytat Tablice w większości języków programowania mogą zawierać null który nie będzie już typem X Tam się NULLa nie czepiaj... na dobrą sprawę prawie wszędzie da się go wepchać. (IMG:style_emoticons/default/smile.gif) Chciałbym też zwrócić uwagę, że to co w PHP nazywamy tablicą jest w rzeczywistości o ile dobrze pamiętam Hash Table (nie wiem jak to poprawnie po polsku nazwać) w C. Ale tutaj się wypowiadać nie będę bo nie jestem dobrze zorientowany w temacie. Jeszcze co do używania wyjątków jako zwykłej konstrukcji sterującej. W gruncie rzeczy masz rację - wyjątki są od sytuacji wyjątkowych. Czasami jednak ich cecha jaką jest wyskoczenie z bloku TRY do właściwego bloku CATCH jest czymś bardzo wygodnym i pożądanym. Elementów języka powinno się używać poprawnie, ale nie popadajmy w paranoję - jeżeli coś gdzieś mogłoby się sprawdzić lepiej warto z tego skorzystać. A co do operatora identyczności to już zupełnie nie rozumiem... jakaś wyższość === nad == w przypadku gdy metoda zwraca true/false - chyba, że to wycinek kodu, gdzie użycie takiego operatora mogłoby mieć sensowne uzasadnienie (zbyt leniwy jestem by teraz sprawdzić skąd wziąłeś ten skrawek :]). @MateuszS: Z tym emailem to masz dwa błędy: 1) Takie coś jak wyrażenie to powinno być wewnętrzną sprawą tego obiektu. Swoją drogą, dlaczego nie skorzystasz z filter_var do sprawdzenia poprawności formatu? 2) Maxlength w przypadku tego walidatora jest pozbawione sensu... raz że sam format definiuje maksymalną długość maila, dwa, że od długości tekstu powinien być jakiś walidator od.. tekstu (nie od emaila). |
|
|
![]() ![]() |
![]() |
Aktualny czas: 15.10.2025 - 13:02 |