![]() |
![]() ![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 697 Pomógł: 47 Dołączył: 19.12.2003 Skąd: Lublin Ostrzeżenie: (0%) ![]() ![]() |
Tak piszę sobie powoli mój framework i ostatnio zobaczyłem w Zendzie fajne rozwiązanie validatora (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . Trochę składnia zenda do mnie nie przemawia wiec napisałem coś swojego.
Dla nieobczajonych. Klasa sprawdza poprawność elementów w tablicy na podstawie określonych wymogów. Przyklad uzycia
* odnosi się do wszystkich zmiennych przy czym wlaściwości elementu sa zawsze nadrzędne(tj. gwiazdka ma zawsze mniejszy priorytet, nawet jesli podamy ją jako ostatni element tablicy). Dostępne wymagania minlength - minimalna długość maxlength - maksymalna dlugosc required - czy element nie moze byc pusty(czy jest wymagany) between - wartosc musi sie miescic w podanym zakresie in - wartosc musi byc identyczna z jedna sposrod podanych type - typ zmiennej (tj. string, boolean, double, integer) lub obiekt klasy własnej. Jeśli deklarujemy wlasny typ to jako type przekazujemy obiekt naszej klasy pamietajac ze musi ona zawietac metodę isValid(), oraz zmienna $errors oraz zmienna $value Wiecej w przykladzie dostepnm w paczce. Pobierz Czekam na komentarze oraz pomysły na rozwinięcie listy wymagań oraz ogólnie opcji validatora. |
|
|
![]()
Post
#2
|
|
Grupa: Zarejestrowani Postów: 662 Pomógł: 45 Dołączył: 26.03.2007 Skąd: Warszawa Ostrzeżenie: (0%) ![]() ![]() |
Bardzo fajne narzędzie, jak wiesz sam coś takiego piszę więc wiem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Ogólnie napiszę co wydaje mi się że mógł byś dodać (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
1) Porządek w kodzie (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) 2) W typie mogły by być wartości standardowe typu telefon, www, e-mail czy nawet ten pesel (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) 3) Jeśli sprawdzę wszystkie pola, a później której osobno to sprawdza od nowa. Mógłbyś gdzieś zapisywać czy pole jest poprawnie wypełnione czy nie aby nie sprawdzać po 2 razy. 4) Komunikaty błędów mogłeś zrobić nie jako wiadomości a jako identyfikatory danego błędu. Można później łatwiej na nich operować (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) To takie drobne uwagi spowodowane tym że sam pisze teraz taką bibliotekę, więc napisałem to co u siebie zrobiłem w inny sposób (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Ale ogólnie klasa spoko i taki sposób walidacji naprawdę ułatwia (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ps. Twoja klasa ma równo 100 linii mniej (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Ale ja u siebie każdą opcje dawałem jako osobną metodę, więc to dla tego (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) |
|
|
![]()
Post
#3
|
|
Grupa: Zarejestrowani Postów: 697 Pomógł: 47 Dołączył: 19.12.2003 Skąd: Lublin Ostrzeżenie: (0%) ![]() ![]() |
@Moli: dzięki za komentarz. Co do opcji
1. Porzadek zrobiony, kod odchudziłem o okolo 40 linijek (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) i jeszcze widzę możliwość optymalizacji(ale to w wolnym czasie) 2. Dodałem obsługę pola e-mail. Pesel jest typowo polskim wymysłem a z numerem telefonu jest tak, że nikt się nie trzyma standardowego zapisu +48.numer, 3. Cachowanie winików dodane 4. Tak jest mi wygodniej (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . Może zastosuje system stałych i komunikatów deklarowanych w jeden z zmiennych klasowych w postaci tablicy. Jeszcze nad tym pomyślę. Dodatkowo dodalem parametr regexp umożliwiający określenie wyrażenia regularnego(w example.php jest pokazane użycie) Nowa paczka: Pobierz Licencja : AGPL3 Pozdrawiam |
|
|
![]()
Post
#4
|
|
Grupa: Zarejestrowani Postów: 6 476 Pomógł: 1306 Dołączył: 6.08.2006 Skąd: Kraków Ostrzeżenie: (0%) ![]() ![]() |
Ja u siebie trochę inaczej to rozwiązałem, ale:
1) Pliki są w kodowaniu UTF-8, dlatego też zamiast np. strlen" title="Zobacz w manualu PHP" target="_manual używaj mb_strlen" title="Zobacz w manualu PHP" target="_manual (pozwole sobie przyspamować http://blog.crozin.com/2008/08/11/operacje...odowanie-utf-8/ (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ) 2) Sprzydałoby się trochę więcej danych w przypadku błędu, dla przykładu: Zdanie ala ma kota ma więcej niż te 8 znaków, ale samo Too long może być czasem trochę niewygodne (np. gdy chcemy zautomatyzować wyświetlanie błędów użytkownikowi). Tutaj mógłbyś zwrocić: False (czyli, że błąd) + Spodziewaną długość (8) + Długość jaką miał ciąg (13) + ew. sam ciąg - aby wygodnie wyświetlić szczegółowy komunikat Cytat Jakieś tam pole może mieć maksymalnie 8 znaków - podano 13. 3) Apropo sposobu zgłaszania błedów - sprzydała by się możliwość ustawiania "indywidualnych" błedów - posłużę się tutaj moim przykładem : I kilka słów wyjaśnienia. Kernel_Validate::customMessage() (btw: właśnie się zooriętowałem, że mam customMassages w orginalnym kodzie (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) ) jako pierwszy i jedyny argument przyjmuje tablice, w której każdy klucz odpowiada zmiennej z $_POST (w tym przypadku) a wartość danego klucza to kolejna tablica, w której jako klucz podawane są stałe, np: Kernel_Validate_Equal::NOT_EQUAL - co oznacza, że jeżeli walidator Equal wyrzuci bład NOT_EQUAL to zamiast domyślnej wiadomości Pole [nazwaPola] musi mieć wartość [wartość]. zostanie zwrócone Musisz zaakceptować regulamin, aby się zarejestrować. Sam walidator działa w miarę podobnie do Twojego, z tym, że reguły walidacji możemy podać jako tablice (jak u Ciebie), albo jako string z "pseudo-kodem", a same walidatory (Equal, sameAs, Email itp.) to klasy implementujące interfejs i dziedziczące po abstrakcyjnej klasie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) PS. Nie wiem dlaczego, ale Twój sposób formatowania kodu jest dla mnie strasznie nieczytelny (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) |
|
|
![]()
Post
#5
|
|
Grupa: Zarejestrowani Postów: 697 Pomógł: 47 Dołączył: 19.12.2003 Skąd: Lublin Ostrzeżenie: (0%) ![]() ![]() |
1) dzięki wielkie za link, przyda się (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) . Widziałem kiedyś w manualu funkcje mb_ ale jakoś nigdy nie wnikałem co czego one służą (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
2,3) Ostatnio zauwazyłem identyczny rodzaj określania komunikatów błędów w Zendzie i przyznam, że rozwiązanie jest dosyć ciekawe i bardzo prawdopodobne, że wprowadzę je u siebie(jednak muszę to zintegrować z moja klasę do obsługi języków stąd te pseudokomunikaty typu "Too long") Dzięki za pomysł z sameAs i equal (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . Pomyślę jak to fajnie złożyć. Pseudojęzyków nie będę wprowadzał. Imho zbędny bajer. PS: nieczytelność kodu to rzecz względna (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . Widzę u Ciebie np. tabulacje przed "=>" w deklaracjach tablic i lekko inny system wcięć. Dodatkowo nie stosujesz spacji między " -> " itd itd. Pozdrawiam |
|
|
![]()
Post
#6
|
|
Grupa: Zarejestrowani Postów: 6 476 Pomógł: 1306 Dołączył: 6.08.2006 Skąd: Kraków Ostrzeżenie: (0%) ![]() ![]() |
Cytat jednak muszę to zintegrować z moja klasę do obsługi języków stąd te pseudokomunikaty typu "Too long" Nic prostszego (pisząc pierwszego posta sam zamieniałem "zmienne językowe" na zwykły tekst)Tutaj właśnie ważne jest to, aby w przypadku wystąpienia błędu nie zwracać tylko False (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Stała Kernel_Validate_Equal::NOT_EQUAL to string notEqual i w przypadku błeu zwracana jest tablica Kod Array( I klucz tablic to nazwa pola, zero - nazwa walidatora, jeden - wartość zmiennej, dwa - argumenty przekazane walidatorowi (np.: że ma być równe (equal) 1)[rules] => Array( [0] => 'notEqual', [1] => '0', [2] => Array( [0] => '1' ) ), [login] => Array( [0] => 'notAlnum', [1] => 'XSS <>"'&', [2] => Array() ) ) Mając te dane wystarczy: To już pisane z palca, ale chyba powinno być dobrze (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) (musisz sobie oczywiście to nieco mocniej rozbudować) |
|
|
![]()
Post
#7
|
|
Grupa: Zarejestrowani Postów: 697 Pomógł: 47 Dołączył: 19.12.2003 Skąd: Lublin Ostrzeżenie: (0%) ![]() ![]() |
Dokładnie tak to u mnie działa (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . Tylko mam zewnętrzną klasę do tego. Ale ogólnie robi to samo. Ważne tylko żebym gotowych komunikatów nie wyciagał a właśnie tablice z komunikatami i odpowiednimi parametrami wtedy mi to się ładnie potłumaczy na inne języki.
Pozdrawiam. |
|
|
![]() ![]() |
![]() |
Aktualny czas: 24.08.2025 - 10:12 |