Discussion :

[splifferwolf]

Bonjour tout le monde,

je viens vers vous pour un problème d'addition depuis des chaîne de caractère...

J'ai ceci :
50.05 €;
25.36 €;

J'ai donc utiliser un replace pour remplacer les " €" par rien (donc supprimer les € et espace vide) puis un parseFloat pour les nombre flottant.

Code :

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

1
2
3
4
5
6
7
8
 
a = "50.05 €"
b = "25.36 €"
 
a = parseFloat(a.replace(" €","")); // Aucune erreur et avec un console.log il me met bien 50.05
b = parseFloat(b.replace(" €","")); // Aucune erreur et avec un console.log il me met bien 25.36
 
alert(a+b); // Résultat 50.0525.36

Mon problème est donc que plutôt que d'additionner il concatène ce qui n'est pas l'objectif évidemment...
Quelqu'un aurait déjà eu ce soucis ?

D'avance merci

[SpaceFrog]

Code :

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

alert(+a+b);

[NoSmoking]

Bonjour,
en l'état je ne vois pas de lézard !

Le code ci-dessous devrait même suffire

Code :

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

1
2
a = parseFloat(a);
b = parseFloat(b);

A vérifier qu'il n'y a pas un caractère non affiché en début de chaine.

[Gnuum]

Ce code ne fonctionnera pas de toute façon à cause des erreurs d'arrondis des floats.
Quand tu fais des calculs de ce type, il faut repasser en entier.


Si tu es sûr d'avoir des centimes (même quand ça vaut "00"), voilà ce que tu peux faire (je suis resté volontairement proche de ton code):

Code JavaScript :

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

1
2
3
4
5
6
7
8
9
a = "50.05 €"
b = "25.36 €"
 
 
var aInt = parseInt(a.replace(/[\.€\s]/g, ''), 10);
var bInt = parseInt(b.replace(/[\.€\s]/g, ''), 10);
 
 
alert((aInt + bInt) / 100);

[NoSmoking]

Dans ta solution il faut impérativement que le nombre de décimal soit le même sans oublier les zéros non significatifs.

[danielhagnoul]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
const
  number1 = '10 235,24',
  number2 = '13714.05';
 
function addEuroString( strNum1 = "1 000,25", strNum2 = "0.50 €"){
  let
    nb1 = parseFloat( strNum1.replace( /,/g, '.' ).replace( /[^0-9.]/g, '') ),
    nb2 = parseFloat( strNum2.replace( /,/g, '.' ).replace( /[^0-9.]/g, '') );
 
  return ( nb1 + nb2 ).toString( 10 ).replace( /\./g, ',' ) + " €";
}
 
console.log( "1 000,25 + 0.50 = ", addEuroString() );
console.log( "number1 + number2 = ", addEuroString( number1, number2 ) );

[SpaceFrog]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
a = "50.05 €"
b = "25.36 €"
 
function Numberize(num){
return +num.match(/[\d.]+/) 
} 
 
function ADDCurrencies(P1, P2){
return Numberize(P1) + Numberize(P2) + " €";
} 
 
alert( ADDCurrencies(a,b))

https://jsfiddle.net/cxhLvub6/


ou encore

Code :

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

1
2
3
4
a = "50.05 €", b= "25.36 €";
var N = (n) => {return +n.match(/[\d.]+/);};
var C = (p1,p2) => {return N(p1) + N(p2) + " €";}
alert( C(a,b));

et pour le fun

Code :

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

1
2
3
a = "50.05 €", b= "25.36 €";
var C = (x,y) => {r=/[\d.]+/; return +x.match(r) + +y.match(r) + " €";}
alert( C(a,b));

[Invité]

Bonjour les zinzins

[EDIT] Je reprends mes billes... (erreur)

N.B. ...vouloir faire des calculs en JavaScript relève de la psychiatrie !

[Gnuum]

Dans ta solution il faut impérativement que le nombre de décimal soit le même sans oublier les zéros non significatifs.

C'est pour ça que j'ai bien précisé:

Si tu es sûr d'avoir des centimes (même quand ça vaut "00")

Vraiment, je ne conseille pas de calcul de prix avec des floats (et ce n'est pas propre au JavaScript). Ca induit des erreurs d'arrondis à coup sûr (car D dense dans D).

[eleydet]

Bonjour,

Le code du premier message de cette discussion, celui proposé par splifferwolf, fonctionne normalement avec Firefox chez moi. Je suis surpris "que plutôt que d'additionner il concatène"...

Avec quel navigateur il concatène au lieu d'additionner?

Sinon, la variante ci-dessous devrait fonctionner aussi.

Code :

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

1
2
3
4
5
6
7
a = "50.05 €";
b = "25.36 €";
 
a = 1*(a.replace("€",""));
b = 1*(b.replace("€",""));
 
alert(a+b);

[SpaceFrog]

normal avec 1* ...

[Auteur]

Bonjour les zinzins

Bonjour les artistes

je ne sais pas pourquoi mais quand j'ai vu les codes postés, j'ai repensé à cette discussion

[NoSmoking]

C'est pour ça que j'ai bien précisé:
(...)

effectivement, mauvaise interprétation de ma part

Si je reprends juste le 1er code :
alert( a+b ); // ici, on a CONCATENATION (des nombres ou chaines)

le nombre de parenthèses n'influe pas

Code :

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

1
2
3
4
5
6
7
8
9
var a = '1';
var b = '2';
alert(a + b);  // -> 12
alert((a + b)) // -> 12
 
a = 1;
b = 2;
alert(a + b);  // -> 3
alert((a + b)) // -> 3

N.B. ...vouloir faire des calculs en JavaScript relève de la psychiatrie !

c'est peut être un peu exagéré quand même, il n'y a pas que des calculs de panier, mais il est vrai que les calculs, et surtout l'affichage, doivent être fait en connaissance des soucis liés aux flottants, ceci ayant déjà été largement abordés sur le forum.


Le soucis que rencontre splifferwolf n'est visiblement pas un problème d'arrondi mais de bien rendu de résultat et donc

en l'état je ne vois pas de lézard !

on va donc attendre son intervention pour plus ample informations.

[Watilin]

Je rajoute que la confusion entre concaténation et addition est surtout liée aux types des variables. Si on fait toujours attention aux types, on n’a pas de soucis.

J’admets que le faible typage de JavaScript autorise certaines mauvaises pratiques ; et que ça oblige à avoir une certaine connaissance du langage pour comprendre le problème. Mais ce n’est pas exclusif à JS, et on est plutôt bien lotis à ce niveau si on compare à PHP, pour qui 'php' == 0 par exemple.

En adoptant de bonnes habitudes on évite la plupart des problèmes. Entre autres :

• Toujours parser les nombres quand on n’est pas sûr de leur type (en particulier quand ils viennent d’un champ de formulaire ; j’ai eu la désagréable surprise de voir que les <input type="number"> avaient une valeur de type string ) ;
• Toujours faire des comparaisons strictes === et !==, au lieu de == et !=.
• Grouper entre parenthèses, même si ce n’est pas nécessaire, quand on veut additionner des variables de différents types, par exemple :
  

  Code :

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

  bidule.style.left = (e.pageX + 42) + 'px';