[PHP5] - CLASSE DE VÉRIFICATION DE FORMULAIRE

Commentaire de webdeb le 02/12/2007 23:54:25

Je pense que tu aurais pu faire une classe dédiée pour les règles et les filtres. Et faire par exemple :

$oFH = new formHandler ($_POST, $_FILES);

$rule = new FormRule('login');
$rule->setIsRequired(true);
$rule->setMinLength(8);
$rule->setMaxLength(15);
...

$oFH->addRule($rule);

++

Commentaire de Zephyr11 le 03/12/2007 12:14:29

Débutant et donc incapable de juger objectivement, mais ce code est parfaitement en phase avec ce dont j'avais besoin. Merci !

Commentaire de webdeb le 03/12/2007 13:21:27

>> Sinon le point qui me chagrine à propos de cette méthode, c'est la validation :
>> On peux faire un $oFH->validate () mais ca va devoir appeler en boucle les classes filles pour voir comment s'est passé les règles.
>> Une idée à propos de ca ?

Nan je ne vois pas où est le problème. On peut améliorer mon exemple comme ceci en fait :

$oFH = new formHandler($_POST, $_FILES);

$oRule = new FormRule();
$oRule->setIsRequired(true);
$oRule->setMinLength(8);
$oRule->setMaxLength(15);
...
$oFH->addNewRule('login', $rule); // Où login est le nom du champ à traiter

Dans ta classe FormHandler, tu as un attribut privé qui est un tableau associatif des règles. Ce qui donne :

class FormHandler()
{
    private $_aRules = array();

    public function addNewRule($sFormField, $oFormRule)
    {
        $this->_aRules[ $sFormField ] = $oFormRule;
    }
}

Donc tu vas stocker les objets de type FormRule dans ton tableau associatif de ta classe FormHandler. Ainsi, dans ta méthode qui valide les données tu peux directement accéder aux informations des objets FormRule de cette manière :

public function validate()
{
    $oRule = null;

    // On parcourt la copie du tableau $_POST (j'admets que tu fonctionne ainsi)
    foreach($aFormFields as $fieldName => $fieldValue)
    {
        // On récupère l'objet rule correspondant à chaque champ
        if(!empty($this->_aRules[ $fieldName ]))
        {
           $oRule = $this->_aRules[ $fieldName ];

           // On vérifie chaque règle
           if(true === $oRule->getIsRequired)
           {
       // On teste la valeur
               if(empty($fieldValue))
   {
                  throw new FormHandlerException('Le champ '. $fieldName .' ne peut rester vide !');
   }
    // On teste la longueur min
               if(strlen($fieldValue) < $oRule->getMinLength())
               {
     throw new FormHandlerException('Le champ '. $fieldName .' doit avoir une valeur minimale de '. $oRule->getMinLength() .' caractères !');
   }
  
   // On teste la longueur max
               if(strlen($fieldValue) > $oRule->getMaxLength())
   {
                  throw new FormHandlerException('Le champ '. $fieldName .' doit avoir une valeur maximale de '. $oRule->getMaxLength() .' caractères !');
               }

           }
        }
    }
}

Tu vois globalement comment tu pourrais procéder ?

++

Hugo.

Commentaire de codefalse le 03/12/2007 14:11:20 administrateur CS

euh perso la deuxieme fois c'est pire ;)
Je vais voir ce que je peux faire la dessus, je ferai une mise à jours prochainement, je vous tiens au courant :)

Merci en tout cas :)

Commentaire de codefalse le 07/12/2007 04:02:27 administrateur CS

Alors voila Webdeb ! j'ai mis la source à jour, avec la modification du code comme prévu.
Dit moi ce que tu en pense :)

Commentaire de codefalse le 07/12/2007 13:51:21 administrateur CS

J'y ai pensé à cela, mais le truc, c'est que je veux vraiment être indépendant du code html derriere (pourquio obligatoirement du code html ? pourquoi pas autre chose ?).
Si je voulais utiliser du code html, j'aurai pris le pacakage PEAR HTML_QUICKFORM2 (je me suis basé sur leur idée pour le formualaire).
Il est tres complet et plutot agréable à utiliser.
Sauf que je ne veux QUE analyser le formulaire, de facon à être libre derriere de la mise en place de mon formulaire (ca devient vite le bordel quand on veux mettre des css particuliers, une mise en page originale, car la classe génére un html précis).
Bon, je peux toujours sortir un xml qui sera interprété par la suite en xslt. Mais perso je préfére faire le code html par un template ou quelque chose du genre :)

Apres rien ne t'oblige à le faire ;)

Commentaire de codefalse le 13/12/2007 13:29:47 administrateur CS

qu'est-ce que tu veux tester dans la liste exactement ?

Commentaire de psykoboy57 le 14/12/2007 04:24:41

Salut, encore moi

Pour ce qui est des listes simples, j'ai fait une fonction isDifferent sur la base de isEqual en mettant l'option
<option value="null">------------</option>
C'est un peu du bricolage mais ca marche.

Par contre pour les listes multiples, tjs rien.
Si je met le nom "animaux[]" en nom de regle, il ne me trouve pas l'objet ! Une idee ?

$oWeb = new formRule ('web');
$oWeb->setIsUrl ('l\'Url de votre site est invalide !');

Ne semble pas marcher. Ne peut-on pas valider un email s'il n'est pas obligatoire ? (il me met un email invalide meme si le champs n'est pas rempli)

Merci encore pour ta classe.

Commentaire de codefalse le 14/12/2007 09:47:13 administrateur CS

"Je me doute que ca a aucun rapport avec ta classe mais mettre value="null" marche-t-il ? est ce considerer comme "null" ou null" Je pense que ce sera considéré comme "null" et pas comme quelque chose de vide.

"Ne peut-on pas valider un email s'il n'est pas obligatoire ? (il me met un email invalide meme si le champs n'est pas rempli)" => Je vais voir ca

"$oWeb = new formRule ('web');
$oWeb->setIsUrl ('l\'Url de votre site est invalide !');

Ne semble pas marcher." => Je vais voir ca aussi !
Pour l'url, ca marche chez moi. Quelle url tu a mis qui ne fonctionne pas ?

Pour ta liste à choix multiple, tu veux faire quoi ? savoir si une valeur au moin à été spécificiée ? si oui tu peux faire

$oRule = new formRule ('animaux.0');
$oRule->setIsRequired ('Au moin un choix doit être effectué !');

Ca marche. En fait, le . dans le formRule, spécifie un tableau, un elt1.elt2.etl3 correspond à elt1[elt2[elt3]]; Donc animaux.0 correspond à animaux[0]
Tu peux aussi faire un $oGenre->setIsNotEqual ('---------', 'Veuillez choisir un élément valide !'); Pour la premiere liste histoire d'obliger de choisir un autre élément.

Merci beaucoup ! c'est grâce à des gens comme toi que les codes évoluent :)

Commentaire de codefalse le 14/12/2007 19:46:01 administrateur CS

tu a bien mis pref.0 (zéro) ?

Ca marche chez moi ! ?! :p

Commentaire de psykoboy57 le 14/12/2007 20:48:57

Ok en fait si je fait un print_r de mon tableau POST lors de la validation du sondage, je vois que le champ "pref" n'apparait pas.
Apres lui avoir mis une valeur selesctionnee par defaut, ca marche...

Mais j'aimerais eviter ! Je ne comprends vraiment pas pourquoi ca marche chez toi et pas chez moi. peux tu tester avec le bout de formulaire suivant :
// faites pas attention a l'id du select, je pense pas que ca joue et en plus il est pas beau
<select size="9"  MULTIPLE="MULTIPLE" id="pref[]" name="pref[]">
<option value="francaise">Chansons francaise</option>
<option value="electro">Electro</option>
<option value="rap">Rap</option>
<option value="reaggae">Reaggae</option>
<option value="autre">Autre</option>
</select>

et la verif suivante :
  $fPref = new formRule('pref.0');
  $fPref->setIsRequired('Donnes nous tes preferences musicales !');

Je comprends vraiment pas

Commentaire de codefalse le 14/12/2007 21:20:58 administrateur CS

Note : je ferai une mise à jour prochainement !

Commentaire de webdeb le 14/12/2007 22:42:12

Salut Codefalse,

Si ça t'intéresse, je viens de publier sur mon site une classe permettant de valider un numéro de compte bancaire français. Tu pourrais intégrer ces vérifications je pense dans la classe de règle.

Tout est ici : http://www.apprendre-php.com/tutoriels/tutoriel-32-classe-de-controle-et-de-validation-de-rib-francais.html

Je n'ai pas réussi à la déposer sur PHPCS car visiblement l'envoi de sources déconne en ce moment :(

++

Hugo.

Commentaire de webdeb le 14/12/2007 22:59:55

Pas de quoi :)

Je trouvai que mon code avait sa place avec le tien ;)

Commentaire de codefalse le 15/12/2007 09:51:05 administrateur CS

exact, là ya un sérieux probleme ...
je vais me pencher dessus !

Commentaire de psykoboy57 le 15/12/2007 21:52:40

sauf le mail non obligatoire que j'ai pas regarde

Commentaire de psykoboy57 le 19/12/2007 01:17:53

des news par ici ? :)

Commentaire de codefalse le 19/12/2007 14:51:34 administrateur CS

Bon, ca y est c'est mis à jour :)
Donc par contre dans le zip il n'y a pas le package genericException, faites attentions ! :)
Vous pourrez le trouver sur mes autres classes ! :p

@Webdeb : je t'ai pas oublié pour ta classe de Rib, mais je suis bloqué sur le comment faire !! :/

Commentaire de webdeb le 19/12/2007 15:26:45

@codefalse : je réfléchis à une solution dans la soirée et je te tiens informé.

++

Commentaire de codefalse le 24/01/2009 11:58:00 administrateur CS

Une des règles du jeu de la POO est de ne jamais mettre d'attributs en public, toujours en private/protected. Pourquoi ?

Imagine ceci :
class MyMath {
   public $_iNum1;
   public $_iNum2;

   public function multiply () {
      return $this->_iNum1 * $this->_iNum2;
   }
}

$o = new MyMath ();
$o->_iNum1 = 'salut';
$o->_iNum2 = 5;
echo $o->multiply ();

Et là, joli plantage !

Maintenant, si tu fait de cette manière :
class MyMath {
   private $_iNum1;
   private $_iNum2;

   public function multiply () {
      return $this->_iNum1 * $this->_iNum2;
   }

   public function __set ($myVar, $myValue) {
      if (!is_int ($myValue))
          throw new Exception ('second parameter must be an Integer !');

      $this->$myVar = $myValue;
   }
}

Du coup, si la personne décide de mettre n'importe quoi sauf du int, là, ca ne passera pas ! C'est une sécurité !

Bon, sur mon exemple j'aurai juste trois points à commenter :
   _ L'utilité de mon exemple est fort contestable dans le fait que des fonctions de type sum, multiply, square, prennent directement les valeurs à travailler en paramètre. Mais c'est un exemple pour montrer le pourquoi du comment des attributs privés/publiques. On peux aussi trouver un domaine d'application ou le multiply ne prendrais pas de valeurs :p
   _ En POO global (sans tenir compte des spécificités du langage php), on aurait mis des getter/setters pour chaque variables (setNum1, getNum1 et setNum2, getNum2), mais vu que php propose la méthode __get/__set, autant en profiter (je n'ai pas mis __get car inutile dans cet exemple)
   _ Je ne fait pas de vérification sur $myVar dans la méthode __set existe, ca devra donc etre _iNum1 ou _iNum2, sinon, ca servira à rien (voir ca risque de générer des erreurs). Encore une fois, ce n'est qu'un exemple, donc je ne l'ai pas amélioré !

J'espère maintenant que tu vois mieux pourquoi il faut éviter de mettre des publics dans une classe :)

Commentaire de marciv le 24/01/2009 18:29:54

Merci codefalse de tes précisions, peut être qu'un jour les classes n'auront plus de secrets pour moi :-) ce jour étant loin je précise :

mon problème n'est pas exactement de voir un si un email à été renseigné mais de faire un test spécifique en fonction de la valeur d'un champs du formulaire,
exemple
formulaire:
<select style="background-color:#D6DEF5;" name="devis_type_de_bien">
<option>-----</option>
<option  value="maison">Maison</option>
<option  value="appartement">Appartement</option>
<option  value="autre">Autre</option>
</select>

Si autre, précisez :
<input name="devis_type_de_bien_autre" value="" maxlength="30" size="30" type="text">

donc j'ai besoin d'un truc du genre :

if (($devis_type_de_bien->_aElementValue=="autre")) {
   $devis_type_de_bien_autre->setIsRequired ('Veuillez indiquer des précisions sur le type de bien !');
   $devis_type_de_bien_autre->setMinLength (3, 'Vos précisions sur le type de bien doivent comporter plus de 3 caractères (<em>Exemple : parking ouvert, commerce rdc, box etc...)</em> !');
}

Je n'ai pas trouvé comment faire autrement...

j'ai bien essayé de rajouter un truc du genre :
public function __getVal() {
return ($this->aElementValue);
}
avec
test sur $devis_type_de_bien_autre->__getVal()
mais ça marche pas

PS au fait quand tu fait le test :
(is_object ($devis_type_de_bien_autre) && $devis_type_de_bien_autre instanceof formRule)
le retour est toujours true que le champs du formulaire soit renseigné ou pas et quand je print j'ai toujours ça :

formRule Object ( [_sDelimiter:protected] => . [_aElementValue] => [_iCountErrors:protected] => 0 [_aErrors:protected] => Array ( ) [_aRules:protected] => Array ( ) )

En tout cas merci de tes précisions et de ta classe qui ne quitte plus mes site :-)

Commentaire de codefalse le 25/01/2009 14:39:59 administrateur CS

à la place du return false, il faisait quoi ? s'il retournait null, fallait tester if (is_null ($tonretour));

pour la condition, essaye plutot :
if (!empty ($this->pro))
   return $this->pro;
else
   return false; // null ?

Ca fait plaisir de voir que ma classe est utilisée sur tes sites :) vraiment :)