Synthaxe et factorisation [Résolu]

[dominos]

Bonsoir,

Voila, je vous soumet le code suivant qui fonctionne bien,
mais par soucis de bien faire, vos conseils m'intéresse.

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
31
32
33
34
35
36
37
38
39
$(function() {
             var select_text = $('#select_text');
	var ass_nom = select_text.text();
	var reg_ass_code =/^(.{3})(.*)$/; 
 
	/**
	* Retourne la liste des codes sélectionnés
	*/
	function get_checked_ass()
	{
		var ass_list = '';
		var ass_checked ='';
		$("[:checkbox]:checked").each(function(i) {
			var id_checkbox = $(this).attr('id');
			// Extrait le code de l'Id
		    var ass_code = id_checkbox.replace(reg_ass_code,'$2');
		    // Concatène les codes 
		    ass_list += ass_code + ','; 
		});
		// Supprime la dernière virgule
		ass_checked = ass_list.substring(0,ass_list.length-1);
		// Affiche/Masque les parenthèses
		var pOn = (ass_checked!='')?' (':''; var pOff = (ass_checked!='')?')':'';
		return pOn + ass_checked + pOff;
    }
 
	/**
	* Ajouter/Suprime les codes sélectionnés
	*/
	$(":checkbox").click(function() {
		var id_checkbox = $(this).attr('id');
		if($(this).is(':checked')) {
			select_text.text(ass_nom + get_checked_ass());
		} else {
			select_text.text(ass_nom + get_checked_ass());
 
		}
	});
});

Merci d'avance.

[Bovino]

Bon... désolé, mais je peux pas m'en empêcher... Tu sais ce que veux dire "ass" en anglais ?
Parce que ça donne une toute autre vision de ton code

Sinon :

Code :

1
2
3
4
5
6
7
8
$(":checkbox").click(function() {
	var id_checkbox = $(this).attr('id');
	if($(this).is(':checked')) {
		select_text.text(ass_nom + get_checked_ass());
	} else {
		select_text.text(ass_nom + get_checked_ass());
	}
});

Euh... je vois mal l'utilité du if

Code :

$(":checkbox").click(get_checked_ass);

ne suffirait-il pas ?

Code :

$("[:checkbox]:checked")

Sauf erreur de ma part, la syntaxe $("[:checkbox]") n'existe pas...
Sinon, à la place de ta concaténation, tu ferais mieux de stocker les résultats dans un tableau et d'utiliser .join(',') en fin de boucle...

[dominos]

Envoyé par Bovino

Bon... désolé, mais je peux pas m'en empêcher... Tu sais ce que veux dire "ass" en anglais ?
Parce que ça donne une toute autre vision de ton code

Très drole.

Euh... je vois mal l'utilité du if

get_checked_ass() supprimée puisque plus nécessaire de factoriser.

Sauf erreur de ma part, la syntaxe $("[:checkbox]") n'existe pas...

Oups ! $(":checkbox:checked") http://www.innovativephp.com/blog/20...kbox-examples/

Sinon, à la place de ta concaténation, tu ferais mieux de stocker les résultats dans un tableau et d'utiliser .join(',') en fin de boucle...

Un exemple vaut mille post.

Et merci.

[Bovino]

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
var ass_list = [];
var ass_checked ='';
$(":checkbox:checked").each(function(i) {
	var id_checkbox = $(this).attr('id');
	// Plus besoin de regexp
	var ass_code = id_checkbox.substr(3);
	// Ajoute au tableau
	ass_list.push(ass_code); 
});
// Supprime la dernière virgule
ass_checked = ass_list.join(',');
// Alternative possible
ass_checked = ass_list.toString();

[dominos]

Très intéressant !