Discussion :

[cireultra]

Bonjour,

Voilà mon instruction if se répète 95 fois, comment simplifier ou raccourcir.

EX :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 if ((quantite.value<=12)&&(departement.value==01)) {
        total.value=24.21;
      }
 
 if ((quantite.value<=12)&&(departement.value==02)) {
        total.value=23.22;
      } 
 
if ((quantite.value<=12)&&(departement.value==03)) {
        total.value=19.95;
      }
 
if ((quantite.value<=12)&&(departement.value==04)) {
        total.value=17.46;
      }

Jusqu’à

Code :

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

1
2
3
if ((quantite.value<=12)&&(departement.value==95)) {
        total.value=35.41;
      }

La quantité ne varie pas (<=12) par contre il y à 95 départements et un total pour chaque département.

Dès que la quantité est (>12) il n’y a qu’une instruction if.

Code :

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

1
2
3
if (tota6.value>12) {
       total.value=Math.round((tota7.value * tota6.value) * 100)/100 ;
      }

Donc comment simplifier la première partie avec ses 95 instructions if ?

Merci

Cireultra

[le_chomeur]

une boucle for + un tableau contenant les valeurs de la somme

[SpaceFrog]

avec un switch :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
if (quantité.value<=12){
      switch (department.value){
                                                 case ('01'): total.value=24.1;
                                                                    break:
                                                 case ('02'): total.value=???;
                                                                    break;
                                                  case ('03'): total.value=???;
                                                                    break;
                                                  ../.. 
                                                 }
}
else { total.value=Math.round((tota7.value * tota6.value) * 100)/100 ;}

attention 01 != '01'

[le_chomeur]

plus optimisé avec le tableau et la boucle for ^^

[cireultra]

Merci pour les réponses.

Je vais tester vos idées.


A +

Cireultra

[cireultra]

Re

J'y suis arrivé avec le switch, mais pas avec une boucle for + un tableau contenant les valeurs de la somme.

Le chomeur : un petit exemple de code svp.

A+

Cireultra

[Bovino]

Euh... pourquoi tu ne mets pas tes valeurs dans un tableau :

Code :

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

var tarifs = ['rien', 24.21, ...]

(la valeur rien au début sert juste à mettre en correspondance les index du tableau avec le n° de département) et ensuite :

Code :

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

1
2
3
if (quantite.value<=12) {
    total.value=prix[parseInt(departement.value,10)];
}

[cireultra]

Merci Bovino.

Tu pourrais m'expliquer cette ligne de ton exemple: total.value=prix[parseInt(departement.value,10)];

A+

Cireultra

[le_chomeur]

c'est ce que je t'expliquais plus haut :

tu stockes tes prix dans un tableau :

var prix['null',23,34 ...] // on met null car un tableau commence a l'index 0

puis :

Code :

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

1
2
3
4
5
6
 
for(var i = 1 ; i < nombre de département ; i++){
 
total.value=prix[i]; récupère le prix ....
 
}

la ligne de bovino :

total.value=prix[parseInt(departement.value,10)];

parseInt(Arguement1 , 10) :
- Argument.value corespondra a l'index
- 10 sépécifie une base de 10
- parseInt transforme la valeur du champs qui est au format texte en entier

au final cela revient au même que faire :

prix[x] ou X est un entier

[Bovino]

@ le_chomeur :
A moins que quelque chose m'ait échappé, il n'y a pas besoin de boucle for

[le_chomeur]

ha vi je pensais qu'on affectais un prix par département
un peu l'ouest ces temps ci

[cireultra]

Merci pour ces explications.

Je vais re tester avec un tableau

A +

Cireultra

[cireultra]

Le chomeur
Oui il y a un prix fixe et différent pour chaque département.

dep 01 -> 24.21
dep 02 -> 23.22
etc
Jusqu'à 95 départements

A +

Cireultra

[javatwister]

donc

Code :

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

total.value= quantite.value<=12 ? prix[parseInt(departement.value,10)] : Math.round((tota7.value * tota6.value) * 100)/100 ;