Discussion :

[loicdvi]

Bonjour,

Je rencontre un problème dans une fonction qui traite les checkbox cochées dans un formulaire.
Mes checkbox ont le même nom, et tout se passe bien quand il y en a au moins deux.

Le code utilisé est le suivant :

Code :

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

1
2
3
4
5
 
demandes = document.forms['monForm'].demandes
for (i=0; i < demandes.length; i++) {
[...]
}

et c'est ce qu'on retrouve de manière systématique partout où l'on parle de parcours de checkbox.

Là où ça se corse, c'est quand il n'y a qu'une seule checkbox : je récupère un input plutôt qu'un tableau, ce qui fait planter la fonction.
Il y a diverses solutions pour vérifier si demandes est un tableau ou non, la plus rapide que j'aie trouvée est de faire

Code :

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

1
2
3
4
5
6
 
if (demande.length) {
  // on traite un tableau
} else {
  // on traite un input seul
}

Mais c'est peut satisfaisant (tant sur le test effectué, que sur l'obligation de faire 2 cas séparés).

Mes questions sont donc :
1. quelle est la meilleure pratique pour tester si "demandes" est un tableau ou un élément simple ? (remarque : "demandes instanceof Array" retourne false)
2. n'y a t-il pas un moyen de récupérer "demandes" sous forme de tableau qu'il n'y ait qu'une seule checkbox ou plusieurs ? (par exemple comme avec document.getElementsByName(), qui retourne toujours un tableau - mais malheureusement "document.forms['monForm'].getElementsByName('demandes')" ne fonctionne pas )

[SpaceFrog]

utilise une syntaxe plus rigoureuse en ne passant pas par les éléments du form avec elements mais par ceux du document avec getElementsBtName

Code :

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

document.getElementsByName('demandes')

tu auras une collection dans tous les cas

ou si tu as plusieurs forms:

Code :

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

document.forms['myform'].getElementsByName('demandes')

[Bovino]

remarque : "demandes instanceof Array" retourne false

Normal, tu récupères une collection, pas un array

n'y a t-il pas un moyen de récupérer "demandes" sous forme de tableau qu'il n'y ait qu'une seule checkbox ou plusieurs ? (par exemple comme avec document.getElementsByName()...)

Si ton ta page est correctement construite, cette syntaxe suffit

[loicdvi]

Merci pour vos réponses.

@SpaceFrog :

Code :

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

1
2
 
document.forms['myform'].getElementsByName('demandes')

ne fonctionne malheureusement pas, comme je l'indiquais dans mon premier message (erreur "document.forms['myform'].getElementsByName is not a function").

Finalement j'ai utilisé le "document.getElementsByName('demandes')" car c'est valable dans mon cas, mais c'est assez frustrant, vous en conviendrez... .

Et que faire le jour où je ne peux pas récupérer mes checkbox par ce moyen (dans le cas de plusieurs formulaires qui ont un même nom pour un élément par exemple) ?

[Matthieu2000]

Code :

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

document.forms['myform'].demandes

Code :

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

document.forms['myform'].elements['demandes']

[loicdvi]

Code :

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

document.forms['myform'].elements['demandes']

Même problème que précédemment : s'il n'y a qu'une seule checkbox, je récupère un input plutôt qu'une collection .

[Matthieu2000]

Même problème que précédemment : s'il n'y a qu'une seule checkbox, je récupère un input plutôt qu'une collection .

Crée un tableau

Code :

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

1
2
var toto =new Array();
toto.push(document.forms['myform'].demandes)

[Bovino]

Finalement j'ai utilisé le "document.getElementsByName('demandes')" car c'est valable dans mon cas, mais c'est assez frustrant, vous en conviendrez... .

Non, pas spécialement pourquoi ?

Et que faire le jour où je ne peux pas récupérer mes checkbox par ce moyen (dans le cas de plusieurs formulaires qui ont un même nom pour un élément par exemple) ?

Tout comme getElementById est une méthode de document parce qu'un id doit être unique dans la page, tu n'es pas sensé avoir plusieurs groupes d'éléments distincts avec le même name, donc ton cas de figure implique plus une erreur de conception qu'une contrainte...

[loicdvi]

Non, pas spécialement pourquoi ?

Tout comme getElementById est une méthode de document parce qu'un id doit être unique dans la page, tu n'es pas sensé avoir plusieurs groupes d'éléments distincts avec le même name, donc ton cas de figure implique plus une erreur de conception qu'une contrainte...

C'est pas faut...

La solution de Matthieu2000 me paraît aussi être une alternative correcte dans le cas où on serait confronté à l'erreur de conception que décrit Bovino.

Merci pour votre aide en tout cas !