Discussion :

[paintbox]

Bonjour,

je suis novice en JavaScript.
J'ai un tableau qui comporte des prix et sur ma page, j'ai des input de type "number". Les input se nomment "number1", "number2"...
Les prix des number sont stockés dans mon tableau ci-dessous.

Code :

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

1
2
 
carte = new Array(11.50, 16.50, 12.50, 10.50);

j'accède sans problème à mes prix et peux les additionner en faisant:

Code :

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

1
2
 
total = parseFloat(carte[1]) + parseFloat(carte[2]);

Je parcours avec une boucle for mon formulaire pour voir quels sont les champs de type "number" qui ont une valeur > 0.
Puis je vais cherche le prix du "number" correspondant dans mon tableau et souhaite additionner les prix des différents qui ont une valeur > 0 et les stocke dans une variable total.
C'est là que cela ne fonctionne plus. J'ai soit NAN ou undefined.
Voici ma boucle ou j'additionne les prix

Code :

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

1
2
3
4
5
6
 
for(var Indice =1; Indice <= 3; Indice++){
if(document.getElementById('number' + Indice).value > 0){
 total = total + parseFloat(carte[Indice - 1]);
}
}

Quelqu'un aurait-il une idée.

Merci pour votre aide.

[ProgElecT]

Salut

Ton array carte = new Array(11.50, 16.50, 12.50, 10.50); contient 4 éléments d'indice 0 à 3.
Ta variable Indice =1; Indice <= 16 prend pour valeur 1 à 16.
Tu affectes/calcules total = total + parseFloat(carte[Indice - 1]);, alors que l'on a vue que ton array carte lui n'allait pas au delà de 3.

[paintbox]

Bonjour ProgElect,

merci pour ta réponse.
En fait mon tableau contient 16 prix mais ici pour l'exemple j'en ai mis que 3 mais je n'ai pas adapté ma boucle for. En effet l'indice max de ma boucle est bien également à mon max tableau.

Ceci dit le problème n'est pas là je pense.

C'est corrigé... comme ça pas de confusion

[paintbox]

Problème finalement résolu en faisant ceci :

Code :

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

1
2
3
4
5
6
 for(var Indice = 1; Indice <= 16; Indice++){
      if(document.getElementById('number'+ Indice).value > 0){
         console.log('number'+Indice);
         total = parseFloat(total) + (parseFloat(carte[Indice-1] * document.getElementById('number'+ Indice).value));
      }
   }

Merci pour votre aide

[Beginner.]

Tu es sûr que cela fonctionne ? Je vois que tu utilises un coup Indice-1 et un coup Indice.

Tu pourrais économiser plusieurs soustractions en faisant commencer ta boucle à l'indice 0.
La fonction parseFloat() n'est pas utile puisque que le tableau carte contient déjà des float par contre dans if(document.getElementById('number'+ Indice).value > 0) tu compares une string au nombre 0...
Tu pourrais optimiser en utilisant une class mais sinon tu peux au moins mettre +document.getElementById('number'+ Indice).value (avec un "+" pour convertir en nombre) dans une variable car tu l'utilises plusieurs fois...

La variable "total" n'est pas déclarée...

[paintbox]

Bonjour Beginner.,

merci pour tes conseils. En fait je commence à 1 parce que ma liste de plats commence à 1, mais c'est vrai que je peux simplifier en commençant à 0. C'est corrigé.

Effectivement en supprimant le parseFloat cela fonctionne aussi. J'avais lu que si on ne l'utilise pas, il prend le nombre comme un string.

Merci pour ton aide