Discussion :

[zazze69]

Bonjour,

Je suis plus que débutant en javascript et j'ai fait une petite formule de calcul dans un formulaire

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
function calcul()
{
 
 
	var rendement = document.getElementById("rendement");
	var puissance   = document.getElementById("puissance").value;
	var zone       = document.getElementById("zone").value;
	var perte       = document.getElementById("perte").value;
if( objForm.puissance.selectedIndex == 0 )
{
alert( "Champ puissance obligatoire !" );
return false;
}
else {
	rendement.value = eval(puissance + ' * ' + zone + ' *' + perte);
	var final = document.getElementById("final");
	var rendement2 = document.getElementById("rendement").value;
	var tarif = document.getElementById("tarif").value;
	final.value = eval(rendement2 + ' * ' + tarif );
 
	return false;}
}

Les id puissance, department et perte son en fait des "select" dans mon formulaire.

J'aimerais afficher une alerte si l'un de ces champs n'est pas renseigné, comme vous pouvez le voir je testé quelque chose mais en vain.
Si vous avez des idées pour optimisé mon code de calcul je suis preneur aussi
Merci pour votre indulgence et vos bons conseils.

[zazze69]

Bon j'ai cherché un peu et j'ai fait ça:

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
function calcul()
{
 
	var rendement = document.getElementById("rendement");
	var puissance   = document.getElementById("puissance").value;
	var zone       = document.getElementById("zone").value;
	var perte       = document.getElementById("perte").value;
	if (document.formulaire1.puissance.value == 'puissance'){
        alert('Veuillez choisir 1blabla'); }
		else if (document.formulaire1.zone.value == 'dep'){
        alert('Veuillez choisir 2blabla'); }
			else if (document.formulaire1.perte.value == 'ori'){
			alert('Veuillez choisir 3blabla'); }
				else if (document.formulaire1.tarif.value == ''){
				alert('Veuillez 4blabla'); }
	else {	
	rendement.value = eval(puissance + ' * ' + zone + ' *' + perte);
	var final = document.getElementById("final");
	var rendement2 = document.getElementById("rendement").value;
	var tarif = document.getElementById("tarif").value;
	final.value = eval(rendement2 + ' * ' + tarif );
 
	return false;}
}

Ca fonctionne nickel mais le hic c'est que le formulaire est envoyé et la page se recharge donc il faut re-remplir les champs....Une idée?

[zazze69]

C bon j'ai rajouté un retrun false à chaque ligne de verif et ca marche nickel:

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
function calcul()
{
 
	var rendement = document.getElementById("rendement");
	var puissance   = document.getElementById("puissance").value;
	var zone       = document.getElementById("zone").value;
	var perte       = document.getElementById("perte").value;
	if (document.formulaire1.puissance.value == 'puissance'){
        alert('Veuillez choisir 1blabla'); return false;
		}
		else if (document.formulaire1.zone.value == 'dep'){
        alert('Veuillez choisir 2blabla'); return false;
		}
			else if (document.formulaire1.perte.value == 'ori'){
			alert('Veuillez choisir 3blabla');return false;
		}
				else if (document.formulaire1.tarif.value == ''){
				alert('Veuillez 4blabla');return false;
		}
	else {	
	rendement.value = eval(puissance + ' * ' + zone + ' *' + perte);
	var final = document.getElementById("final");
	var rendement2 = document.getElementById("rendement").value;
	var tarif = document.getElementById("tarif").value;
	final.value = eval(rendement2 + ' * ' + tarif );
 
	return false;}
}

Avant d'aller me la péter vers mon entourage parce que e me suis débrouillé comme un grand y'a t il qqun pour me donner un avis quand à l'optimisation de code ??
Il n'ya pas encore d'envoi en bdd c juste une "calculette" pour l'instant.
Merci

[Bovino]

Déjà, si tu fais

Code :

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

var rendement = document.getElementById("rendement");

à quoi ça te sers dans la condition de récupérer une nouvelle fois le champ ?

Code :

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

if (document.formulaire1.puissance.value == 'puissance')

Code :

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

if (puissance == 'puissance')

est largement suffisant et préférable...

Code :

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

rendement.value = eval(puissance + ' * ' + zone + ' *' + perte);

Il ne faut pas utiliser eval() !!!
Surtout que dans ce cas précis, son utilité est à démontrer !

Code :

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

rendement.value = puissance * zone * perte;

[SpaceFrog]

voir la FAQ
http://www.developpez.net/forums/d55...on-formulaire/

[zazze69]

OK merci pour les amélioration a apporter !!

Pour ce qui est de la validation je vais lire la FAQ je commence a m’intéresser au JavaScript j'aimerais bien apprendre ce langage plus profondément avant d'utiliser les lib comme jquery , prototype...etc.

J'aime bien comprendre ce que je fait et ce site est vraiment géniale pour avancer.

Merci encore.

[zazze69]

J'ai lu la FAQ et je crois avoir saisi les prémices de la validation en js, mais il s'agit d'une fonction js.

Comment faire pour l'incorporer à ma fonction de calcul?