Test nombre inférieur à un autre

**devlm** · 16/11/2012, 14h15

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 :

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
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;
}

**Michaeljackfan** · 16/11/2012, 17h57

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 :

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

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 :

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

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 :

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
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;
}

**andry.aime** · 16/11/2012, 18h13

Bonsoir,

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

**Auteur** · 17/11/2012, 15h19

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

**devlm** · 29/11/2012, 18h11

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

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

**Michaeljackfan** · 29/11/2012, 22h04

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 :

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

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 :

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
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;
}

**SylvainPV** · 29/11/2012, 22h19

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

**SpaceFrog** · 30/11/2012, 08h35

http://javascript.developpez.com/faq...ript#NBDecimal

**Michaeljackfan** · 30/11/2012, 09h05

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.

**SpaceFrog** · 30/11/2012, 09h07

ça fait toujours plaisir que la FAQ ne serve a rien

**SylvainPV** · 30/11/2012, 10h58

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

Test nombre inférieur à un autre

JavaScript

Discussions similaires

Partager

Partager