Test nombre inférieur à un autre

Version imprimable

Bonjour,

je récupère des champs de mon formulaire, je souhaite comparer 2 valeurs, l'une extraite du formulaire, l'autre calculé.

Si la valeur montant_commission_epargne est superieure au plafond, on doit afficher le plafond, sinon on affiche la valeur de montant_commission_epargne

Cela ne marche pas, il m'affiche toujours le plafond.

Comment puis-je m'assurer que les 2 variables sont du meme format pour les comparer ?

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
39
40
41
42
43
//met a jour le montant de la Commission Epargne 
function maj_montant_commission_epargne()
{
		//recupere le montant du versement epargne
		var montant_versement_epargne=parseFloat(document.getElementById('montant_versement_epargne').value);	
 
		//pourcentage de la commission epargne
		var pourcentage_versement_epargne=parseFloat(document.getElementById('pourcentage_versement_epargne').value);
 
		//plafond de la commission epargne a ne pas depasser
		var plafond_commission_epargne=parseFloat(document.getElementById('plafond_commission_epargne').value);		
 
 
		//montant calcule
		var montant_commission_epargne=parseFloat((montant_versement_epargne*pourcentage_versement_epargne)/100);
 
		var plafond_commission_epargne=round(plafond_commission_epargne,2);
 
		var montant_commission_epargne=round(parseFloat(montant_commission_epargne),2);
 
 
		if(plafond_commission_epargne!=0)
		{
 
			if(montant_commission_epargne<plafond_commission_epargne)
			{
 
			//ecrit le montant de la commission 
			document.getElementById('affichage_montant_commission_epargne').value=montant_commission_epargne;
			document.getElementById('montant_commission_epargne').value=montant_commission_epargne;
 
				} //si le montant de la commission epargne est inferieur au plafond epargne
				else if(montant_commission_epargne>=plafond_commission_epargne)
				{ //sinon la commission correspond au plafond
				document.getElementById('affichage_montant_commission_epargne').value=plafond_commission_epargne;
				document.getElementById('montant_commission_epargne').value=plafond_commission_epargne;
				} 
 
		}
 
		var montant_versement_epargne=round(montant_versement_epargne,2);
		document.getElementById('montant_versement_epargne').value=montant_versement_epargne;
}

Tu es débutante en javascript, je vois.

- les noms de variables doivent être explicites, mais ce n'est pas la peine de les faire exagérément longues.
Je propose : versement, pourcentage, plafond, commission

- On met le mot-clef var lorsqu'on déclare une variable. Par la suite, ce n'est pas la peine de le réutiliser (c'est même une erreur)

- Inutile de parser toutes les deux lignes !!!
À la limite, utilises-le une seule fois lorsque tu prends des valeurs depuis un champ.
Par la suite, javascript les gardera dans le type indiqué.
Lors des calculs, le résultat prend automatiquement le type des opérandes.

Lorsqu'un test est inutile, on évite de le mettre.
Par exemple :
Code:

1 2 3 4 5 if(nombre<10) // action A else if(nombre>=10) // action B
S'il n'est pas plus petit que 10, il est forcément plus grand ou égal.
Il serait donc mieux écrit ainsi :
Code:

1 2 3 4 5 if(nombre<10) // action A else // action B
Ton code "nettoyé" (rien que ça pourrait le faire fonctionner. À tester) :
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 39 //met a jour le montant de la Commission Epargne function maj_montant_commission_epargne() { //recupere le montant du versement epargne var versement=parseFloat(document.getElementById('montant_versement_epargne').value); //pourcentage de la commission epargne var pourcentage=parseFloat(document.getElementById('pourcentage_versement_epargne').value); //plafond de la commission epargne a ne pas depasser var plafond=parseFloat(document.getElementById('plafond_commission_epargne').value); plafond=round(plafond,2); //montant calcule var commission=(versement*pourcentage)/100; commission=round(commission,2); versement=round(versement,2); if(plafond!=0) { //si le montant de la commission epargne est inferieur au plafond epargne if(commission<plafond) { //ecrit le montant de la commission document.getElementById('affichage_montant_commission_epargne').value=commission; document.getElementById('montant_commission_epargne').value=commission; } //sinon la commission correspond au plafond else { document.getElementById('affichage_montant_commission_epargne').value=plafond; document.getElementById('montant_commission_epargne').value=plafond; } } document.getElementById('montant_versement_epargne').value=versement; }

16/11/2012, 18h13
andry.aime

Bonsoir,

Citation:

Envoyé par Michaeljackfan

- On met le mot-clef var lorsqu'on déclare une variable. Par la suite, ce n'est pas la peine de le réutiliser (c'est même une erreur)

Ce n'est pas obligatoire en javascript mais on doit l'utiliser pour utiliser une variable locale ayant le même non qu'une variable globale.

A+.
17/11/2012, 15h19
Auteur

Citation:

Envoyé par andry.aime

Bonsoir,

Ce n'est pas obligatoire en javascript mais on doit l'utiliser pour utiliser une variable locale ayant le même non qu'une variable globale.

A+.

Que ta variable soit locale ou globale, il faut le mettre :
http://javascript.developpez.com/faq...arer-variables ;)

J'ai suivi vos conseils, j'ai donc repris l'ensemble de la fonction

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
39
//met a jour le montant de la Commission Epargne 
function maj_montant_commission_epargne()
{
		//recupere le montant du versement epargne
		var versement=parseFloat(document.getElementById('montant_versement_epargne').value);	
 
		//pourcentage de la commission epargne
		var pourcentage=parseFloat(document.getElementById('pourcentage_versement_epargne').value);
 
		//plafond de la commission epargne a ne pas depasser
		var plafond=parseFloat(document.getElementById('plafond_commission_epargne').value);
		plafond=round(plafond,2);
 
 
		//montant calcule
		var commission=(versement*pourcentage)/100;
		commission=round(commission,2);
		versement=round(versement,2);
 
 
		if(plafond!=0)
		{
			//si le montant de la commission epargne est inferieur au plafond epargne
			if(commission<plafond)
			{
				//ecrit le montant de la commission 
				document.getElementById('affichage_montant_commission_epargne').value=commission;
				document.getElementById('montant_commission_epargne').value=commission;
			}
			//sinon la commission correspond au plafond
			else
			{ 
				document.getElementById('affichage_montant_commission_epargne').value=plafond;
				document.getElementById('montant_commission_epargne').value=plafond;
			} 
 
		}
		document.getElementById('montant_versement_epargne').value=versement;
}

J'ai un versement de x euros, je calcule par rapport au pourcentage la commission a verser à 4,5%. Si je dépasse le plafond alors le plafond s'applique.

Seulement, mon plafond est à 1500 €, je met un versement entre 0 et 45 €, il me met bien une commission correcte, dès que je mets 45 € de versement, il me prends le plafond.

Exemple concret, je mets 100 € de versement, à 4.5%
dans le js en mettant des alertes, j'obtiens :

if(commission<plafond)
versement vaut 100.00
commission vaut 4.50

mon plafond qui vaut 1500.00 est selectionne

Salut,

Je suppose que la fonction "round()" a été définie par toi ou quelqu'un d'autre ? Il y a peut-être un problème à ce niveau là.

J'ai essayé avec ma propre méthode, et ça fonctionne bien.

Je t'explique le fonctionnement, au cas où tu serais intéressée
Code:

1 2 3 4 5 function round(x,y){ y=Math.pow(10,y); x=Math.round(x*y)/y; return x; }
x est le nombre à arrondir
y est le nombre de décimales voulues
Si le nombre doit être arrondi à 2 décimales, y vaudra 10^2=100
Ensuite, il multiplie x par 100 (y), avant d'appeler la méthode Math.round() (qui arrondi à l'entier le plus proche).
Puis il la divise par 100 (y), afin d'avoir 2 chiffres après la virgule.
10.53564 / multiplié par 100 (y)
1053.564 / arrondi à l'entier le plus proche (Math.round())
1054 / divisé par 100 (y)
10.54

Évidemment, y sera calculé à 1000 si l'on veut 3 décimales (10^3), ce qui aura pour effet d'arrondir x à 3 décimales, etc.

Le code entier, qui fonctionne bien chez moi :
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 39 40 41 42 43 44 45 46 // arrondit le nombre x à y décimales function round(x,y){ y=Math.pow(10,y); x=Math.round(x*y)/y; return x; } //met a jour le montant de la Commission Epargne function maj_montant_commission_epargne() { //recupere le montant du versement epargne var versement=parseFloat(document.getElementById('montant_versement_epargne').value); //pourcentage de la commission epargne var pourcentage=parseFloat(document.getElementById('pourcentage_versement_epargne').value); //plafond de la commission epargne a ne pas depasser var plafond=parseFloat(document.getElementById('plafond_commission_epargne').value); plafond=round(plafond,2); //montant calcule var commission=(versement*pourcentage)/100; commission=round(commission,2); versement=round(versement,2); if(plafond!=0) { //si le montant de la commission epargne est inferieur au plafond epargne if(commission<plafond) { //ecrit le montant de la commission document.getElementById('affichage_montant_commission_epargne').value=commission; document.getElementById('montant_commission_epargne').value=commission; } //sinon la commission correspond au plafond else { document.getElementById('affichage_montant_commission_epargne').value=plafond; document.getElementById('montant_commission_epargne').value=plafond; } } document.getElementById('montant_versement_epargne').value=versement; }

29/11/2012, 22h19
SylvainPV

Bien trouvé la fonction Michael :ccool:
ça marche même avec les négatifs : round(1054,-1) === 1050
Simple et efficace, ils devraient compléter le prototype de Math.round pour ça...
30/11/2012, 08h35
SpaceFrog

http://javascript.developpez.com/faq...ript#NBDecimal
30/11/2012, 09h05
Michaeljackfan

Citation:

Bien trouvé la fonction Michael
ça marche même avec les négatifs : round(1054,-1) === 1050
Simple et efficace, ils devraient compléter le prototype de Math.round pour ça...

Merci, ça fait toujours plaisir !

Cela-dit, puisque la méthode var.toFixed(x) existe, mieux vaut l'utiliser, elle est sûrement plus optimisée. En plus, on ne risque pas d'avoir une erreur parce qu'on a oublié d'importer la fonction.
30/11/2012, 09h07
SpaceFrog

ça fait toujours plaisir que la FAQ ne serve a rien :roll:
30/11/2012, 10h58
SylvainPV

Citation:

Envoyé par Michaeljackfan

Merci, ça fait toujours plaisir !

Cela-dit, puisque la méthode var.toFixed(x) existe, mieux vaut l'utiliser, elle est sûrement plus optimisée. En plus, on ne risque pas d'avoir une erreur parce qu'on a oublié d'importer la fonction.

Oui enfin attention, toFixed cast en string et on peut avoir des surprises avec. C'est bien pour l'affichage mais pour les calculs, je préfère round