Simplification d"equation + Correction de bug

**guillaumeIOB** · 07/03/2012, 16h12

Bonjour à tous, ça faisait loingtemps que je ne vous avez pas embeté avec une de mes questions tordues...

Je suis en train t'ecrire un programme qui permet la création d'input en cliquant sur un lien, puis à chaque fois qu'une cellule est remplie, un total doit s'afficher dans la case colonne, juste en dessous...

Mon programme fonctionne presque, à savoir que tout est bon sauf que aprés avoir saisie une donnée, si on la modifie par la suite, on perd le total (au vu du programme ça parait logique puisque à chaque fois j'appel seulement la fontcion corrrespondant a la celulle, donc ne NB, qui n'additionne que cette seule celulle), mais en meme temps plus je solutionne et plus mon programme est une usine à gaz, et il faudrait que je la simplifie, mais je n'y arrive pas, meme avec des boucle "for", donc je sollicite, s'il vous plait votre aide, pour que je trouve enfin la solution a chaque modification de cellule....

Voici le code en entier :

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
 
<html>
 
<html>
 
<head>
<meta http-equiv="Content-Type" content="text/html; charset=windows-1252">
<meta name="GENERATOR" content="Microsoft FrontPage 4.0">
<meta name="ProgId" content="FrontPage.Editor.Document">
<title>Nouvelle page 1</title>
 <script type="text/javascript">
function totM(nb) {
 
if (nb==1){
mns1=document.simul.mns1.value; (mns1==NaN) ? mns1=0 : mns1=mns1; mns1 = (mns1*1);
mns=mns1;}
if (nb==2){
mns1=document.simul.mns1.value; (mns1==NaN) ? mns1=0 : mns1=mns1; mns1 = (mns1*1);
mns2=document.simul.mns2.value; (mns2==NaN) ? mns2=0 : mns2=mns2; mns2 = (mns2*1);
mns=mns1 + mns2;}
if (nb==3){
mns1=document.simul.mns1.value; (mns1==NaN) ? mns1=0 : mns1=mns1; mns1 = (mns1*1);
mns2=document.simul.mns2.value; (mns2==NaN) ? mns2=0 : mns2=mns2; mns2 = (mns2*1);
mns3=document.simul.mns3.value; (mns3==NaN) ? mns3=0 : mns3=mns3; mns3 = (mns3*1);
mns=mns1 + mns2 + mns3;}
if (nb==4){
mns1=document.simul.mns1.value; (mns1==NaN) ? mns1=0 : mns1=mns1; mns1 = (mns1*1);
mns2=document.simul.mns2.value; (mns2==NaN) ? mns2=0 : mns2=mns2; mns2 = (mns2*1);
mns3=document.simul.mns3.value; (mns3==NaN) ? mns3=0 : mns3=mns3; mns3 = (mns3*1);
mns4=document.simul.mns4.value; (mns4==NaN) ? mns4=0 : mns4=mns4; mns4 = (mns4*1);
mns=mns1 + mns2 + mns3 + mns4;}
if (nb==5){
mns1=document.simul.mns1.value; (mns1==NaN) ? mns1=0 : mns1=mns1; mns1 = (mns1*1);
mns2=document.simul.mns2.value; (mns2==NaN) ? mns2=0 : mns2=mns2; mns2 = (mns2*1);
mns3=document.simul.mns3.value; (mns3==NaN) ? mns3=0 : mns3=mns3; mns3 = (mns3*1);
mns4=document.simul.mns4.value; (mns4==NaN) ? mns4=0 : mns4=mns4; mns4 = (mns4*1);
mns5=document.simul.mns5.value; (mns5==NaN) ? mns5=0 : mns5=mns5; mns5 = (mns5*1);
mns=mns1 + mns2 + mns3 + mns4 + mns5;}
document.simul.M_AV.value= mns;
}
 
 
</script>
 
<script>
function create_credit(i) {
var i = i;
var i2 = i + 1;
document.getElementById('lecredit_'+i).innerHTML = '<input type="text" size="10" name="mns'+i+'" onchange=totM('+i+')>>';
document.getElementById('lecredit_'+i).innerHTML += (i <= 4) ? '<br><span id="lecredit_'+i2+'"><a href="javascript:create_credit('+i2+')">Ajouter un Crédit</a></span>' : '';
}
</script>
</head>
 
<body>
 
<form method="POST" action="" name="simul">
<span id="lecredit_1"><a href="javascript:create_credit(1)">Ajouter un credit</a></span><br /><br />
 
Totaux<br>
<input type="text" size="10" name="M_AV" value="0" ><br />
</form>
 
</body>
 
</html>

Avec tous mes remerciements ...

**SpaceFrog** · 07/03/2012, 16h18

un switch serait plus lisible ...

**Bovino** · 07/03/2012, 16h25

Plutôt un while() même...

**guillaumeIOB** · 07/03/2012, 16h27

Merci SpaceFrog mais le switch m'apporterait le meme nombre de ligne, j'ai tenté de le mettre en place :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
switch (nb) {
 case 1:
 Liste d'instructions; (1ligne)
 break;
 case 2:
 Liste d'instructions; (2ligne)
 break;
 case rX:
 Liste d'instructions; (3ligne)
 break;
etc....
 
}

-------------------------------------
Merci Bovino , mais pour le while, il se pose un pb comme pour le FOR :

C'est que pour x==1 il y aura qu'une seule ligne
mais pour x== 2 il faut deux lignes, etc....
Donc comment écrire une équation en boucle qui écrit autant de ligne que le

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
X
ex :
x=1;
while (x<=10) {
 if (x == 1) {
  ///// 1 ligne
  continue;
 }
 if (x == 2) {
  ///// 2 lignes
  continue;
 }
x++;
}

**SpaceFrog** · 07/03/2012, 16h36

pas si tu enlèves les breaks de façon astucieuse ...

**guillaumeIOB** · 07/03/2012, 16h42

Envoyé par SpaceFrog

pas si tu enlèves les breaks de façon astucieuse ...

Alors la je veux bien un exemple, s'il te plait, parce que a part gagner une ligne seulement par NB, je vois pas trop l'economie....

Mais je veux bien le voir ...

**SpaceFrog** · 07/03/2012, 16h51

Avec le while va falloir passer par des window[] pour les variables ...
ou alors modifer pour un array ...

avec le switch :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
mns=0;
switch (nb) {
 case 5 : mns5=document.simul.mns5.value; (mns5==NaN) ? mns5=0 : mns5=mns5; mns5 = (mns5*1);	
          mns+= mns5;
 case 4 : mns4=document.simul.mns4.value; (mns4==NaN) ? mns4=0 : mns4=mns4; mns4 = (mns4*1);
 			mns+=mns4
 case 3 : mns3=document.simul.mns3.value; (mns3==NaN) ? mns3=0 : mns3=mns3; mns3 = (mns3*1);
          mns+=mns3
 case 2 : mns2=document.simul.mns2.value; (mns2==NaN) ? mns2=0 : mns2=mns2; mns2 = (mns2*1);
 		   mns+=mns2
 case 1 : mns1=document.simul.mns1.value; (mns1==NaN) ? mns1=0 : mns1=mns1; mns1 = (mns1*1);
  			mns+= mns1
  			}

**Bovino** · 07/03/2012, 16h57

Code :

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

1
2
3
4
5
var i=0,
    mns = 0;
while(document.simul.elements['mns' + (++i)]){
    mns += (+document.simul.elements['mns' + i].value);
}

Au fait,

Code :

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

(mns1==NaN) ? mns1=0 : mns1=mns1; mns1 = (mns1*1)

C'est vraiment pas terrible !

Déjà, testes

Code :

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

alert(NaN == NaN);

pour voir que cette comparaison n'a aucun sens.
Ensuite,

Code :

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

mns1=mns1

l'intérêt me semble limité !

**guillaumeIOB** · 07/03/2012, 17h17

Envoyé par SpaceFrog

Avec le while va falloir passer par des window[] pour les variables ...
ou alors modifer pour un array ...

avec le switch :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
mns=0;
switch (nb) {
 case 5 : mns5=document.simul.mns5.value; (mns5==NaN) ? mns5=0 : mns5=mns5; mns5 = (mns5*1);	
          mns+= mns5;
 case 4 : mns4=document.simul.mns4.value; (mns4==NaN) ? mns4=0 : mns4=mns4; mns4 = (mns4*1);
 			mns+=mns4
 case 3 : mns3=document.simul.mns3.value; (mns3==NaN) ? mns3=0 : mns3=mns3; mns3 = (mns3*1);
          mns+=mns3
 case 2 : mns2=document.simul.mns2.value; (mns2==NaN) ? mns2=0 : mns2=mns2; mns2 = (mns2*1);
 		   mns+=mns2
 case 1 : mns1=document.simul.mns1.value; (mns1==NaN) ? mns1=0 : mns1=mns1; mns1 = (mns1*1);
  			mns+= mns1
  			}

Effectivement cela fonctionne sauf que si aprés avoir saisie plusieurs données on ne change qu'une seule, le total est faussé....

**SpaceFrog** · 07/03/2012, 17h18

faudrait benchmarker, mais je pense que mon code est plus rapide Beef

**guillaumeIOB** · 07/03/2012, 17h19

Envoyé par Bovino

Code :

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

1
2
3
4
5
var i=0,
    mns = 0;
while(document.simul.elements['mns' + (++i)]){
    mns += (+document.simul.elements['mns' + i].value);
}

Au fait,

Code :

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

(mns1==NaN) ? mns1=0 : mns1=mns1; mns1 = (mns1*1)

C'est vraiment pas terrible !

Déjà, testes

Code :

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

alert(NaN == NaN);

pour voir que cette comparaison n'a aucun sens.
Ensuite,

Code :

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

mns1=mns1

l'intérêt me semble limité !

ALORS LA CHAPEAU BAS MONSIEUR BOVINO ........
Non seulement c'est bcp bpc plus propre mais en plus ca solutionne le PB des modifications de cellules a posterior, moi je dis MERCI MONSIEUR BOVINO

**guillaumeIOB** · 07/03/2012, 17h21

Envoyé par SpaceFrog

faudrait benchmarker, mais je pense que mon code est plus rapide Beef

MERCI bcp de ton aide SpaceFrog, je ne sais pas si ton code est plus rapide, mais il maintient le second pb la bonne addition des celulles aprés une modification. Merci merci bcp de ton aide....

**SpaceFrog** · 07/03/2012, 17h24

ben il faut relance le calcul à chaque modification avec le nb correct ...

Simplification d"equation + Correction de bug

JavaScript

Discussions similaires

Partager

Partager