Discussion :

[hind111]

je suis entrain de faire une vérification d'un formulaire si les champs sont vide et si le numéro de téléphone est un nombre j'ai fais cette fonction mais ça pas marché

Code :

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

1
2
3
4
5
6
7
8
9
10
 
function verifForm(contact) {
 
	if ((document.contact.nom.value == "") || (document.contact.prenom.value == "") || (document.contact.etablissement.value == "") || (document.contact.activite.value == "")|| (document.contact.tel.value == ""))
	{alert('Veuillez remplir tout les champs! Merci.');
	else(isNaN(document.contact.tel.value)
	   {alert('Veuillez saisir un numero de telephone valide');
 
	}
	}

merci d'avance

[vermine]

Re bonjour,

"contact", c'est le nom du formulaire ? Ce n'est pas nécessaire de le passer comme paramètre.

Ici, il manque qqch. Vous vouliez faire else if ?

Code :

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

1
2
else if(isNaN(document.contact.tel.value))

[E.Bzz]

Bonjour,
un peu comme dans l'exemple de la FAQ, quoi ...
Pour le n° de téléphone, regarde du côté des RegExp.

A+

[hind111]

merci pour la réponse
exactement "contact" c'est le nom du formulaire
maintenant j'ai fais:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
function verifForm(contact) {
 
	if ((document.contact.nom.value == "") || (document.contact.prenom.value == "") || (document.contact.etablissement.value == "") || (document.contact.activite.value == "")|| (document.contact.tel.value == ""))
	{alert('Veuillez remplir tout les champs! Merci.');
	else if(isNaN(document.contact.tel.value)
	   {alert('Veuillez saisir un numero de telephone valide');
 
	}
	}

et toujours ça marche pas

[Ryan D]

Je ne sais pas si c'est une erreur de copier/coller ou pas mais en plus du "else if" à la place du "else" il manque une accolade fermante dans ton premier "if"

[vermine]

Oui, merci et la parenthèse de fin du if déjà signalée plus haut.

[hind111]

j'ai ajouter une accolade

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
 
	<SCRIPT language="javascript">
 
function verifForm(contact) {
 
	if ((document.contact.nom.value == "") || (document.contact.prenom.value == "") || (document.contact.etablissement.value == "") || (document.contact.activite.value == "")|| (document.contact.tel.value == ""))
	{alert('Veuillez remplir tout les champs! Merci.');
	else if(isNaN(document.contact.tel.value)
	   {alert('Veuillez saisir un numero de telephone valide');
 
	}
	}
	}
 
</SCRIPT>

et toujours rien

[E.Bzz]

j'ai ajouter une accolade

Y avait 2 ou 3 autres trucs à lire entretemps

A+

[vermine]

J'indente votre code pour que vous visualisiez mieux:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
function verifForm(contact) 
{
     if ((document.contact.nom.value == "") || ...)
     {
          alert('Veuillez remplir tout les champs! Merci.');
     else if(isNaN(document.contact.tel.value)
     {
          alert('Veuillez saisir un numero de telephone valide');
     }
     }
}

Corrigez les erreurs et complètez les trous. (sauf les ..., c'était pour que ce soit plus visible).

[hind111]

pardon E.Bzz j'ai pas comprit ce que vous voulez me dire (je maitrise pas parfaitement le français)

[E.Bzz]

j'ai pas comprit ce que vous voulez me dire (je maitrise pas parfaitement le français)

Pas de problème : je te rappelais juste que je t'avais donné un lien vers la FAQ, où tu trouveras une fonction toute prête

un peu comme dans l'exemple de la FAQ, quoi ...
Pour le n° de téléphone, regarde du côté des RegExp.

A+

[hind111]

ah oui merci j'ai recherché dans la faq une fonction mais j'ai trouvé des fonctions un peu compliqué à ce que je cherche
quand je fait

Code :

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

1
2
3
4
5
 
	if ((document.contact.nom.value == "") || ....)
	{alert('Veuillez remplir tout les champs! Merci.');
	}
	}

sa marche bien mais quand j'ajoute le test du numéro ça marcha pas donc j'ai pas bien construit la fonction avec else...

[E.Bzz]

ah oui merci j'ai recherché dans la faq une fonction mais j'ai trouvé des fonctions un peu compliqué

Heu ... dans celle que je t'ai donné (clique sur le lien !) tu n'as que le nom du formulaire à modifier

Pour le n° de téléphone, il faudra un test spécifique (cf. RegExp).

A+

[vermine]

sa marche bien mais quand j'ajoute le test du numéro ça marcha pas donc j'ai pas bien construit la fonction avec else...

Oui oui et on vous a déjà dis les erreurs:

- Une accolade de fin "}" avant le else if.
- Une parenthèse de fin ")" de else if.
- Suppression d'une accolade de fin "}" que vous avez rajouter au mauvais endroit.

[E.Bzz]

@Vermine : la syntaxe n'est pas correcte (dépréciée). De plus, pas évolutif car à modifier en cas de nouveaux champs du form ...

Laisse-le tester la fonction de la FAQ : elle est générique

A+

[Ryan D]

Tu as une erreur dans la synthaxe de ta fonction, vérifie bien toute tes accolades !

Je te conseil tout de même d'aller jeter un coup d'oeil sur le lien que te donne E.Bzz... Car si un jour tu dois changer les champs de ton formulaire le script de la FAQ te facilitera les choses...

[hind111]

oui merci
vraiment vermine c'était la parenthèse de fin

[vermine]

hind111, permettez-moi également d'insister sur le code trouvé dans la FAQ. Je vous conseille de le tester et de l'adopter.

[hind111]

d'accord merci pour votre help