Au moins un checkbox obligatoire

Version imprimable

Bonjour j'ai fais une fonction qui doit vérifier si les checkbox du formulaire sont selectionné mais elle ne marche pas, elle ne renvoie rien :?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
function selection_obligatoire_checkbox(id,nb_page)
	{
	id_checkbox="id_page"+id;
	for(i=0;i<=nb_page;i++)
		{
		if(document.getElementById.id_checkbox[i].checked==true)
			{
			alert("Ok");
			}
		else
			{
			alert("Selection obligatoire!");
			}
		}
	}

Code:

1
2
3
4
5
6
7
8
9
 
<form action='rapport.php' method='post'>
 
<?php
echo "<input type='checkbox' name='id_page".$compteur_name_page."' value=".$num_page.">";
?>
<input type='submit' value='Générer un rapport' name="envoyer_rapport" OnClick="selection_obligatoire_checkbox($compteur_name_page,$nb_pages)"/>
 
</form>

14/09/2008, 17h50
SpaceFrog

http://www.developpez.net/forums/d55...on-formulaire/

Meme s'il y a pas beaucoup d'infos sur les checkbox, j'ai essayé les fonctions données dans le lien et ça ne marche toujours pas :

Code:

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
 
function VerifForm(form)
{ 
var AllIsOk=0;
 
for (var i=0; i<form.elements.length; i++) 
	{
	var inp = form.elements[i]; //on recupere le champ dans une variable referente, cela evite de se trimballer avec 12ko de code :D
	if (inp.className.match(/\brequired\b/)) 
		{ //on check si l'élément a la classe required.
		switch(inp.type) { // s'il a la classe on regarde son type et on sort une erreur en fonction de son type
			case "text":  
			case "file":
			case "password":
			case "textarea":
			if (inp.value=="") AllIsOk++;
			break;
			case "checkbox":
			case "radio":
			if (!inp.checked) AllIsOk++;
			break;
			case "select-one":
			case "select-multiple":
			if(inp.selectedIndex==0) AllIsOk++;
			break;
			}
		}
	}
return (AllIsOk++)
}

Code:

1
2
3
4
5
6
 
<form name="myform" action="javascript:alert('Envoyé')" onsubmit="return VerifForm(this)" />
 
echo "<input type='checkbox' class='required' name='id_page".$compteur_name_page."' value=".$num_page.">";
 
<input type='submit' value='Générer un rapport' name="envoyer_rapport"/>

J'ai l'alerte "Envoyé" alors que j'ai rien coché..

Oui, tu renvoies 0 ou 1 alors qu'il faut retourner false pour empêcher le submit...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
function VerifForm(form)
{ 
var AllIsOk=true;
 
for (var i=0; i<form.elements.length; i++) 
	{
	var inp = form.elements[i]; //on recupere le champ dans une variable referente, cela evite de se trimballer avec 12ko de code :D
	if (inp.className.match(/\brequired\b/)) 
		{ //on check si l'élément a la classe required.
		switch(inp.type) { // s'il a la classe on regarde son type et on sort une erreur en fonction de son type
			case "text":  
			case "file":
			case "password":
			case "textarea":
			if (inp.value=="") AllIsOk=false;
			break;
			case "checkbox":
			case "radio":
			if (!inp.checked) AllIsOk=false;

Dans ce cas le formulaire n'envoie aucun message, rien se passe..

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
function VerifForm(form)
{ 
var AllIsOk=true;
 
for (var i=0; i<form.elements.length; i++) 
	{
	var inp = form.elements[i]; //on recupere le champ dans une variable referente, cela evite de se trimballer avec 12ko de code :D
	if (inp.className.match(/\brequired\b/)) 
		{ //on check si l'élément a la classe required.
		switch(inp.type) 
			{ // s'il a la classe on regarde son type et on sort une erreur en fonction de son type
			break;
			case "checkbox":
			case "radio":
			if (!inp.checked) AllIsOk=false;
			break;
			}
		}
	}
return (AllIsOk++)
}

15/09/2008, 11h51
Bovino

Code:

return (AllIsOk++)

???
Ce serait plutôt :

Code:

return (AllIsOk)

Mais pourquoi ça marche pas!!! :arf:

j'ai le message "ok" dans les deux cas (coché ou pas coché) :evilred:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
function VerifForm(form)
{ 
var AllIsOk=true;
 
for (var i=0; i<form.elements.length; i++) 
	{
	var inp = form.elements[i]; //on recupere le champ dans une variable referente, cela evite de se trimballer avec 12ko de code :D
	if (inp.className.match(/\brequired\b/)) 
		{ //on check si l'élément a la classe required.
		switch(inp.type) 
			{ // s'il a la classe on regarde son type et on sort une erreur en fonction de son type
			break;
			case "checkbox":
			case "radio":
			if (!inp.checked) AllIsOk=false;
			break;
			}
		}
	}
return (AllIsOk)
}

Code:

1
2
3
4
5
6
7
8
9
10
 
 
<form name="myform" action="javascript:alert(ok)" onsubmit="return VerifForm(this)" />
 
// Nombre de checkbox
echo "<input type='hidden' name='nombrepage' value='$nb_pages'>";
 
echo "<input type='checkbox' class='required' name='id_page".$compteur_name_page."' value=".$num_page.">";
 
<input type='submit' value='Générer un rapport' name="envoyer_rapport"/>

Code:

1
2
3
4
5
6
7
8
9
switch(inp.type) 
			{ // s'il a la classe on regarde son type et on sort une erreur en fonction de son type
			break;
			case "checkbox":
			case "radio":
			if (!inp.checked) AllIsOk=false;
			break;
			}
		}

il y a un break un peu prématuré dans ton switch...

Code:

1
2
3
4
5
6
7
8
switch(inp.type) 
			{ 
			case "checkbox":
			case "radio":
			if (!inp.checked) AllIsOk=false;
			break;
			}
		}

15/09/2008, 15h51
bassiste

Ce code il est nul 8O