Discussion :

[okoweb]

Bonjour,
J'ai ce code

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
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.

[E.Bzz]

Bonjour,

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)

Code :

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

parseFloat(un_element.value, 10)

A+

[okoweb]

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.

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

Code :

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

parseFloat(un_element.value, 10)

A+

Je ne savais pas, merci...

[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 :

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

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

ceci est à la fois plus lisible et plus efficace :

Code :

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

document.getElementById('machin').value

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

[okoweb]

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, ...

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

Okay pour onkeyup

Code :

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

document.getElementById('machin').value

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

Code :

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

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;

[E.Bzz]

Ce bazar ne marche pas pour les select

Si !

As-tu testé ?
Sur quel navigateur ?

Sinon

Je ne savais pas, merci...

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

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

As-tu seulement essayé quelque chose ?

A+

[dukej]

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

Code :

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

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.

[okoweb]

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

[dukej]

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.