Discussion :

[suckthewindow]

Bonjour a tous,
Je debute en Jquery et j'ai du mal a comprendre la notion de boucle!
J'ai un ensemble de checkbox que l'utilisateur peut cocher afin de supprimer les donnees cochées! Du coup j'aimerai recuperer leur nom, les concatener et demander a l'utilisateur si il veut les supprimer !
Seulement j'ai un probleme avec le each !!! je n'arrive pas a lui faire retourner ce que je veux :'(:

exemple:

Code :

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

1
2
3
4
5
 
 
msg = $("input[type=checkbox][checked]").each(function(msg) {
         return msg + $(this).attr("name");
         });

mais ca ne marche pas !
Quelle est la bonne methode ?
merci
Jerome

[Sourrisseau]

Ton code un est un peu confus. Mais nous allons éclaircir tout ca.

tu utiles la variable msg a plusieurs endroits. Clairement each(function(msg) ici msg est utilise par jquery pour récupérer le nombre d'itération de ta boucle. Par conséquent il va prendre comme valeur 0,1 ....n. Conclusion tu utilises une variable qui représente 2 choses différentes

Ensuite dans ta boucle tu effectue un return ce qui n'est pas faisable car cela doit à mon avis arrêter immédiatement le traitement.

Enfin j'ai un doute , mais je n'en suis pas certain sur ta notation : $("input[type=checkbox][checked]")

Au final je te proposerai un code du type

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
function val_checkbox(){
   msg="";
   $("input[type=checkbox]:checked").each(function(x) {
         msg += $(this).attr("name");
         });
   return msg;
}
 
// on fait un alert pour tester
 
alert(val_checkbox());

Je n'ai pas testé ce code mais si cela ne marche pas dit le moi.

[suckthewindow]

ca marche nikel
merci :p

[JeitEmgie]

Ensuite dans ta boucle tu effectue un return ce qui n'est pas faisable car cela doit à mon avis arrêter immédiatement le traitement.

le return sort de la fonction passée en paramètre du each pas du each lui-même…
la raison pour laquelle cela ne marche pas est que personne ne collecte les résultats…

techniquement parlant msg += $(this).attr("name"); est la solution,
mais esthétiquement ça va pas être très lisible…

Code :

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

1
2
3
4
5
6
7
8
 
String.prototype.listAppend = function(toAppend, separator)
{
	if (this == "")
		return toAppend ;
 
	return this + separator + toAppend ;
}

et dans le each

Code :

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

1
2
 
         msg = msg.listAppend( $(this).attr("name"), ", ") ;