Verification bouton radio JAVASCRIPT

Résolu
sluggogle -  
 Pseudo ... -
Bonjour,

je cherche a verifier si un choix a été fait sur un bouton radio. Seulement la condition n'est pas vérifiée.

// ------ VERIFICATION FREQUENCE CONTRAT ----
for(i=0;i<3;i++)
{
if(document.formulaire.frequence[i].checked == true)
{
frequence = 1;
}
else
{
frequence = 0;
}
}
if(frequence == 0)
{alert("Veuillez choisir la frequence des visites !");
document.formulaire.frequence[i].focus();
return false;}

je ne vois pas ce qui cloche, normalement, si aucun bouton choisit (checked retourne toujours 'false' donc 'frequence = 0') je dois avoir mon message d'alert. Sinon c'est ook
A voir également:

4 réponses

Arsenic33 Messages postés 46 Date d'inscription   Statut Membre Dernière intervention   42
 
S'il vous plaît apprenez à coder proprement, c'est pour votre bien ;)

for(i = 0; i < document.formulaire.frequence.length; i++){
	if(document.formulaire.frequence[i].checked) return true;
}
alert("Veuillez choisir la fréquence des visites !"); 
return false;


La même chose en 5 lignes indentées sans variables inutiles
37
keime
 
je n'aime pas trop cette ligne
if(document.formulaire.frequence[i].checked) return true;

si tu veux jouer au mec qui sait coder
return document.formulaire.frequence[i].checked;
0
jc
 
gnié? as-tu au moins essayé ton code keime?
0
Yuki
 
keime -> effectivement, faire un if(x==true) return true; else return false; est une erreur de débutant. Mais pas la peine de vouloir faire celui qui sait coder quand il ne sait pas lire du code. Ton truc renvoie true ou false dès la première itération de la boucle et arrête la fonction. Dès lors, l'alert ne passe même plus dans le cas où tout est faux et si jamais le premier élément est faux mais qu'il y en a un vrai par la suite, ton truc renverra quand même faux.
0
sweety
 
@keime -> pourquoi c'est une erreur de débutant de mettre des if et des else ? Mois je trouve ça tout simplement plus propre et lisible!
0
coucou
 
moi je pense qu'on peut retirer les crochets si on veut...
0