Discussion :

[kelvir]

Salut !

J'ai un soucis pour faire un insert sur une bdd mssql 2005 via odbc

Voici mon code :

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
26
27
28
29
 public void button3_Click(object sender, EventArgs e)
        {
 
 
 
            OdbcConnection connection = new System.Data.Odbc.OdbcConnection("DSN=" + textBox2.Text + ";UID="+textBox3.Text+";PWD="+textBox4.Text+";");
 
 
 
 
 
                connection.CreateCommand();
              OdbcCommand myCommand = new OdbcCommand("INSERT INTO [table] VALUES " + textBox7.Text + " , connection");
 
              try
              {
                  connection.Open();
                  myCommand.ExecuteNonQuery();
              }
              catch (Exception ex)
              {
                  MessageBox.Show(ex.Message);
              }
 
 
 
 
 
        }

Voici l'erreur :

ExecuteNonQuery : la propriété connection n'a pas été initialisée .

[olibara]

connection.CreateCommand();
OdbcCommand myCommand = new OdbcCommand("INSERT INTO [table] VALUES " + textBox7.Text + " , connection");

D'apres moi tu n'associe pas bien ta connection et ta commande !!

Mais (meme si ca ne change rien a ton problème de syntaxe) pourquoi utilise tu odbc alors que VS propose le connecteur ADO pour SQL Server

[kelvir]

L'habitude de bosser avec odbc !

Personne n'a d"idées ???

[olibara]

L'habitude de bosser avec odbc !
Personne n'a d"idées ???

A mon avis c'est le moment de changer d'habitude puisque ca ne marche pas
Personellement je n'ai pas l'habitude d'ODBC mais mon expérience de ADO me permet de supputer une erreur dans ton code que je j'avais montré dans le précédant message

Code :

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

new OdbcCommand("INSERT INTO [table] VALUES " + textBox7.Text + " , connection");

moi je ferais plutot (en priant tres fort pour que textBox7.Text contienne une syntaxe correcte pour l'insert)

Code :

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

1
2
string CommandText=string.Format("INSERT INTO [table] VALUES {0}",textBox7.Text);
new OdbcCommand(CommandText , connection);

Mais je trouve que cette maniere de faire est plus que pas propre.

[kelvir]

Je vais essayer ta méthode ! (un peu de bricolage peut pas faire de mal )

PS : Que reproche tu a odbc ??? Je m'en sert sur plusieurs autres support et je n'ai jamais rencontré de problème ! L'avantage est que s'est plus "adaptatif" , on peut changer le sgbd sans changer la source

[kelvir]

Arf!!!! Nouveau problème

Syntaxe incorrecte vers AIP

La textbox 7 contient : AIP Robots

[hegros]

La syntaxe du INSERT n'est pas bonne. Les valeurs sont obligatoirement entre () il me semble..

Code :

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

INSERT INTO TABLE VALUES(AIP Robots);

et non pas

Code :

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

INSERT INTO TABLE VALUES AIP Robots;

[kelvir]

Bah non c'est une variable !

Code :

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

string CommandText = string.Format("INSERT INTO [table] VALUES "+textBox7.Text+" ");

[hegros]

Je voulais dire comme ça

Code :

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

string CommandText = string.Format("INSERT INTO [table] VALUES ("+textBox7.Text+")");

A moins que textBox7.Text contient la string "(AIP Robots)"

Ps : voir les requêtes paramétrées ou procédure stockée parce que là ca craint un peu...

[Nathanael Marchand]

Requêtes parametrées!
http://msdn.microsoft.com/fr-fr/libr...arameters.aspx

[kelvir]

Bah justement , textbox7.text contient AIP robot

[hegros]

Bah justement , textbox7.text contient AIP robot

Donc c'est faux Elles sont où les parenthèses pour VALUES ???

La requête finale devrait être

Code :

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

INSERT INTO TABLE VALUES(AIP Robots);

et pas

Code :

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

INSERT INTO TABLE VALUES AIP Robots;

Tu vois la différence ?

[Nathanael Marchand]

Ca marche ca une chaine à l'arrache et sans quotes?

[StormimOn]

Ca marche ca une chaine à l'arrache et sans quotes?

J'en doute, une raison de plus pour passer par des requêtes paramétrées. On s'évite pas mal de galères avec ^^

Et sinon, un INSERT d'une seule valeur sans préciser le champ me semble étrange. Parce qu'il y a au moins un champ de clé primaire en plus du champ texte que tu veux remplir. Il faut indiquer explicitement les champs en général.

[hegros]

Ca marche ca une chaine à l'arrache et sans quotes?

C'est approximatif à l'arrache, de toute façon VALUES doit être suivi par ( et se terminer par ) ce qui n'est pas le cas dans son exemple.

[kelvir]

Effectivement , c'est juste un exemple Test pour vérifier le modèle .


Voici le code qui fonctionne :

Code :

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

string CommandText = string.Format("INSERT INTO [table] VALUES ('"+textBox7.Text+"') ");

[StormimOn]

C'est approximatif à l'arrache, de toute façon VALUES doit être suivi par ( et se terminer par ) ce qui n'est pas le cas dans son exemple.

Le problème ce ne sont pas les parenthèses, qui doivent être effectivement présentent, mais le fait qu'une chaîne doit être entre quote (i.e. 'ma chaine').

Encore une fois, le meilleur moyen d'éviter ce genre de désagréments ce sont les requêtes paramétrées. C'est simple à mettre en place, ça évite les problèmes d'injection, ça gère les quotes à l'intérieur d'une chaîne et j'en passe. Il y a un article à ce sujet dans la section .Net

[kelvir]

Le problème ce ne sont pas les parenthèses, qui doivent être effectivement présentent, mais le fait qu'une chaîne doit être entre quote (i.e. 'ma chaine').

Encore une fois, le meilleur moyen d'éviter ce genre de désagréments ce sont les requêtes paramétrées. C'est simple à mettre en place, ça évite les problèmes d'injection, ça gère les quotes à l'intérieur d'une chaîne et j'en passe. Il y a un article à ce sujet dans la section .Net

Je vais y jeter un coup d'oeuil

Merci

[hegros]

Le problème ce ne sont pas les parenthèses, qui doivent être effectivement présentent, mais le fait qu'une chaîne doit être entre quote (i.e. 'ma chaine').

Il y avait 2 problèmes les parenthèses et les quotes. L'un ou l'autre seul cela m'étonnerait que ca marche, il faut les 2.

Et je suis complètement d'accord avec toi sur l'utilisation des requêtes paramétrées pour éviter les problématiques que tu cites

[olibara]

Oups !

J'ai loupé plusieurs etapes de ce thead !
En gros ce qui a été di je l'avais dis hier

moi je ferais plutot (en priant tres fort pour que textBox7.Text contienne une syntaxe correcte pour l'insert)

Mais comme la souligné Stormimon je nécessaire d'utiliser une vraie requete parametrées plutot que prier pour ne pas faire d'erreur de syntaxe SQL a la saisie de Textbox7 (joli nom explicite par ailleurs )