Simplification d'un if/else

Bonjour à tous,

Etant vraiment débutant en javascript, j'aurai aimé savoir si le code suivant était simplifiable :

Code:

---

1 2 3 4 5 6

if (this.value == '1') {$('part1').show(); $('part2').hide(); $('part3').hide();} else { if (this.value == '2') {$('part2').show(); $('part1').hide(); $('part3').hide();} else { $('part3').show(); $('part2').hide(); $('part1').hide();}}

---

Il s'agit de l'option onChange d'une balise select qui a pour seul but de n'afficher qu'une partie d'un formulaire à la fois.

Merci beaucoup pour votre aide :)

Pas forcément simplifié pais certainement plus lisible:

Code:

---

1 2 3 4 5 6 7

switch (this.value){ case '1': $('part1').show(); $('part2').hide(); $('part3').hide(); break; case '2': $('part2').show(); $('part1').hide(); $('part3').hide(); break; default: $('part3').show(); $('part2').hide(); $('part1').hide(); }

---

c'est toujours la même séquence d'actions. on peut aussi faire

Code:

---

1 2 3 4 5 6 7

var id1 = this.value == 1 ? "part1" : this.value == 2 ? "part2" : "part3"; var id2 = this.value == 1 ? "part2" : this.value == 2 ? "part1" : "part3"; var id3 = this.value == 1 ? "part3" : this.value == 2 ? "part2" : "part1";   $(id1).show(); $(id2).hide(); $(id3).hide();

---

ceci dit c'est moins efficace car on réévalue plusieurs fois la condition.
on peut aussi utiliser le switch de spaffy pour déterminer les ID et appeler les fonctions après le switch (plus lisible)

EDIT : ou bien

Code:

---

1 2 3 4 5 6 7 8 9 10 11

var ids = null; switch(this.value) { case "1" : ids = "part1,part2,part3".split(","); break; case "2" : ids = "part2,part1,part3".split(","); break; default : ids = "part3,part2,part1".split(","); break; }   $(ids[0]).show(); $(ids[1]).hide(); $(ids[2]).hide();

---

Faut pas non plus vouloir simplifier juste pour le principe, parce qu'apres le code devient bordélique...

Merci beaucoup messieurs :) J'ai un peu honte de ne pas avoir pensé au switch... :oops:

Je vais faire au mieux, car comme l'a bien précisé DoubleU, le but est de rentre le code facile à maintenir.

Bonne fin de journée

A la limite pour un truc super clair je ferais :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

function affiche(){ $('part'+arguments[0]).show(); for (i=1;i>arguments.length;i++){ $('part'+arguments[i]).hide() } } switch (this.value){ case '1': affiche(1,2,3); break; case '2': affiche(2,1,3); break; default: affiche(3,1,2); }

---