Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [PHP][OOP]Walidacja, Klasa walidujaca,
MateuszS
post
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.

  1. //wlasciwa klasa w moim nastepnym poscie w tym temacie ;P


Ten post edytował MateuszS 21.04.2010, 17:23:14
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
Crozin
post
Post #2





Grupa: Zarejestrowani
Postów: 6 476
Pomógł: 1306
Dołączył: 6.08.2006
Skąd: Kraków

Ostrzeżenie: (0%)
-----


Cytat
3. Mógłbyś wytłumaczyć o co chodzi z
Cytat
walidatory powinny być osobnymi obiektami implementującymi jakiś interfejs
Chodzi o to, że walidatory powinny być osobnymi obiektami wyspecjalizowanymi w walidacji konkretnego rodzaju. Daje to możliwość, bym utworzył sobie swój własny walidator, niewpisany na sztywno w kod Twojej klasy - innymi słowy: zwiększamy elastyczność narzędzia.

Tutaj masz przykładowy kod, który ilustruje jak mogło by to wyglądać (oczywiście brak tutaj dziesiątek rzeczy (ustawienia czytelnej treści błędu, wielu walidatorów dla jednego elementu, przedefiniowanych opcji dla walidatorów i całej masy innych rzeczy)):
  1. <?php
  2.  
  3. namespace Crozin\Validation {
  4. interface Validatable {
  5. public function configure(array $params = array());
  6. public function getOptions();
  7. public function getOption($key);
  8.  
  9. public function isValid($subject);
  10. }
  11.  
  12. class Validator {
  13. protected $subjects = array();
  14. protected $validators = array();
  15.  
  16. protected $errors = array();
  17. protected $data = array();
  18.  
  19. public function isValid() {
  20. $isValid = true;
  21.  
  22. foreach ($this->subjects as $key => $subject) {
  23. $result = $this->validators[$key]->isValid($subject);
  24.  
  25. if ($result instanceof Response) {
  26. $isValid = false;
  27. $this->errors[$key] = $result;
  28. } else {
  29. $this->data[$key] = $result;
  30. }
  31. }
  32.  
  33. return $isValid;
  34. }
  35.  
  36. public function addSubjects(array $subjects) {
  37. foreach ($subjects as $key => $subject) {
  38. $this->addSubject($key, $subject);
  39. }
  40. }
  41.  
  42. public function addSubject($key, $subject) {
  43. $this->subjects[$key] = $subject;
  44. }
  45.  
  46. public function addValidators(array $validators) {
  47. foreach ($validators as $key => $validator) {
  48. $this->addValidator($key, $validator);
  49. }
  50. }
  51.  
  52. public function addValidator($key, Validatable $validator) {
  53. $this->validators[$key] = $validator;
  54. }
  55.  
  56. public function getData() {
  57. return $this->data;
  58. }
  59.  
  60. public function getErrors() {
  61. return $this->errors;
  62. }
  63. }
  64.  
  65. class Response {
  66. protected $msg;
  67. protected $subject;
  68. protected $validator;
  69.  
  70. public function __construct($msg, $subject, Validatable $validator) {
  71. $this->msg = $msg;
  72. $this->subject = $subject;
  73. $this->validator = $validator;
  74. }
  75.  
  76. public function getMsg() {
  77. return $this->msg;
  78. }
  79.  
  80. public function getSubject() {
  81. return $this->subject;
  82. }
  83.  
  84. public function getValidator() {
  85. return $this->validator;
  86. }
  87. }
  88. }
  89.  
  90. namespace Crozin\Validation\Validator {
  91. use \Crozin\Validation\Validatable;
  92. use \Crozin\Validation\Response;
  93.  
  94. abstract class Base {
  95. protected $options = array();
  96.  
  97. public function __construct(array $options = array()) {
  98. $this->configure($options);
  99. }
  100.  
  101. public function configure(array $options = array()) {
  102. $this->options = $options;
  103. }
  104.  
  105. public function getOptions() {
  106. return $this->options;
  107. }
  108.  
  109. public function getOption($key) {
  110. return isset($this->options[$key]) ? $this->options[$key] : null;
  111. }
  112. }
  113.  
  114. class NIP extends Base implements Validatable {
  115. public function isValid($subject) {
  116. /* NIP ma nieprawidłowy format (np. podano litery) */
  117. if (1 == mt_rand(1, 2)) {
  118. return new Response('format', $subject, $this);
  119. }
  120.  
  121. /* NIP ma nieprawidłową sumę kontrolną */
  122. if (1 == mt_rand(1, 2)) {
  123. return new Response('control.sum', $subject, $this);
  124. }
  125.  
  126. return $subject;
  127. }
  128. }
  129.  
  130. class String extends Base implements Validatable {
  131. public function isValid($subject) {
  132. if ($this->getOption('trim')) {
  133. $subject = trim($subject);
  134. }
  135.  
  136. if (($min = $this->getOption('min')) && strlen($subject) < $min) {
  137. return new Response('min', $subject, $this);
  138. }
  139.  
  140. if (($max = $this->getOption('max')) && strlen($subject) > $max) {
  141. return new Response('max', $subject, $this);
  142. }
  143.  
  144. return $subject;
  145. }
  146. }
  147.  
  148. class Email extends Base implements Validatable {
  149. public function isValid($subject) {
  150. /* Nieprawidłowy format */
  151. if (1 == mt_rand(1, 2)) {
  152. return new Response('format', $subject, $this);
  153. }
  154.  
  155. /* Jeżeli wpisy MX nie są dobre */
  156. if (1 == mt_rand(1, 2)) {
  157. return new Response('mx', $subject, $this);
  158. }
  159.  
  160. return $subject;
  161. }
  162. }
  163.  
  164. class NumberRange extends Base implements Validatable {
  165. public function isValid($subject) {
  166. if ($this->getOption('natural.only') && false == ctype_digit($subject)) {
  167. return new Response('not.natural', $subject, $this);
  168. }
  169.  
  170. if ($subject < $this->getOption('min') || $subject > $this->getOption('max')) {
  171. return new Response('out.of.range', $subject, $this);
  172. }
  173.  
  174. return $subject;
  175. }
  176. }
  177. }
  178.  
  179. namespace {
  180. use \Crozin\Validation\Validator;
  181. use \Crozin\Validation;
  182.  
  183. error_reporting(E_ALL | E_STRICT);
  184.  
  185. $_POST = array(
  186. 'firstname' => ' Krzysiek ',
  187. 'nickname' => 'Crozin',
  188. 'age' => '7',
  189. 'mail' => 'k...s@gmail.com'
  190. );
  191.  
  192. $v = new Validator();
  193. $v->addSubjects($_POST);
  194. $v->addValidators(array(
  195. 'firstname' => new Validator\String(array('trim' => true, 'min' => 6)),
  196. 'nickname' => new Validator\String(),
  197. 'age' => new Validator\NumberRange(array('natural.only' => true, 'min' => 9, 'max' => 99)),
  198. 'mail' => new Validator\Email()
  199. ));
  200.  
  201. if ($v->isValid()) {
  202. var_dump('Poprawne dane:');
  203. var_dump($v->getData());
  204. } else {
  205. foreach ($v->getErrors() as $error) {
  206. var_dump(sprintf('%s - %s - %s - %s', get_class($error->getValidator()), $error->getSubject(), $error->getMsg(), print_r($error->getValidator()->getOptions(), 1)));
  207. }
  208. }
  209. }
Ale dlaczego to rozwiązanie jest o niebo lepsze? Chcę utworzyć nowy walidator... wystarczy mi obiekt dowolny, który implementuje interfejs \Crozin\Validation\Validatable; Chcę spersonalizować dany walidator? Podaję mu parametr. Itp. itd.

PS. Całość pisana "z palca" bez zastanowienia (jedynie sprawdziłem czy w ogóle działa) - więc jest to na pewno kod do poprawki (przemyślenia sposobu działania, przerobienia), ale daje jako taki zarys pewnej elastyczności.

Ten post edytował Crozin 21.04.2010, 22:03:19
Go to the top of the page
+Quote Post

Posty w temacie
- MateuszS   [PHP][OOP]Walidacja   21.04.2010, 15:58:09
- - Crozin   Dlaczego ten kod się do niczego nie nadaje? 1) P...   21.04.2010, 16:55:47
- - MateuszS   To tak: 1. Miałem tragiczny błąd w metodzie, nie p...   21.04.2010, 17:21:55
- - Spawnm   destruct masakryczny , echo w modelu ? Dodatkowo l...   21.04.2010, 17:29:51
- - Wicepsik   Walidacja NIPu nie polega tylko na policzeniu jego...   21.04.2010, 17:42:50
- - MateuszS   Spawnm, tak wiem, to nie jest problem zamienić na ...   21.04.2010, 18:24:03
- - Crozin   Cytat3. Mógłbyś wytłumaczyć o co chodzi z Cytatwa...   21.04.2010, 19:41:32
- - MateuszS   Niestety jest to dla mnie czarna magia co napisałe...   21.04.2010, 19:55:49
- - Crozin   CytatPoza tym wywala błędy z tym namespaceWidoczni...   21.04.2010, 20:17:58
- - MateuszS   CytatWidocznie masz PHP w wersji poniżej 5.3 - ale...   21.04.2010, 21:03:41
- - Crozin   CytatNo ja przecież napisałem skrypt który ma za z...   21.04.2010, 22:06:30
- - MateuszS   CytatRównież uważam, że branie się za FW w momenci...   22.04.2010, 11:51:06
- - Crozin   CytatNo nie przesadzajmy, podstawy pewne mamTego n...   22.04.2010, 13:30:24
- - zzeus   @Crozin mógłbyś rzucić kilka linków do artykułów k...   22.04.2010, 13:34:31
- - marcio   Cytat@Crozin mógłbyś rzucić kilka linków do artyku...   22.04.2010, 13:50:12
- - MateuszS   Crozin, to że nie użyłem w mojej klasie np. dziedz...   22.04.2010, 13:57:49
- - Crozin   CytatCrozin, to że nie użyłem w mojej klasie np. d...   22.04.2010, 16:50:25
- - MateuszS   No a jak wy się OOP uczyliście? Też mieliście taki...   22.04.2010, 19:31:53
- - marcio   Cytat(MateuszS @ 22.04.2010, 20:31:53...   22.04.2010, 20:50:39
- - MateuszS   He czyli głupie pytania kluczem do sukcesu To kol...   22.04.2010, 20:58:21
- - marcio   Taka ze dziedziczysz po klasie ktora rozszerza kla...   22.04.2010, 21:27:49
- - MateuszS   IMHO nie rozszyfrowałem skrótu acl. Mądrze piszesz...   22.04.2010, 21:52:07
- - Crozin   CytatJakis praktyczny przyklad masz komponent News...   23.04.2010, 01:02:49
- - marcio   Jak pisalem przyklady wymyslilem w momencie. Cytat...   23.04.2010, 09:29:00
- - MateuszS   Czyli jednak moja intuicja jeszcze ma się całkiem ...   23.04.2010, 13:12:58
- - Crozin   CytatKiedyś też tak myślałem ale doszedłem wtedy d...   23.04.2010, 13:57:22
- - MateuszS   Crozin, dało by się jakoś uprościć twoje klasy? Ta...   23.04.2010, 14:24:57
- - Crozin   Nie chcę już zaśmiecać wątku kolejnym długim listi...   23.04.2010, 16:42:30
- - marcio   CytatCóż... proszę rozwiń dlaczego uznałeś, że był...   23.04.2010, 17:09:22
- - MateuszS   Wielkie dzięki Crozin! To teraz pytania: 1. ...   23.04.2010, 17:31:27
- - smentek   Tablice mają to do siebie, że przechowują dane róż...   23.04.2010, 22:12:42
- - Crozin   CytatTwój walidator zwraca błędy (do tego poprzez ...   24.04.2010, 15:08:03
|- - smentek   Cytat(Crozin @ 24.04.2010, 16:08:03 )...   28.04.2010, 19:00:58
- - marcio   CytatTablice w swojej znanej z "normalnych...   24.04.2010, 17:57:00
|- - smentek   CytatTablice w swojej znanej z "normalnych...   28.04.2010, 18:43:24
- - MateuszS   Hmm... to ja może od nowa spróbuję napisać klasę, ...   24.04.2010, 22:54:01
- - Crozin   Skoro "filozofii", któregoś kawałka nie ...   25.04.2010, 00:34:00
- - MateuszS   Ok to tak, nie bardzo rozumiem jeszcze czemu używa...   25.04.2010, 08:03:55
- - Crozin   To zacznijmy od najłatwiejszego - przestrzeni nazw...   25.04.2010, 10:40:21
- - MateuszS   Albo jestem za głupi albo... właściwie to tylko to...   25.04.2010, 15:01:29
- - Crozin   JAk już napisałem... w PHP interfejsy nie grają is...   25.04.2010, 16:01:50
- - MateuszS   Ok, czyli nie wolno pojedynczemu obiektowi w tym p...   26.04.2010, 12:36:27
- - Crozin   Nie ma chyba jednoznacznej granicy. Po prostu... o...   26.04.2010, 14:34:22
- - MateuszS   Dzięki Crozin za cierpliwość (wiem że jej zasób ju...   27.04.2010, 19:11:54
- - Crozin   Cóż... możesz zawsze wrzucić kod - jeżeli już niko...   27.04.2010, 22:41:47
- - marcio   Ogolnie w jezykach takich jak Ruby/Python standart...   28.04.2010, 01:53:08
- - MateuszS   Jak najlepiej rozwiązać problem dodawania nowych o...   28.04.2010, 18:58:46
- - MateuszS   No dobra smentek, ale może wypowiedz się na temat,...   28.04.2010, 19:23:36
- - Crozin   CytatTablice w większości języków programowania mo...   28.04.2010, 20:12:27
- - MateuszS   No ok, ale Ty w swojej klasie zamieściłeś coś w st...   28.04.2010, 21:17:33
- - Crozin   To co ja zamieszczałem jako parametry to były właś...   28.04.2010, 21:20:40
- - smentek   Cytat(MateuszS @ 28.04.2010, 20:23:36...   4.05.2010, 15:19:32
- - Cysiaczek   Myślę, że iloczyn logiczny stanu walidatorów pozwa...   7.05.2010, 06:44:32


Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 8.10.2025 - 21:45