Discussion :

[M1000]

Bonjour,

Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
 
   ttProd = 3;
   //Calcul du total de la commande
   var totalCommande = 0;
   for(x=0;x<ttProd;x++) {
      total_num = 'total_'+ x; //Récup le contenu de mon champ de formulaire
      var totalProd = document.getElementById(total_num).value;
      totalCommande = totalCommande+totalProd;
   }

cela m'affiche : 020 et non pas 20 ( qui est le résultat de 0+20)

PS: total_num sont des input type="text"

Ma question pourquoi je n'ai pas le résultat de 0+20 = 20 et 020 qui s'affiche ?

Merci.

[E.Bzz]

Bonjour,

Ma question pourquoi je n'ai pas le résultat de 0+20 = 20 et 020 qui s'affiche ?

parce que les .value sont de type caractère pas défaut, et que pour du caractère, le signe "+" représente la concaténation ...
parseInt() permet de convertire du texte en integer :

Code :

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

var totalProd = parseInt(document.getElementById(total_num).value, 10);

A+

[E.Bzz]

... ceci dit, il faut sortir la déclaration de ta boucle (var total_prod = ...).
Sinon, multidéclaration

A+

[M1000]

Quand je remplace cette ligne

Code :

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

1
2
 
var totalProd = document.getElementById(total_num).value;

par cette ligne

Code :

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

1
2
 
var totalProd = parseInt(document.getElementById(total_num).value, 10);

j'obtiens un NaN c'est ce que tu voulias dire en sortant ma déclaration de la boucle ? mais comment la sortir sachant que j'en ais besoin dans la boucle ?

Merci

[E.Bzz]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
   ttProd = 3;
var totalProd = 0;
var LaValeur;
   //Calcul du total de la commande
   var totalCommande = 0;
   for(x=0;x<ttProd;x++) {
      total_num = 'total_'+ x; //Récup le contenu de mon champ de formulaire
      LaValeur = document.getElementById(total_num).value;
      LaValeur = (isNaN(LaValeur))?"0":LaValeur;
      totalProd = parseInt(LaValeur, 10);
      totalCommande = totalCommande+totalProd;
   }

isNaN = is Not a Number
A toi d'ajouter un contrôle de saisie pour n'autoriser que des valeurs numériques ...

A+

[M1000]

Je dois avoir du mal mais j'ai tjrs un NaN lorsque je souhaite afficher ce dernier comme ceci :

Code :

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

1
2
 
document.getElementById("total").value = totalCommande;

[E.Bzz]

Peux-tu faire un copié/collé de la version actuelle de ton code ?

A+

[M1000]

Mon script :

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
 
<script language="javascript">
function caculLgPrice(prix, nbre, numProd, ttProd) {
   priceTTLg = prix*nbre;
   total_numProd = 'total_'+numProd;
   document.getElementById(total_numProd).value = priceTTLg;
 
   //Calcul du total de la commande
  var totalProd = 0;
  var LaValeur;
   //Calcul du total de la commande
   var totalCommande = 0;
   for(x=0;x<ttProd;x++) {
      total_num = 'total_'+ x; //Récup le contenu de mon champ de formulaire
      LaValeur = document.getElementById(total_num).value;
      LaValeur = (isNaN(LaValeur))?"0":LaValeur;
      totalProd = parseInt(LaValeur, 10);
      totalCommande = totalCommande+totalProd;
   }
  document.getElementById("total").value = totalCommande;
}
</script>

Mon HTML/PHP

Code :

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

1
2
3
4
5
6
 
echo '<form id="commande" enctype="multipart/form-data" method="post" autocomplete="off">';
echo '<div id="product_unid"><input type="text" name="unid_'.$prod.'" value="1" onChange="caculLgPrice('.$price.', this.value, '.$prod.', '.$nbreProd.' )"/></div>';
echo '<div id="product_bonif"><input type="text" name="bonif_'.$prod.'" id="bonif_'.$prod.'" value="" readonly /></div>';
echo '<div id="product_total"><input type="text" name="total_'.$prod.'" id="total_'.$prod.'" value="" readonly /></div>';
echo '</form>';

[E.Bzz]

Code :

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

priceTTLg = prix*nbre;

tes paramètres sont aussi de type string : à convertir avant de les traiter en numérique

A+

[M1000]

C'est la que tu te dis qu'il faut songer à dormir la nuit

Merci de l'aide c'est une affaire résolu !!!!