boucle javascript

Question

Bonjour,
je voulais pour réduire mon code mettre une boucle dans ce qui suit

{

if ((document.forms[0].elements[0].checked==false)
&& (document.forms[0].elements[1].checked==false)
&& (document.forms[0].elements[2].checked==false)
&& (document.forms[0].elements[3].checked==false)
&& (document.forms[0].elements[4].checked==false)
&& (document.forms[0].elements[5].checked==false)
&& (document.forms[0].elements[6].checked==false)
&& (document.forms[0].elements[7].checked==false)
&& (document.forms[0].elements[8].checked==false)
&& (document.forms[0].elements[9].checked==false)
&& (document.forms[0].elements[10].checked==false)
&& (document.forms[0].elements[11].checked==false)
&& (document.forms[0].elements[12].checked==false)
&& (document.forms[0].elements[13].checked==false)
&& (document.forms[0].elements[14].checked==false)
&& (document.forms[0].elements[15].checked==false)
)
{alert("Merci de sélectionner au moins un atelier!");
return false;

}
else
{
document.forms[0].submit();
}
}

j'ai cru que c'était ca

<script language="javascript">

</script>

mais en fait ca marche pas donc j'ai pas du comprendre correctement

quelqu'un peut il m'aider
merci beaucoup

piplette

Electron · Answer

Bonjour, 
même si le code est un peu long, il est préférable à une boucle comme vous souhaitez la faire et dont le code exact est le suivant :
for(i = 0; i<16; i++) 
    		{ 
        		if(document.forms[0].elements[i].checked==false) 
        			{ 
						alert("Merci de sélectionner au moins un atelier!");
return false;
        			} 
else
{
document.forms[0].submit();
}
    		}
}
L'inconvénient est que cette boucle va vous générer autant de boites d'alertes qu'y aura de champs vide !
Cordialement.

Reivax962 · Answer

Electron,

Mettre un return dans une boucle for, c'est tout sauf propre. D'ailleurs votre boucle ne génèrera qu'une seule boite alert à cause de ce return.
Une syntaxe beaucoup plus claire, à mon avis, serait plutôt celle-ci :
var atelierChecked = false;
for(i = 0; i<16; i++)
	atelierChecked = atelierChecked || document.forms[0].elements[i].checked;
if (!atelierChecked)
{ 
	alert("Merci de sélectionner au moins un atelier!");
	return false;
} 
else
{
	document.forms[0].submit();
}Xavier

piplette · Answer

merci beaucoup pour tous ces conseils

Boucle javascript

3 réponses

Discussions similaires

Newsletters