Discussion :

[Emmanuel Lecoester]

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 :

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

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 ; }

[danielhagnoul]

Bonsoir

Je suggère de tester :

Code :

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

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 ;
	}
});

[Emmanuel Lecoester]

Génial çà marche

[Darkaurora]

Code javascript :

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

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

[NoSmoking]

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.

[Darkaurora]

Il est mieux payer que moi ça compte pas

[danielhagnoul]

Il est mieux payer que moi ça compte pas

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/.

[Darkaurora]

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.