Plusieurs calculs en fonction d'un montant

Version imprimable

bonjour à tous ,

j'ai un petit souçis, j'ai créé une fonction pour faire un calcul de frais d'agence en fonction du montant , lé problème que je rencontre , c'est que dans se calcul j'ai des addition et des mulitiplication , et je ne sais comment faire pour que cela fonctionne .

j'ai essayé différente choses , mais il n'y a que le dernier calcul( multiplication ) qui fonctionne .

voila le code que j'ai fais au départ

merci pour votre aide sachant que je suis débutant

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
<script>
        function Calcul() {
        console.log("toto");
        Prix = document.getElementById('prix').value;
        if(Prix >= 1 && Prix <= 99999){
        Taux_com = ('6000');
        }
        if(Prix >= 100000 && Prix <= 200000){
        Taux_com = ('1.06');
        }
        if(Prix >= 200001 && Prix <= 400000){
        Taux_com = ('1.05');
        }
        if(Prix > 400001 ){
        Taux_com = ('1.04');
        }
        var Prixtot = Prix*Taux_com;
        var Com = Prixtot-Prix;
        document.getElementById('prix_fai').value= Prixtot.toFixed();
        document.getElementById('commission').value= Com.toFixed();
 
        }
        </script>

ce que j'ai essayé de modifier :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
<script>
       function Calcul() {
       console.log("toto");
       Prix = document.getElementById('prix').value;
       if(Prix >= 1 && Prix <= 99999){
       Taux_com = ('6000');
       }
       if(Prix >= 100000 && Prix <= 200000){
       Taux_com = ('1.06');
       }
       if(Prix >= 200001 && Prix <= 400000){
       Taux_com = ('1.05');
       }
       if(Prix > 400001 ){
       Taux_com = ('1.04');
       }
       if(Prix < 350001 ){
       Taux_com = ('1.04');
       }
       if(Prix <= 99999){
       var Prixtot = Prix+Taux_com;
       }
       else
       {
       var Prixtot = Prix*Taux_com;
       }
       var Prixtot = Prix*Taux_com;
       var Com = Prixtot-Prix;
       document.getElementById('prix_fai').value= Prixtot.toFixed();
       document.getElementById('commission').value= Com.toFixed();
 
       }
       </script>

03/07/2019, 09h38
javatwister

Bonjour,

Tu veux dire que tu as un message d'erreur? Ou que le résultat des opérations est faux?

En testant, je constate que ça "fonctionne", même si ta fonction est trop bavarde;
03/07/2019, 09h41
anti91

bonjour,

non pas d'erreur , sur le premier code , lé problème c'est que le premier calcul est une addition et pas une multiplication .

donc si le montant est inférieur à 100000 il multiplie le montant par 6000 au lieu de l'additionner ;

et dans le deuxième il ne prend en compte que le dernier calcul ....

j'ai peut être ommis quelques choses ....
03/07/2019, 09h45
javatwister

Oui mais si tu veux additionner, utilise

Code:

+

et non

Code:

*

;
03/07/2019, 10h07
anti91

oui ,

mais comment je fais pour utiliser + ou * en fonction du montant , car les essais que j'ai fais ne fonctions pas ... as tu un exemple à me montrer

merci

:salut:

Sous réserve d'avoir bien compris le problème :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
    function Calcul() {
        let Prix = document.getElementById('prix').value;
        let Taux_com = 0;
        let Prixtot = 0;
        let Com = 0;
 
        if (Prix >= 1 && Prix <= 99999) {
            Taux_com = ('6000');
        }
        if (Prix >= 100000 && Prix <= 200000) {
            Taux_com = ('1.06');
        }
        if (Prix >= 200001 && Prix <= 400000) {
            Taux_com = ('1.05');
        }
        if (Prix > 400001) {
            Taux_com = ('1.04');
        }
        if (Prix < 350001) {
            Taux_com = ('1.04');
        }
 
        if (Prix < 100000) {
            Prixtot = Prix + Taux_com;
        } else {
            Prixtot = Prix * Taux_com;
        }
 
        Com = Prixtot - Prix;
 
        document.getElementById('prix_fai').value = Prixtot.toFixed();
        document.getElementById('commission').value = Com.toFixed();
    }

03/07/2019, 10h47
anti91

bonjour danielhagnoul,

merci pour ta réponse , je ne connaissais pas ' let ' .....

je test cela et reviens vers toi ...

bon malheureusement cale ne fonctionne pas pour les montant en dessous de 100000 .....il ne prend pas en compte l'addition ... je ne comprends pas pourquoi ....

au dessus pas de soucis ....

Salut @anti91,
dans ta version modifée il y a une instruction vers la fin, var Prixtot = Prix * Taux_com qui écrase la valeur précédente de Prixtot. La version de @danielhagnoul corrige ce problème, normalement.

Après, Daniel n’a pas remis en question ton test avec la valeur 350 001 qui écrase les taux calculés avant.

Je propose de réécrire ce code avec des if / else enchaînés qui auront deux avantages :

on perçoit un peu mieux le « flot logique » du calcul ;
on écrit moins de chiffres en dur, et donc on a moins de chances de faire des fautes de frappe.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 function Calcul() { "use strict"; // voir https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Strict_mode console.log("Africa"); let Prix = document.getElementById('prix').value; let Taux_com; if (Prix < 1) { throw Error("Calcul non défini pour prix < 1"); } else if (Prix < 100000) { Taux_com = 6000; } else if (Prix < 200000) { Taux_com = 1.06; } // que faire autour de 350000 ? // // ... // else if (Prix < 400000) { Taux_com = 1.05; } else { // Prix >= 400000 Taux_com = 1.04; } let Prixtot; if (Prix < 100000) { Prixtot = Prix + Taux_com; } else { Prixtot = Prix * Taux_com; } let Com = Prixtot - Prix; document.getElementById('prix_fai').value = Prixtot.toFixed(); document.getElementById('commission').value = Com.toFixed(); }
Tu noteras également que j’ai passé toutes les valeurs numériques en nombres. En JavaScript, il faut se méfier des conversions de type implicites comme de la peste. D’ailleurs, je te suggère une petite amélioration sur la façon de récupérer la valeur Prix.
Pour la clarté, je vais mettre une référence sur l’input dans une variable :

Code:

const inputPrix = document.getElementById('prix');

si l’input #prix est de type="number", tu peux utiliser inputPrix.valueAsNumber ;
sinon, convertis explicitement avec parseInt(inputPrix.value, 10).

Si, comme moi, tu n’aimes pas compter les zéros, tu peux utiliser la notation scientifique 1e5, 2e5, 3.5e5, etc. à la place de 100 000, 200 000, 350 000, respectivement.

À l’affichage, pour le confort de l’utilisateur ou de l’utilisatrice, tu peux utiliser un formateur :
Code:

1 2 const formatter = Intl.NumberFormat(); console.log(formatter.format(350000)); // "350 000"
Un exemple un peu plus complexe :
Code:

1 2 const formatter = Intl.NumberFormat('fr-FR', { style: 'currency', currency: 'EUR' }); console.log(formatter.format(350000)); // "350 000,00 €"
Pour finir, tu n’es pas à l’abri de petits problèmes de précision, mais on va garder ça pour plus tard ;)

Bonjour,

Code:

Taux_com = 6000;

c'est totalement anti-logique d'associer la valeur "6000" à la variable Taux_com.... :roll:
C'est comme mélanger les carottes et les serviettes !

Il faut utiliser UNE AUTRE variable :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 let Taux_com = 1; // valeur par défaut let Prix_plus = 0; // valeur par défaut if (Prix < 1) { throw Error("Calcul non défini pour prix < 1"); } else if (Prix < 100000) { Prix_plus = 6000; } else if (Prix < 200000) { Taux_com = 1.06; } ....... let Prixtot = (Prix + Prix_plus) * Taux_com; // calcul valable dans TOUS les cas