Discussion :

[MikeV]

Bonjour,
Par soucis de faire un code propre, j'aimerais trouver une façon de faire cette vérification if dans ma fonction verif_formulaire pour chacun de mes champs. Pour l'instant je vérifie seulement mon champs poste_vise avec cette fonction, mais elle pourrais aussi servir aux champs prénom, nom, adresse etc.

Donc je ne voudrais pas avoir 15 if dans ma fonction...est-ce qu'il y a un moyen d'utiliser 1 if pour tous mes champs?

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
function verif_formulaire(){
 
		var poste = document.getElementById('poste_vise').value;
 
		if( (poste.length < 4) || 
			(poste.indexOf("http") != -1) ||
			(poste.indexOf(">") != -1) ||
			(poste.indexOf("http") != -1) ||
			(poste.indexOf("HTTP") != -1) ){
			alert("Le champs 'poste visé' est invalide");
			document.form1.poste_vise.style.backgroundColor = "#F3C200";
			document.form1.poste_vise.focus();
		   	return false;
		   }else{
			document.form1.poste_vise.style.backgroundColor = "#FFFFFF";
		   }
 
 
	}

Merci !

[E.Bzz]

Bonjour,

Code :

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

1
2
3
function verif_formulaire(champ_verif){
		var poste = document.getElementById(champ_verif).value;
		...

Non ?
Bien sûr il faut que les tests à faire soit les mêmes pour tout les champs ...

A+

[Tellounet]

Peut-être pourrais-tu essayer de faire ça dynamiquement en mettant des id à tes champs?

edit: grillé par la fraîcheur ^^ oui, évidemment il faut que les tests à faire soient les mêmes, ou bien passer la liste des champs et des tests dans un tableau, et boucler

[MikeV]

Bon j'ai essayer quelque chose comme ca :

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
function verif_formulaire(){
 
		thenames = new Array("poste_vise", "prenom", "nom");
 
		for (var i = 0; i < thenames.length; i++){
    		alert("boucle for");
			if( document.form1.thenames[i].value == "" ){
			alert("test");
			document.form1.thenames[i].backgroundColor = "#F3C200";
			document.form1.thenames[i].focus 
		   	return false;
		    }else{
			document.form1.thenames[i].backgroundColor = "#FFFFFF";
		   }
 
		}
 
 
	}

j'ai 2 problèmes :
- mon alert "boucle for" ne s'affiche qu'une seule fois malgré qu'il y a 3 éléments dans "thenames"
- rien ne passe dans mon if (absence d'affichage de l'alert test)

..je continues à l'améliorer, je sens que j'y arriverai, d'ici là si quelqu'un peut maider, ca serait TRÈS apprécié

[Tellounet]

Bonjour,


A tout hasard essaie de remplacer ton tableau par celui-là:

Code :

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

thenames = new Array(poste_vise, prenom, nom);

Les guillemets indiquent une string et je ne pense pas qu'il soit possible de retravailler dessus de cette manière

[E.Bzz]

Les guillemets indiquent une string et je ne pense pas qu'il soit possible de retravailler dessus de cette manière

Non, effectivement, mais pas à cause des string

Code :

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

1
2
document.forms["form1"].elements[thenames[i]].style.backgroundColor = "#F3C200";
document.forms["form1"].elements[thenames[i]].focus();

A+