Discussion :

[kobrasnip]

Bonjour à tous,

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

Code :

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

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

[SpaceFrog]

Pas forcément simplifié pais certainement plus lisible:

Code :

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

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

[bigboomshakala]

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

Code :

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

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 :

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

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

[DoubleU]

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

[kobrasnip]

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

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

[SpaceFrog]

A la limite pour un truc super clair je ferais :

Code :

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

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