Bonjour,
À mon avis, l'ensemble du calendrier pourrait être traité en 1 seule ligne de moins de 23 caractères ... Nan, je plaisante !
Plus sérieusement, il n'y a sans doute pas grand chose à réduire (ou alors au détriment de la lisibilité du code qu'il ne faut pas négliger). Voici cependant quelques remarques.
1) si tu es sûr que l'on ne peut pas avoir 2 fêtes le même jour (je ne sais pas si c'est vraiment le cas), il vaut mieux faire des if ... else if ... else if ... plutôt que des if ... if ... if ...
L'exécution sera plus rapide puisqu'on arrêtera dès qu'on aura trouvé au lieu de continuer à tester jusqu'au bout.
2) si tu as très peu de "colors" différentes, voire si c'est toujours "white", tu peux ajouter la couleur à l'objet "data" après la série de conditions. Si tu as 50 fêtes pour lesquelles data.color vaut "white", ce serait ballot d'écrire 50 fois color: "white" alors qu'un seul data.color="white"; pourrait suffire.
if (dayMonth === '0812' && immaculateConceptionDay !== 7 || dayMonth === '0912' && immaculateConceptionDay === 7) data = {name: "Immaculée Conception de la Bienheureuse Vierge Marie", grade: "1", rank: "3"};
else if (dayMonth === baptismOfTheLord && epiphany !== ('0701' || '0801') || dayMonth === '0801' && epiphany === '0701' || dayMonth === '0901' && epiphany === '0801') data = {name: "Le Baptême du Seigneur", grade: "3", rank: "5"};
data.color="white";
Tu as peut-être d'autres améliorations du même genre de possible pour grade et rank, en regroupant les tests pour les datas de même grade ou de même rank.
Mais bon, on gagne pas forcément beaucoup et on diminue la clarté du code.
3) dans les conditions, je mettrais des parenthèses un peu partout et je ne ferais pas confiance à mon cerveau pour calculer les priorités des opérateurs. Quand on a beaucoup d'opérateurs dans une condition, on a vite fait de faire une erreur sur leurs priorités.
if (((dayMonth === '0812') && (immaculateConceptionDay !== 7)) || ((dayMonth === '0912') && (immaculateConceptionDay === 7))) data = {name: "Immaculée Conception de la Bienheureuse Vierge Marie", grade: "1", rank: "3"};
Comme ça, t'es sûr de ce que ça fait.
Je signale cependant que mon avis est minoritaire sur ce point.
4) Il n'est pas nécessaire d'employer ici !== et ===. Les opérateurs != et == suffisent.
Certes, y en a qui disent de toujours utiliser === et !==. Je ne suis pas de cet avis (et je ne suis pas le seul).
En particulier quand il s'agit de comparer des nombres ou des chaines de caractères au contenu d'une variable (ce qui est ton cas ici), il n'y a pas de bonnes raisons d'utiliser === et !==.
Par exemple, pour la comparaison entre immaculateConceptionDay et 7, c'est risqué. Si on emploie !==, il faut être sûr avant de faire la comparaison que immaculateConceptionDay est bien un nombre et pas une chaine de caractère : c'est souvent une prise de tête. Or il est clair ici que même si immaculateConceptionDay vaut par inadvertance "7" (et non pas 7), on attend que la comparaison renvoie la même chose que si c'était 7 quand même.
Parfois, c'est encore plus sournois que ça. Par exemple :
var a="a";
var b=new String("b");
var r;
r=(a=="a"); // r vaut true
r=(a==="a"); // r vaut true
r=(b=="b"); // r vaut true
r=(b==="b"); // r vaut false !!!
Ceci veut dire qu'il faudrait être sûr, avant d'utiliser === quand on compare 2 chaines de caractères, qu'elles ont bien été créées de manière "compatible. Pas toujours évident !
Par contre, dans certains cas, il faut privilégier === et !==, mais en pratique, c'est souvent quand l'un des opérateurs vaut false ou l'un de ses nombreux équivalents (0, "", undefined, null, etc...).
5) Pour des éventuelles optimisations des conditions, il faudrait avoir une connaissance plus précise de l'ensemble du projet (par exemple, comment sont calculés immaculateConceptionDay, baptismOfTheLord, ...). En particulier, je me suis demandé pourquoi ne suffit-il pas d'écrire :
if (dayMonth == baptismOfTheLord) data = {name: "Le Baptême du Seigneur", color: "white", grade: "3", rank: "5"};
J'imagine que tu as de bonnes raisons d'avoir écrit des conditions à rallonge. Et si elles sont nécessaires, il y a assez peu de gains à espérer.
Amicalement,
Modifié par parsimonhi (27 Nov 2020 - 20:00)
