[debutant] réécriture de code : lecture de checkbox pour mettre dans un tableau

Bonjour à tous,

Comme vous le savez je ne suis pas un superdoué de JS.
Ci-dessous un code qui fonctionne mais n'existe-t-il pas un moyen de le rendre un peu plus simple à lire ?

Merci beaucoup !

Code:

---

1 2 3 4 5 6 7 8 9

nb_byweekday =1 ; byweekday = {}; if ($('#repeat-event-dialog-weekly-repeat-day-0').is(':checked')) { byweekday [nb_byweekday] = 0 ; nb_byweekday = nb_byweekday +1 ; } if ($('#repeat-event-dialog-weekly-repeat-day-1').is(':checked')) { byweekday [nb_byweekday] = 1 ; nb_byweekday = nb_byweekday +1 ; } if ($('#repeat-event-dialog-weekly-repeat-day-2').is(':checked')) { byweekday [nb_byweekday] = 2 ; nb_byweekday = nb_byweekday +1 ; } if ($('#repeat-event-dialog-weekly-repeat-day-3').is(':checked')) { byweekday [nb_byweekday] = 3 ; nb_byweekday = nb_byweekday +1 ; } if ($('#repeat-event-dialog-weekly-repeat-day-4').is(':checked')) { byweekday [nb_byweekday] = 4 ; nb_byweekday = nb_byweekday +1 ; } if ($('#repeat-event-dialog-weekly-repeat-day-5').is(':checked')) { byweekday [nb_byweekday] = 5 ; nb_byweekday = nb_byweekday +1 ; } if ($('#repeat-event-dialog-weekly-repeat-day-6').is(':checked')) { byweekday [nb_byweekday] = 6 ; nb_byweekday = nb_byweekday +1 ; }

---

Bonsoir

Je suggère de tester :

Code:

---

1 2 3 4 5 6

$( "[id^='repeat-event-dialog-weekly-repeat-day-']" ).each( function( i, item){ if ( $( item ).is( ":checked" ) ){ byweekday [nb_byweekday] = this.id.slice( -1 ); nb_byweekday = nb_byweekday +1 ; } });

---

Génial çà marche :merci:

Code:

---

1 2 3 4 5 6

byweekday = {};   for( var i = 0; i < 7; i++ ) if( $('#repeat-event-dialog-weekly-repeat-day-'+i ).is( ':checked' ) ) byweekday[i+1] = i;

---

pour ma part

Bonjour,
Darkaurora, la méthode présentée par daniel s’affranchit du nombre d'éléments à traiter ce qui en cas de modification est toujours préférable, de plus il existe la méthode each() donc autant s'en servir ;)

Coté performance, il n'y a qu'une seule recherche des l’éléments et non une à chaque itération.

Il est mieux payer que moi ça compte pas :)

Citation:

---

Envoyé par Darkaurora

Il est mieux payer que moi ça compte pas :)

---

:calim2:

Les personnes qui participent à ce forum soit pour poser des questions, soit pour y répondre sont toutes des bénévoles. Les titres et les points sont fonction de l'implication du bénévole dans le fonctionnement de DVP.

Je te conseille une saine lecture : http://club.developpez.com/regles/.

Je serais plus sérieux à l'avenir, la vérité est que je ne connaissait pas le sélecteur présenté par danielhagnoul. Etant du genre mauvais développeur, je me contente de faire en sorte que ça fonctionne et ensuite d'optimiser.

PS: je ne souhaitais pas offenser qui que ce soit... Un simple brin d'humour franchement douteux.