Conversion de devises

Version imprimable

Bonjour,
J'ai ce code

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
 
<script type="text/javascript">
<!--
function convertMoney(){
 getamount(valeur);
 getTTC();
}
 
function getamount(){
     var devise = document.getElementById('devise').options[document.getElementById('devise').selectedIndex].value;
     var ht = document.getElementById("prixht").value;
     var tva = document.getElementById('tva').options[document.getElementById('tva').selectedIndex].text;
     document.getElementById("prixttc").value = ht + (ht*tva/100) +' '+devise;
}	
 
function getTTC(){
     var devise = document.getElementById('devise').options[document.getElementById('devise').selectedIndex].value;
     var ht = document.getElementById("prixht").value;
     var tva = document.getElementById('tva').options[document.getElementById('tva').selectedIndex].text;
     document.getElementById("prixttc").value = ht + (ht*tva/100) +' '+devise;
}	
// -->
</script>
 
<form action="">
<label>Prix HT</label><input type="text" name="prixht" id="prixht" onblur="getamount();" /><br />
<label>TVA</label><select name="tva" id="tva" onchange="getTTC();">
<option value="19.6">19.6</option>
<option value="5.5">5.5</option>
</select><br />
<label>Prix TTC</label><input type="text" name="prixttc" id="prixttc" readonly="readonly" />
<select name="devise" id="devise" onchange="convertMoney();">
<option value="1.00">Euro</option>
<option value="1.31524">Dollar</option>
<option value="0.836586">Livre</option>
</select>
</form>

1. Je souhaite en rentrant un prix HT, que le prix TTC soit calculé et affiché automatique dans le champ prixttc qui est en readonly. Ce TTC est calculé par defaut avec la devise par defaut au chargement de la page (€) et la tva par defaut 19.6.

2. En changeant de TVA que le TTC soit reculculé et affiché.

3. En changeant de devise que le HT soit recalculé et affiché dans le champs approprié ainsi que TTC.

Meilleures salutations.

17/09/2010, 14h05
E.Bzz

Bonjour,

Citation:

Envoyé par okoweb

1. Je souhaite ....

Oui, et ?
Quel est le problème avec ton code actuel ?

A priori que tu tentes de faire des calculs sur des chaînes de caractères ?
(les .value sont des string)

:fleche:

Code:

parseFloat(un_element.value, 10)

A+
17/09/2010, 15h12
okoweb

Citation:

Envoyé par E.Bzz

Oui, et ?
Quel est le problème avec ton code actuel ?

En changeant de devise, je voudrai que le prixht soit converti et aussi prixttc en fonction de la devise choisie.

Citation:

Envoyé par E.Bzz

A priori que tu tentes de faire des calculs sur des chaînes de caractères ?
(les .value sont des string)

:fleche:

Code:

parseFloat(un_element.value, 10)

A+

Je ne savais pas, merci...
18/09/2010, 14h58
Watilin

Bonjour Oko,
quelques remarques.
- J'ai la farouche impression que tes deux fonctions getTTC() et getamount() font la même chose.

- As-tu envisagé onkeyup à la place de onchange sur ton <input> ?

- Au lieu de faire :

Code:

document.getElementById('machin').options[document.getElementById('machin')].value

ceci est à la fois plus lisible et plus efficace :

Code:

document.getElementById('machin').value

Voilà pour le calcul des TTC. Pour les devises, on verra après. Tiens-nous au courant ;)

Citation:

Envoyé par Watilin

Bonjour Oko,
quelques remarques.
- J'ai la farouche impression que tes deux fonctions getTTC() et getamount() font la même chose.

Oui car leur role est de prendre les données ca et là et recalculer le TTC, ...

Citation:

Envoyé par Watilin

- As-tu envisagé onkeyup à la place de onchange sur ton <input> ?

Okay pour onkeyup

Citation:

Envoyé par Watilin

Code:

document.getElementById('machin').value

Ce bazar ne marche pas pour les select, on fait plutot :
Code:

1 2 3 4 5 //pour la valeur document.getElementById('machin').options[document.getElementById('machin')].value; //pour le texte document.getElementById('machin').options[document.getElementById('machin')].text;

21/09/2010, 16h26
E.Bzz

Citation:

Envoyé par okoweb

Ce bazar ne marche pas pour les select

Si !

As-tu testé ?
Sur quel navigateur ?

Sinon

Citation:

Je ne savais pas, merci...

Je n'avais pas répondu pour faire joli ou pour passer le temps :roll:

C'est la réponse à ta question de départ.

As-tu seulement essayé quelque chose ?

A+

Citation:
Envoyé par okoweb
Ce bazar ne marche pas pour les select, on fait plutot :
Code:

1 2 3 4 5 //pour la valeur document.getElementById('machin').options[document.getElementById('machin')].value; //pour le texte document.getElementById('machin').options[document.getElementById('machin')].text;
Il faut arrêter la drogue
select.value, contient la valeur du option sélectionné et ce depuis IE6
mais encore il faudrait que tes options possèdent tous un value.

22/09/2010, 11h09
okoweb

Citation:

Envoyé par dukej

Il faut arrêter la drogue
select.value, contient la valeur du option sélectionné et ce depuis IE6
mais encore il faudrait que tes options possèdent tous un value.

Je ne savais pas qu'on pouvait le faire ainsi. Il faudrait revoir les documentations à propos, la FAQ par exemple. J'ai pas essayé mais je reste prudent. Merci
22/09/2010, 13h15
dukej

Citation:

Envoyé par okoweb

Je ne savais pas qu'on pouvait le faire ainsi. Il faudrait revoir les documentations à propos, la FAQ par exemple. J'ai pas essayé mais je reste prudent. Merci

La faq a certains conseils qu'il vaut mieux éviter à tout prix.