Problème de conversion en double : Trop de chiffres après la virgule

**Scots** · 26/09/2014, 15h34

Bonjour,

Petit soucis :

J'ai un label affichant un prix (9,99).
Dans mon code behind, je le convertis en Double pour l'insérer en base (champ type double).

Je fais donc un

Code :

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

double prixttc = Convert.ToDouble(label.Text);

La conversion se fait bien mais j'obtiens 9.9900000000000002 ... Du coup ça ne rentre pas en base.

Quelqu'un aurait une idée ?

Merci d'avance.

**tomlev** · 27/09/2014, 18h55

Envoyé par Scots

J'ai un label affichant un prix (9,99).
Dans mon code behind, je le convertis en Double pour l'insérer en base (champ type double).

Déjà tu es mal parti... un label sert à afficher une valeur, et uniquement à ça. En aucun cas ça ne doit être le "stockage de référence" de la valeur. Tu devrais avoir la valeur d'origine quelque part dans une variable, une propriété d'un objet, une DataTable ou quelque chose comme ça, pas la relire à partir de l'interface graphique (sauf bien sûr dans le cas où c'est une valeur saisie par l'utilisateur)

D'autre part, le type Double n'est pas adapté pour les valeurs financières. Il faut utiliser le type Decimal, qui est fait pour ça.

Envoyé par Scots

Je fais donc un

Code :

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

double prixttc = Convert.ToDouble(label.Text);

La conversion se fait bien mais j'obtiens 9.9900000000000002 ... Du coup ça ne rentre pas en base.

Le type Double ne peut pas stocker des valeurs décimales exactes, parce qu'il représente un nombre en virgule flottante binaire ; à part quelques valeurs qui tombent juste, la valeur qui est effectivement stockée dans la variable est une approximation. En l'occurrence 9.99 n'a pas de représentation exacte en virgule flottante binaire.

Le type Decimal, en revanche, représente un nombre en virgule flottante décimale, et permet donc de stocker des valeurs décimales exactes. C'est pour ça qu'il faut toujours utiliser ce type pour des valeurs financières.

De plus, quand tu dis "ça ne rentre pas en base", que veux-tu dire exactement ? Quel est le type de la colonne dans la DB ?

**Scots** · 28/09/2014, 09h57

Merci Tomlev,

J'avais déjà essayé avec DECIMAL mais j'avais oublié le (10,2) et laissé le (10,0) par défaut ...

Du coup, ok. Et je vois mon erreur : Lorsque je fais ma conversion de String en Decimal, il met une virgule au lieu d'un point. Du coup j'ai une erreur sur le Insert.

**tomlev** · 28/09/2014, 13h15

Envoyé par Scots

Du coup, ok. Et je vois mon erreur : Lorsque je fais ma conversion de String en Decimal, il met une virgule au lieu d'un point. Du coup j'ai une erreur sur le Insert.

Un Decimal, en soit, ne contient pas de virgule ou de point dans un decimal ; c'est seulement dans sa représentation sous forme de chaine que ça apparaît.

Je pense que tu construis ta requête SQL en concaténant la valeur dans la chaine ; il ne faut absolument pas faire ça, c'est une source de bugs et de failles de sécurité (injection SQL). Il faut utiliser une requête paramétrée : http://johannblais.developpez.com/tu...s-donnees/#LIV

**Scots** · 28/09/2014, 15h20

Pas du tout, je ne fais aucune concaténation de chaine.

Je lis une valeur en base Table1, je l'affiche dans un Label, je clique sur un bouton, je l'insère en base Table2.

Je lis un décimal, je l'affiche en string et je veux le réinsérer en décimal ...

**tomlev** · 28/09/2014, 16h01

Envoyé par Scots

Je lis une valeur en base Table1, je l'affiche dans un Label, je clique sur un bouton, je l'insère en base Table2.

Fais voir le code que tu utilises pour l'insertion

**Scots** · 28/09/2014, 16h13

Voici le bout de code concerné :

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
decimal prixttc = Convert.ToDecimal(prixLabel.Text);
 
        //Récupération du dernier numéro de facture
        string CommandFact="SELECT numero_facture FROM facture LIMIT 1";
        MySqlConnection myConnectionFact = new MySqlConnection(ConnectionString);
        MySqlCommand myCommandFact = new MySqlCommand(CommandFact, myConnectionFact);
 
        myConnectionFact.Open();
        string fact = Convert.ToString(myCommandFact.ExecuteScalar());
        myConnectionFact.Close();
 
        string numFact = fact + 1;
 
        //On stock en session les infos utiles
        Session["Internaute_Email"] = user;
        Session["Abonnement_Id"] = idLabel.Value;
 
        //On créé la facture
        string Command = "INSERT INTO facture (numero_facture, email, produit, ttc, date_facture, paiement) VALUES ('" + numFact + "', '" + user + "', '" + idLabel.Value + "', '" + prixttc + "', '" + datecr + "', '0')";
        MySqlConnection myConnection = new MySqlConnection(ConnectionString);
        MySqlCommand myCommand = new MySqlCommand(Command, myConnection);
 
        myConnection.Open();
        myCommand.ExecuteScalar();
        myConnection.Close();

**tomlev** · 28/09/2014, 17h05

Donc c'est bien ce que je disais, tu crées ta requête par concaténation :

Code :

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

string Command = "INSERT INTO facture (numero_facture, email, produit, ttc, date_facture, paiement) VALUES ('" + numFact + "', '" + user + "', '" + idLabel.Value + "', '" + prixttc + "', '" + datecr + "', '0')";

Il ne faut jamais inclure la valeur directement dans la requête.
- Ca pose des problèmes de format des nombres et des dates, car ils ne sont pas formatés pareil selon la langue ; par exemple en français les nombres décimaux s'écrivent avec une virgule, et en SQL il faut les écrire au format américain, donc avec un point. C'est pour ça que tu as des problèmes pour insérer la valeur en base.
- C'est une faille béante de sécurité, en particulier dans une application web (ce qui semble être ton cas), car ton code est vulnérable à une attaque par injection SQL.

Regarde le tutoriel que je t'ai indiqué plus haut pour utiliser une requête paramétrée.

**Scots** · 28/09/2014, 17h16

J'avais cru que tu me parlais de concaténation de champs, oui tu raison. Je fais déjà ça pour tester mais bien sûr j'avais prévu de faire une requête paramétrée.

Tu pense que mon problème serait résolu avec ça ???