Problème requete SQL

Bonjour,
j'ai un problème a la modification de la variable QTDISP qui est de type double. Quand je clique sur la bouton pour modifier il s'affiche un message d'erreur. Si par exemple je mets 10,53 il m'affiche

Citation:

---

syntaxe incorrect vers '53'

---

et si je mets 10.53 il affiche aussi un message

Citation:

---

conversion invalide de la chaine 10.53 vers double.

---

Merci de m'aider svp

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

Private Sub Button1_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button1.Click   connexion.con.Open() ' ouverture de la connexion vers la base   Dim tx8 As Double = TextBox8.Text     Dim cmd3 As New SqlCommand("UPDATE [Base-Ets-Mtiri].[dbo].[ARTICLE] SET QTDISP=" & tx8 & " "WHERE ART='" & tx1 & "'", connexion.con) ' la commande cmd3.ExecuteNonQuery() connexion.con.Close()   End Sub

---

Bonsoir,

Rien qu'en regardant les couleurs de la requête on voit que le problème se situe au niveau de la construction de celle-ci...

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

Private Sub Button1_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button1.Click   connexion.con.Open() ' ouverture de la connexion vers la base   Dim tx8 As Double = TextBox8.Text     Dim cmd3 As New SqlCommand("UPDATE [Base-Ets-Mtiri].[dbo].[ARTICLE] SET QTDISP=" & tx8 & " "WHERE ART='" & tx1 & "'", connexion.con) ' la commande cmd3.ExecuteNonQuery() connexion.con.Close()   End Sub

---

Il ne faut jamais concaténer les requêtes.
Si vous ne voulez pas avoir se problème, utilisez les requêtes paramétrées (voir FAQ)...

Cherche de l'info sur les requêtes paramétrées

Citation:

---

Envoyé par StringBuilder

Cherche de l'info sur les requêtes paramétrées

---

Mais j'ai avec ce colonne 10 autre champ et ca marche bien sauf pour le cas ou je tape le type Double ce n'est pas un problème de la forme de la requête.

Ce n'est pas le type doule qui pose problème mais c'est la concaténation de champ, comme je l'ai marqué, regardez la colorisation automatique et au premier coup d'oeil on voit qu'il y a un problème... (Et par conséquent, on sait même le résoudre)

Le meilleur moyen d'éviter ce problème (et beaucoup d'autres) est dt'utiliser les requêtes paramétrées.

Donc regardez de ce coté.

Citation:

---

Envoyé par Sankasssss

Ce n'est pas le type doule qui pose problème mais c'est la concaténation de champ, comme je l'ai marqué, regardez la colorisation automatique et au premier coup d'oeil on voit qu'il y a un problème... (Et par conséquent, on sait même le résoudre)

Le meilleur moyen d'éviter ce problème (et beaucoup d'autres) est dt'utiliser les requêtes paramétrées.

Donc regardez de ce coté.

---

Mais non la différence de coloration parce que j'ai pas importé tous le code j'ai aucun erreur mon erreur est seulement sur le double j'ai dit que j'ai 10 textbox tous ca fonctionne bien de plus si je met un entier ca passe tres mais mais si je mais le double c'est le problème malgré que la colonne dans la base est de type double.

Donc le code que vous nous donné est mauvais, donc on cherche dans la mauvaise direction car il y a un guillemet en trop avant "WHERE ART"

donc ceci : (mauvaise coloration syntaxique)

Code:

---

Dim cmd3 As New SqlCommand("UPDATE [Base-Ets-Mtiri].[dbo].[ARTICLE] SET QTDISP=" & tx8 & " "WHERE ART='" & tx1 & "'", connexion.con) ' la commande

---

est cela : (bonne coloration syntaxique)

Code:

---

Dim cmd3 As New SqlCommand("UPDATE [Base-Ets-Mtiri].[dbo].[ARTICLE] SET QTDISP=" & tx8 & "WHERE ART='" & tx1 & "'", connexion.con) ' la commande

---

Le problème de conversion ne change rien à la réponse précédente, utilisez cette requête paramétrée et elle se chargera de faire les bonnes conversions, d'éviter les injections SQL, de corriger les string contenant des guillemets en les doublant etc. etc...

:idea: Des que des paramètres proviennent d'autre part que la requête en elle même, il faut prendre le pli d'utiliser ces requêtes paramétrées, question de bon sens.

Citation:

---

Envoyé par Sankasssss

Donc le code que vous nous donné est mauvais, donc on cherche dans la mauvaise direction car il y a un guillemet en trop avant "WHERE ART"

donc ceci : (mauvaise coloration syntaxique)

Code:

---

Dim cmd3 As New SqlCommand("UPDATE [Base-Ets-Mtiri].[dbo].[ARTICLE] SET QTDISP=" & tx8 & " "WHERE ART='" & tx1 & "'", connexion.con) ' la commande

---

est cela : (bonne coloration syntaxique)

Code:

---

Dim cmd3 As New SqlCommand("UPDATE [Base-Ets-Mtiri].[dbo].[ARTICLE] SET QTDISP=" & tx8 & "WHERE ART='" & tx1 & "'", connexion.con) ' la commande

---

Le problème de conversion ne change rien à la réponse précédente, utilisez cette requête paramétrée et elle se chargera de faire les bonnes conversions, d'éviter les injections SQL, de corriger les string contenant des guillemets en les doublant etc. etc...

:idea: Des que des paramètres proviennent d'autre part que la requête en elle même, il faut prendre le pli d'utiliser ces requêtes paramétrées, question de bon sens.

---

OH LA LA
J'ai dit que c'est un bout de code c'est pourquoi il y a le probleme dans le couleur mais autre fois mon problème dans le type double du tx8

Code:

---

1 2	Dim cmd3 As New SqlCommand("UPDATE [Base-Ets-Mtiri].[dbo].[ARTICLE] SET ART='" & tx1 & "',DESIG='" & tx2 & "',FAMILLE='" & tx3 & "',PVHTGRO=" & tx4 & ",REMIGRO='" & tx5 & "',REMIDET=" & tx6 & ",CTVA=" & tx7 & ", QTDISP=" & tx8 & ",QTINV=" & tx9 & "WHERE ART='" & tx1 & "'", connexion.con)

---

Citation:

---

Envoyé par oami89

OH LA LA
J'ai dit que c'est un bout de code c'est pourquoi il y a le probleme dans le couleur mais auuuuuuutre fois mon problème dans le type double du tx8

---

tu aurais pas fait la boulette, il n'y aura pas eut cette perte de temps, donc tu ne peux t'en prendre qu'à toi

as tu regardé le résultat de la concaténation avant qu'elle parte au sgbdr ?
quel est le sgbdr ?
quel est le paramétrage de windows (paramètres régionaux) et le paramétrage de ton sgbdr concernant le séparateur de décimal ? (et je vais le redire car tu ne sembles pas l'avoir compris, utiliser les dbparameters permet de se soustraire du paramétrage de windows et du sgbdr et donc d'éviter ce genre de bug !)

J'ai vous dit que ce n'est pas un problème de requete c'est un problème du float j'ai ajouté ce ligne seulement sans rien changer

Code:

---

tx8 = Replace(tx8, ",", ".")

---

et c'est bien marché.

Et on vous dit que l'on avait compris et que ce n'est pas la bonne méthode.
Un requête paramétrée l'aurait fait automatiquement ;) mais quand on est têtu... :mouarf:

Citation:

---

Envoyé par oami89

J'ai vous dit que ce n'est pas un problème de requete c'est un problème du float j'ai ajouté ce ligne seulement sans rien changer

Code:

---

tx8 = Replace(tx8, ",", ".")

---

et c'est bien marché.

---

ca a bien marché sur le pc sur lequel tu testes, mais tu trouveras un jour un pc sur lequel ca plantera et seules les requêtes paramétrés font que ca marchera sur tous les pc

Citation:

---

Envoyé par Pol63

ca a bien marché sur le pc sur lequel tu testes, mais tu trouveras un jour un pc sur lequel ca plantera et seules les requêtes paramétrés font que ca marchera sur tous les pc

---

Mais je suis débutant c'est mon 1er programme il faut que je le corrige maintenant puis je vais apprendre les requêtes paramétrés que je les connait pas si vous avez un exemple prêt merci de me donné.

Citation:

---

Envoyé par oami89

J'ai vous dit que ce n'est pas un problème de requete c'est un problème du float j'ai ajouté ce ligne seulement sans rien changer

Code:

---

tx8 = Replace(tx8, ",", ".")

---

et c'est bien marché.

---

Et si demain ton programme est utilisé par un anglais, tu fais comment ?

@Stringbuilder : merci de rester poli et calme stp.

@oami89 : tu es débutant ? Les gens qui t'ont répondu ici ne le sont pas. Et s'ils te conseillent d'utiliser les requêtes paramétrées, ce n'est pas par plaisir d'embêter les débutant. C'est parce que c'est une bonne pratique de développement qu'il est impératif que tu maîtrises si tu veux t'éviter de longues sessions de débugage de concaténations de chaînes de caractères et sécuriser ton application.

Les requêtes paramétrées te permettent de t'affranchir de tout un tas d'erreurs potentielles liées à des apostrophes, guillemets, points, virgules, etc.

Elles sont qui plus est très simples à utiliser :

Code:

---

1 2 3

Dim cmd3 As New SqlCommand("UPDATE [Base-Ets-Mtiri].[dbo].[ARTICLE] SET QTDISP=@QTDISP WHERE ART=@ART'", connexion.con) ' la commande cmd3.Parameters.Add("@QTDISP ", SqlDbType.LeType, LaLongueur).Value = tx1 cmd3.Parameters.Add("@ART", SqlDbType.LeType, LaLongueur).Value = tx1

---