Discussion :

[Jiraiya42]

Salut !

J'essaie de mettre en place une vérification javascript pour chaque champ seulement le code devient vite lourd quand le formulaire devient grand. Et surtout cela rend le script moins réutilisable

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
 
function couleur(obj) {
     obj.style.backgroundColor = "#FFFFFF";
}
 
function check() {
	var msg = "";
 
		if (document.formulaire.mail.value != "")	{
		indexAroba = document.formulaire.mail.value.indexOf('@');
		indexPoint = document.formulaire.mail.value.indexOf('.');
		if ((indexAroba < 0) || (indexPoint < 0))		{
		document.formulaire.mail.style.backgroundColor = "red";
			msg += "Le mail est incorrect\n";
		}
	}
	else	{
		document.formulaire.mail.style.backgroundColor = "red";
		msg += "Veuillez saisir votre mail.\n";
	}
 
if (document.formulaire.nom.value == "")	{
		msg += "Veuillez saisir votre nom\n";
		document.formulaire.nom.style.backgroundColor = "red";
	}
 
if (document.formulaire.prenom.value == "")	{
		msg += "Veuillez saisir votre prenom\n";
		document.formulaire.prenom.style.backgroundColor = "red";
	}
 
	if (msg == "") return(true);
	else	{
		alert(msg);
		return(false);
	}
}

Les if se succèdent... Serait-il possible de faire en sorte de passer le nom du champ en paramètre dans une boucle ? Pour qu'il y ait une boucle avec un seul if ? En php je saurais faire mais en javascript je patauge !

Quelqu'un peut m'aider svp ?

Merci d'avance...

[bigboomshakala]

salut. tu peux doter chaque élément de ton formulaire d'une fonction check que tu invoques quand tu veux valider le formulaire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
monInputText.check = function() { /*action*/ };
...
function checkGlobale()
{
  //pour tous les items du formulaires
  item.check();
}

[m312]

Les if se succèdent... Serait-il possible de faire en sorte de passer le nom du champ en paramètre dans une boucle ? Pour qu'il y ait une boucle avec un seul if ? En php je saurais faire mais en javascript je patauge !

Quelqu'un peut m'aider svp ?

Merci d'avance...

Y'a moyen aussi de passer par document.formulaire.elements[n], et donc de faire une boucle (puisque tes input ont l'air d'avoir des noms sympathiques )

un truc du genre

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
function check() {
var msg = "";
var total = document.formulaire.length;
for (i=0;i<total;i++) {
if ( document.formulaire.elements[i].value == "") {
msg += "Veuillez saisir votre "+document.forumlaire.elements[i].name+"\n";
document.formulaire.elements[i].backgroundColor = "red";
}
}

Aux erreurs de point virgule pres

Bien entendu ca marche que pour verifier si c'est vide ...

[buzzkaido]

Perso, j'avais fait une fonction generique, du genre checkField :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
function checkField(idDuChamp, Format)
{
    // on test si le champ respecte l'expression reguliere du format
    // si c'est ok => on renvoit true
 
    // sinon :
    alert("Le champ " + idDuChamp + " ne respecte pas le format attendu : " + Format);
    document.getElementById(idDuChamp).focus();
}

Avec comme parametres :
- idDuChamp = id de l'input
- Format = format du champ

"Format" est en fait la clé d'un tableau d'expression regulieres :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
tableauFormat = Array();
tableauFormat['chaine'] = expression reguliere
tableauFormat['chiffre'] = expression reguliere
tableauFormat['email'] = expression reguliere
....

Si l'input verifie l'expression reguliere, on renvoit true, sinon false.

Comme ça, il suffit de faire onSubmit="verifierFormulaire();"

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
function verifierFormulaire()
{
   if (!checkField("nom", "chaine")) return false;
   if (!checkField("prenom", "chaine")) return false;
   if (!checkField("age", "chiffre")) return false;
   ....
   return true;
}