Discussion :

[farfarawi]

Bonjour tous le monde ,

Je suis en train de programmer une application sur C# avec sql server 2008 , et je suis confronté à un problème que j'ai essayé de contourner par tous les moyens mais j'y arrive pas et j'ai besoin d'aide mon problème est le suivant :
sur mon visual studio le type double est reconnu sous la forme (##,#) mais sur sql server le typle Float est reconnu sous la forme (##.#) alors quand j’exécute mon code avec des virgules je me retrouve avec une erreur "Column name or number of supplied values does not match table definition.", ce qui est normal car sql server met les virgules entre les champs et donc il prend la variable double comme 2 champs ex(remise =12,5 il prend 12 comme valeur et 5 comme une autre valeur ).

J'ai essayé de faire :

Code C# :

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

1
2
3
string v = "0";
v = tb_pu_ht.Text.Replace(",", ".");
pu_ht = Convert.ToDouble(v, new CultureInfo("en-US"));

Mais au niveau de SqlCommand il ne prend toujours que la virgule même si je remplace , par .

Voici le code C# :

Code C# :

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

1
2
3
4
 
  cmd.CommandText = "insert into ligne_devis values (" + pu_ht + "," + qte + ",'" + tb_conditionnement.Text + "'," + remise + "," + pu_net + "," + montant_ht + "," + montant_ttc + "," + cb_tva.Text + ")";
  cmd.Connection = con;
  cmd.ExecuteNonQuery();

Et le code SQL :

Code SQL :

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

1
2
3
4
5
6
7
8
9
10
CREATE TABLE [dbo].[ligne_devis](
	[pu_ht] [float] NOT NULL,
	[qte] [int] NOT NULL,
	[conditionnement] [nvarchar](250) NULL,
	[remise] [float] NULL,
	[pu_net] [float] NULL,
	[montant_ht] [float] NOT NULL,
	[montant_ttc] [float] NULL,
	[tva] [float] NULL
)

Merci de me donner votre avis sur ce point ou une solution

[Jamming Ed]

A mon avis :

Code :

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

1
2
v = tb_pu_ht.Text.Replace(",", ".");
pu_ht = Convert.ToDouble(v, new CultureInfo("en-US"));

Le Replace ne sert pas à grand chose puisque la conversion en double fera réapparaître la virgule !

Il vaudrait mieux faire :

Code :

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

cmd.CommandText = "insert into ligne_devis values (" + tb_pu_ht.Text.Replace(",", ".") + "," + qte + ",'" + tb_conditionnement.Text + "'," + remise + "," + pu_net + "," + montant_ht + "," + montant_ttc + "," + cb_tva.Text + ")";

edit : petite erreur dans ma réponse

[PixelJuice]

Pour apporter plus d'infos :

Lorsque tu mets ton double dans la requête SQL , une conversion en string implicite est faite , et lorsqu'on converti un double en string , le "." se transforme en ","
c'est pour ça que autant utiliser directement le string comme Jamming Ed l'a montré.

[farfarawi]

Merci infiniment
Jamming Ed et PixelJuice vous me sauvez la vie j'ai tt essayé sauf ça hhh
probléme résolu ouuuf

[Jaco67]

Il faudrait surtout que tu utilises les requêtes paramatrées, ça t'éviterait ce genre de problème, sans compter les injections sql...

Quelque chose comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
cmd.CommandText = "insert into ligne_devis values (@pu_ht, @qte, @conditionnement, @remise, @pu_net, @montant_ht, @montant_ttc, @tva)";
cmd.Parameters.AddWithValue("@pu_ht", pu_ht);
cmd.Parameters.AddWithValue("@qte", qte);
cmd.Parameters.AddWithValue("@conditionnement", tb_conditionnement.Text);
cmd.Parameters.AddWithValue("@remise", remise);
cmd.Parameters.AddWithValue("@pu_net", pu_net);
cmd.Parameters.AddWithValue("@montant_ht", montant_ht);
cmd.Parameters.AddWithValue("@montant_ttc", montant_ttc);
cmd.Parameters.AddWithValue("@tva", cb_tva.Text);
cmd.Connection = con;
cmd.ExecuteNonQuery();

[DotNetMatt]

Sinon, il suffit tout simplement d'utiliser les requêtes paramétrées. Plus besoin de s'occuper du format, et en prime ça protège contre les attaques par injection de SQL...

Il ne faut pas hésiter à les utiliser de partout, à chaque fois qu'il y a besoin de passer des paramètres à une requête. C'est une best-practice.

[EDIT] Grillé par Jaco67

[farfarawi]

Il faudrait surtout que tu utilises les requêtes paramatrées, ça t'éviterait ce genre de problème, sans compter les injections sql...

Quelque chose comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
cmd.CommandText = "insert into ligne_devis values (@pu_ht, @qte, @conditionnement, @remise, @pu_net, @montant_ht, @montant_ttc, @tva)";
cmd.Parameters.AddWithValue("@pu_ht", pu_ht);
cmd.Parameters.AddWithValue("@qte", qte);
cmd.Parameters.AddWithValue("@conditionnement", tb_conditionnement.Text);
cmd.Parameters.AddWithValue("@remise", remise);
cmd.Parameters.AddWithValue("@pu_net", pu_net);
cmd.Parameters.AddWithValue("@montant_ht", montant_ht);
cmd.Parameters.AddWithValue("@montant_ttc", montant_ttc);
cmd.Parameters.AddWithValue("@tva", cb_tva.Text);
cmd.Connection = con;
cmd.ExecuteNonQuery();

Oui c'est vrai je viens de voir cette façon de faire je suis prenant pour cette méthode merci bcp pour votre réactivité

[DotNetMatt]

N'oublie pas de marquer le sujet comme résolu si tu as obtenu une réponse satisfaisante.

[ed73170]

Bonjour,

Plutôt que Replace, tu peux utiliser ceci :

pu_ht.ToString(CultureInfo.InvariantCulture)