Discussion :

[dubitoph]

Bonjour,

Dans la boucle "for" qui se trouve dans le code suivant, celle ci n'est pas réexécutée après l'instruction

Code :

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

eval(tab[cpt] + ".style.backgroundColor = \"#FFFFFF\"");

. L'instruction est exécutée, puis ça sort de ma boucle. Avant l'instruction, j'ai contrôlé mon "cpt" qui est bien à "0". Mais lorsque je mets un "alert()" après l'instruction, il ne s'affiche pas. Quelqu'un peut-il m'aider. Merci d'avance.

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
function verifTout()//vérification de tous les champs du formulaire
{
 var cpt = 0;
 var tab = new Array("document.commentaire.prenom.value","document.commentaire.nom.value");
 var index = 0;
 var chaine = "";
 var chaineEl = "";
 for(cpt; cpt < tab.length; cpt++)
  {
   if(verifChaine(eval(tab[cpt]))==1)
   {
    chaineEl1 = tab[cpt];
	index = chaineEl1.indexOf("value");
	chaine = tab[cpt].substring(0,(index - 1));
	eval(chaine + ".style.backgroundColor = \"#F3C200\"");
    return (false);
   }
   else
	 eval(tab[cpt] + ".style.backgroundColor = \"#FFFFFF\"");
  }
 if(verifMail(document.commentaire.email.value)==1)
 {
  document.commentaire.email.style.backgroundColor = "#F3C200";
  return (false);
 }
  else
  document.commentaire.email.style.backgroundColor = "#FFFFFF";
 if(verifCommentaire(document.commentaire.commentaireClient.value)==1)
 {
  document.commentaire.commentaireClient.style.backgroundColor = "#F3C200";
  return (false);
 }
  else
  document.commentaire.commentaireClient.style.backgroundColor = "#FFFFFF";
 return(true);
}

[Celelibi]

Bonjour,

Pour commencer, les eval c'est pas bien. Ça oblige à un reparsing, et c'est une très mauvaise idée de prendre l'habitude de les utiliser car ça ne peut exister que pour les langages interprêtés.

Donc moi je dirais un truc comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
[...]
var tab = new Array(document.commentaire.prenom, document.commentaire.nom);
[...]
  for (cpt; cpt < tab.length; cpt++) {
    if (verifChaine(tab[cpt].value) == 1) {
      tab[cpt].style.backgroundColor = "#F3C200";
      return (false);
    }
    else
      tab[cpt].style.backgroundColor = "#FFFFFF";
  }
[...]

[dubitoph]

C'est ok, merci.