requete sql via odbc "la propriété connection n'a pas été initialisée ."

Version imprimable

Salut !

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

Voici mon code :

Code:

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 :

Citation:

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

16/05/2010, 09h41
olibara

Citation:

Envoyé par kelvir

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
16/05/2010, 21h37
kelvir

L'habitude de bosser avec odbc ! 8-)

Personne n'a d"idées ???
16/05/2010, 22h07
olibara
Citation:

Envoyé par kelvir

L'habitude de bosser avec odbc ! 8-)
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:

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:

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.
17/05/2010, 18h15
kelvir

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

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 :P
17/05/2010, 19h00
kelvir

Arf!!!! Nouveau problème :aie:

Citation:

Syntaxe incorrecte vers AIP

La textbox 7 contient : AIP Robots
17/05/2010, 21h23
hegros

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

Code:

INSERT INTO TABLE VALUES(AIP Robots);

et non pas

Code:

INSERT INTO TABLE VALUES AIP Robots;
17/05/2010, 21h50
kelvir

Bah non c'est une variable !

Code:

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

:calim2:
17/05/2010, 21h58
hegros

Je voulais dire comme ça

Code:

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...
17/05/2010, 22h02
Nathanael Marchand

Requêtes parametrées!
http://msdn.microsoft.com/fr-fr/libr...arameters.aspx
17/05/2010, 22h03
kelvir

Bah justement , textbox7.text contient AIP robot :mouarf:
17/05/2010, 22h05
hegros

Citation:

Envoyé par kelvir

Bah justement , textbox7.text contient AIP robot :mouarf:

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

La requête finale devrait être

Code:

INSERT INTO TABLE VALUES(AIP Robots);

et pas

Code:

INSERT INTO TABLE VALUES AIP Robots;

Tu vois la différence ?
17/05/2010, 22h09
Nathanael Marchand

Ca marche ca une chaine à l'arrache et sans quotes?
17/05/2010, 22h15
StormimOn

Citation:

Envoyé par PitMaverick78

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.
17/05/2010, 22h23
hegros

Citation:

Envoyé par PitMaverick78

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.
17/05/2010, 22h24
kelvir

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

Voici le code qui fonctionne :

Code:

string CommandText = string.Format("INSERT INTO [table] VALUES ('"+textBox7.Text+"') ");
17/05/2010, 22h26
StormimOn

Citation:

Envoyé par hegros

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
17/05/2010, 22h28
kelvir

Citation:

Envoyé par StormimOn

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 :zoubi:

Merci
17/05/2010, 22h29
hegros

Citation:

Envoyé par StormimOn

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
17/05/2010, 22h54
olibara

Oups !

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

Citation:

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 )