Discussion :

[jmarey]

Bonjour,

je valide une série de boutons radio ds un formulaire pour qu'au moins un bouton soit coché et cela ne marche que sur le 1er bouton !
je pense que ce ne dois pas etre grand chose...si quelqu'un voit ou ca coince.
Merci
Alain

function ControlForm()
{
var NbFiches=document.Form.Accueil.length; // Form est le nom du formulaire et Accueil le nom des boutons

for (i=1;i<=NbFiches;i++)
{
if (document.Form.Accueil[i].checked==false)
{
alert("Oh les gars, faut cocher un choix\nQue diable !");return false;
}
return true;
}
}

[SpaceFrog]

vire les return

[Swoög]

Algorithme de base : "vérifier qu'au moins un" :

Code :

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

for(......) { if (condition sur en cours) { return true; } } return false;

[jmarey]

Apres modif, ouvre un mesage et me redirige automatiquement sur la page suivante.............désolé d'etre à ce niveau

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
 
function ControlForm()
{ 
var NbFiches=document.Form.Accueil.length; // Form est le nom du formulaire et Accueil le nom des boutons
 
for (i=1;i<=NbFiches;i++)
{ 
if (document.Form.Accueil[i].checked==false)
{
alert("Oh les gars, faut cocher un choix\nQue diable !");
} return true;
 
}
return false; 
}

[SpaceFrog]

j'avais pourtant l'impression de parler français ...
le return fait sortir de la fonction ...

[jmarey]

Oui, j'ai compris que le 1er return ne faisait sortir de la fonction, mais maintenant en enlevant ce return, plusieurs fenetres s'ouvrent avant de ma rediriger sur la page précédente.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
function ControlForm()
{ 
var NbFiches=document.Form.Accueil.length; // Form est le nom du formulaire et Accueil le nom des boutons
 
for (i=1;i<=NbFiches;i++)
	{ 
	if (document.Form.Accueil[i].checked==false)
		{
		alert("Oh les gars, faut cocher un choix\nQue diable !");
		} 
	} return false
}

en fait j'avais fait un bout de code simple genre

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
function ControlForm()
{
if 
(
(document.Form.Accueil[1].checked==false) && 
(document.Form.Accueil[2].checked==false)&& 
(document.Form.Accueil[3].checked==false)&& 
(document.Form.Accueil[4].checked==false)
)
 {alert("Oh les gars, faut cocher un choix\nQue diable !");return false;}
return true;
}

qui fonctionne bien, mais j'avais envie d'améliorer ce script

Merci de vos lumieres

Alain