SICKofitALL misanthrope | non, je voyais plus un truc plus générique du style :
Code :
- var test = fonction ()
- {
- if (!this.Pseudo) {
- return "Merci de renseigner un pseudonyme.";
- }
- if (!this.Pass) {
- return "Merci de renseigner un mot de passe.";
- }
- if (!this.Mail) {
- return "Merci d'indiquer votre adresse e-mail.";
- }
- if (!this.Mail2) {
- return "Merci d'indiquer votre adresse e-mail une seconde fois.";
- }
- if (!this.Nom) {
- return "Merci de donner un nom à votre chien.";
- }
- if (!this.NomP) {
- return "Merci de renseigner votre nom de famille.";
- }
- if (!this.Prenom) {
- return "Merci de renseigner votre prénom.";
- }
- return false;
- };
- // la liste de tes variables sous forme d'objet
- var aTester = {
- Pseudo: null,
- Pass: null,
- Mail: null,
- Mail2: null,
- Nom: null,
- NomP: null,
- Prenom: null
- },
- texte;
- // ... tu initialises tes variables du genre :
- // aTester.Pseudo = monFormulaire.Pseudo.value;
- //
- // Pour peu que tes variables ont le même nom que tes champs dans ton formulaire, tu pourrais faire en une seule ligne :
- // for (var f in aTester) { aTester[f] = monFormulaire[f].value; }
- // tu appelles la fonction qui test la validité de tes champs, avec comme scope l'objet aTester (donc "this" = "aTester" )
- texte = test.call (aTester);
- if (texte) {
- return alert (texte); // texte est considéré comme true, donc non vide -> on affiche et on quitte.
- }
|
J'ai pas testé avec ce code spécifiquement, mais c'est comme ca perso que je ferais sans imbriquer des if et des else en cascade. En plus je peux rajouter et virer des champs à tester sans trop de grosses modifs. ---------------
We deserve everything that's coming...
|