Discussion :

[davleo]

Bonjour à tous,
Après plusieurs recherches je suis toujours bloqué dans mon test de vérification si une case est coché.
mon problème semble être la variable frm_group[i] qui ne fonctionne pas
J'ai épuré le code pour plus de lisibilité car trop long
Voici mon code :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
 
<SCRIPT>
function validate(theForm) 
{	
  for(i=0;i<theForm.frm_group.length;i++)
    {
	   if (theForm.frm_group[i].checked == true)     
        {
         theForm.submit(); 
         return true;
        }  
	}
        alert ("cocher au moins une case"); return false;
 
}
 
 
</SCRIPT>
 
<FORM NAME="add" METHOD="post"  onSubmit="return validate(document.add)">
<?php // je recherche dans ma table combien de region a $user_id
 
$query = "SELECT * FROM `allocates` WHERE `type`= 4 AND `resource_id` = '$user_id'";            
$result = mysql_query($query) or do_error($query, 'mysql query failed', mysql_error(),basename( __FILE__), __LINE__);
 
while ($row2 = stripslashes_deep(mysql_fetch_assoc($result)))  // pour chaque region j'affiche une checkbox avec le nom associé et je donne un nom qui s’incrémente frm_group[] car différent selon le user 
{  
 
?>
   
<div>
<INPUT TYPE='checkbox'  name='frm_group[]'  VALUE=' <?php echo $row2['group'];?>'><?php echo $row2['group'];?></INPUT>
</div>
 
 <?php
}
?>
<INPUT TYPE="submit"   VALUE="Valider"  onClick="validate(document.add);" STYLE = 'margin-top:4px;'>
</FORM>

Merci pour toute solution (correction devrais-je dire)
Merci David

[Auteur]

bonjour,

peut-on voir le code HTML généré par ton script PHP ?

[davleo]

le voici

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
 
 
<SCRIPT>
function validate(theForm) 
{	
  for(i=0;i<theForm.frm_group.length;i++)
    {
	   if (theForm.frm_group[i].checked == true)     
        {
         alert ("une case cochée"); 
         return true;
        }  
	}
        alert ("cocher au moins une case"); return false;
 
}
 
 
</SCRIPT>
 
<FORM NAME="add" METHOD="post"  onSubmit="return validate(document.add)">  
 
<div>
<INPUT TYPE='checkbox'  name='frm_group[]'  VALUE="1">1</INPUT>
</div>
 
 
<div>
<INPUT TYPE='checkbox'  name='frm_group[]'  VALUE="3">3</INPUT>
</div>
 
 
<div>
<INPUT TYPE='checkbox'  name='frm_group[]'  VALUE="4">4</INPUT>
</div>
 
 
<div>
<INPUT TYPE='checkbox'  name='frm_group[]'  VALUE="5">5</INPUT>
</div>
 
 <INPUT TYPE="submit"   VALUE="Valider"  onClick="validate(document.add);" STYLE = 'margin-top:4px;'>
</FORM>

[Auteur]

Plusieurs remarques :
code HTML :
- prends l'habitude d'écrire les balises et les attributs en minuscules (onsubmit pas onSubmit, type et non TYPE, etc.)
- la balise input est auto fermante :

Code html :

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

<input type='checkbox'  name='frm_group[]'  value="5"/>5

Tu peux associer un label à une balise input si tu le souhaites.
- inutile d'ajouter un événement onsubmit dans un bouton de type submit, c'est sous entendu.
- il manque l'attribut action à ta balise form.

javascript :
- tu as écrit onsubmit="return validate(document.add)". Pourquoi document.add ? Pourquoi ne pas directement transmettre l'objet form à ta fonction ? De cette manière onsubmit="return validate(this)".
-

Code :

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

theForm.frm_group[i].checked

cette syntaxe pour accéder à un élément de ton formulaire n'est pas recommandée. Comme l'objet form est transmis à la fonction autant l'utiliser :

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
function validate(theForm)
{
	var inputList = theForm.getElementsByTagName("input"); // retourne tous les objets de type input dans le formulaire
	var i;
	for (i=0;i<inputList.length; i++)
	{
		if (inputList[i].checked)
		{
			alert("une case cochée"); // une case est cochée...
			return true;  // le formulaire est envoyé (pas besoin de submit())
		}
 
	}
	alert("Aucune case cochée"); // le formulaire n'est pas envoyé
	return false;
}

[davleo]

Merci beaucoup Auteur tout fonctionne parfaitement, j'ai bien pris note de tes recommandations