Discussion :

[bressan]

Bonjour,

Je tente de savoit si aucune de mes chekbox n'est cochée. Le nombre de checkbox étant variable.

Voilà la syntaxe que j'utilise mais il y a un hic dans mon if, malheureusement, je ne vois pas d'ou ça vient. Si quelqu'un peut m'aider ce sera sympa.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
function validation ()
{
	var test_nb = 0;
	for (i = 1; i <= <? echo ($nb_cri); ?>; i++)
	{
		if (document.form.enreg.tab_add_cri[i].checked==false)
		{
			test_nb++;
		}
	}
	alert ("test=" + test_nb);
}

Le nom de mon formulaire est enreg et je nomme mes checkbox comme suit : name="tab_add_cri[0]", name="tab_add_cri[1]" ...

Puisque le nombre de checkbox n'est pas fixe, $nb_cri correspond au nombre de checkbox.

Merci d'avance de votre aide.

[yjuliet]

Salut.

Juste comme ça, tu expliques que tu nommes tes checkboxes

Code javascript :

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

tab_add_cri[0 .. (n-1)]

et que ta boucle for est écrite

Code javascript :

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

for (i = 1; i <= n ; i++) {...}

Tu ne testes donc jamais la case tab_add[0] mais par contre, tu essayes systématiquement d'accéder à la case tab_add[n] ce qui risque d'entrainer un plantage du script.

Tu n'as pas réellement besoin du paramètre PHP pour récupérer le nombre de cases, puisqu'en JavaScript, tu as la propriété length d'un tableau. Tu pourrais donc faire quelquechose comme :

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
11
12
function validation ()
{
	var test_nb = 0;
	for (i = 0; i <= document.form.enreg.tab_add_cri.length ; i++)
	{
		if (document.form.enreg.tab_add_cri[i].checked==false)
		{
			test_nb++;
		}
	}
	alert ("test=" + test_nb);
}

[javatwister]

soit, peut-être:

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
<form id="f" action="" onsubmit="return verif(this.elements['box[]'])" />
 
<div>
<input type="checkbox" name="box[]" value="box0" />truc<br />
<input type="checkbox" name="box[]" value="box1" />truc<br />
<input type="checkbox" name="box[]" value="box2" />truc<br />
<input type="checkbox" name="box[]" value="box3" />truc<br />
</div>
<p>
<input type="submit" />
</p>
</form>
 
<script type="text/javascript">
 
 
function verif(b){
	for(i=0;i<b.length;i++){
		if(b[i].checked==true){
			return true
		}
	}
	return false
}
</script>

[yjuliet]

à part le

Code :

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

this.elements['box[]']

que j'aurais eu tendance à passer différemment :

Code :

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

this.elements['box']

, mais dont je ne suis pas certain, je pense que ça devrait ressembler à ça ...

[javatwister]

en fait, comme php reçoit un tableau "box", il est nécessaire de nommer les checkbox ainsi pour récupérer les données;

[bressan]

Merci des réponses, je vais voir ce que je peux faire avec ça.
Je vous tiens au courant.

[yjuliet]

en fait, comme php reçoit un tableau "box", il est nécessaire de nommer les checkbox ainsi pour récupérer les données;

je suis d'accord pour le nommage des checkboxes, mais c'était plus dans le script que je me posais la question. mais je n'en reste qu'au stade de la question... et de l'intuition

[bressan]

C'était sympa de répondre mais malheureusement rien ne marche.

J'ai essayé de passer les deux syntaxes en argument :

Code :

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

1
2
3
 
onsubmit="javascript:validation (this.elements['critere']); return (false);"
onsubmit="javascript:validation (this.elements['critere[]']); return (false);"

J'ai changé le nom de mon tableau en me disant que les "_" pouvaient y faire quelque chose mais évidemment ça ne vient pas de ça.

Avec toujours la fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
function validation (critere)
{
	var test_nb = 0;
	for (i = 1; i <= critere.length; i++)
	{
		if (critere[i].checked==false)
		{
			test_nb++;
		}
	}
	alert ("test=" + test_nb);
}

En fait c'est dès que j'utilise mon tableau critere que rien ne va plus.
J'imagine que c'est au niveau syntaxique que se trouve l'erreur mais je ne vois pas où elle peut être.
Si vous avez d'autres idées je suis preneur.

Merci.

[javatwister]

ce qui m'embête un peu, c'est que tu n'aies pas tenu compte des propositions de code html;

[marcha]

Salut

Je te propose une manière un peu différente pour consulter tes checkbox
car le fait que le nom soit composé de crochets [] peut poser des problèmes
de syntaxe.

Ce que je fait dans ces cas là c'est de consulter tous les input (du document
ou du formulaire) avec getElementsByTagName(...)

Puis je test le type et le nom

Dans ton cas cela pourrait ressembler à ceci pour savoir si une case
au moins est cochée

Code :

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

1
2
3
4
5
6
7
8
9
 
function auMoinsUneCase(nom_du_groupe) {
  var champs = document.getElementsByTagName('input');
  for(var i=0;i<champs.length;i++) {
    var c = champs[i];
    if(c.type.toLowerCase()=='checkbox' && c.name==nom_du_groupe && c.checked) return true;
  }
  return false;
}

[bressan]

Merci pour ces réponses.

javatwister, j'ai essayé tous les codes qui m'ont été donnés. Mais comme je n'ai réussi à en faire marcher aucun, j'étais revenu au début, c'est à dire à ce que j'avais fait dans un premier temps.

Le seul code qui ne fait pas d'erreur est celui de marcha mais il avec ce code, même quand je coche une case, ça ne le prend pas en copte, cela doit venir d'un problème de nom.

Je dois faire autre chose d'urgent mais après je vous tiens au courant si je réussi quelque chose.