Correction instruction Sql Insert into

**Ashfor76** · 18/11/2014, 09h49

Bonjour!
Je suis en train de concevoir une application de gestion du personnel

j'utilise une base de données access que j'ai importé sur visual c#

dans cette base j'ai la table Agent qui est en relation avec table service

j'ai crée un form qui contient à la fois les champs de la table Agent et Service

j'ai crée un bouton save pour enregistrer avec code mais je reçois l'erruer suivante:

ErreurSystem.Data.OleDbException: "Erreur de syntaxe dans l'instruction Insert into"

Voici le 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
try
            {
                connection.Open();
                OleDbCommand command = new OleDbCommand();
 
                command.Connection = connection;
 
                command.CommandText ="Insert into AGENT(Matricule,Nom_Prénom,Date_naiss,Lieu_naiss,Age,Sexe,Situation_matrimon,Type_agent,Corps,Cadre,CATEGORIE,Grade,Echelon,Adresse,Personne_à_contacter) values('" + textBox1.Text + "','" + textBox2.Text + "','" + dateTimePicker1.Value + "','" + textBox8.Text + "','" + textBox5.Text + "','" + comboBox1.SelectedValue + "','" + comboBox2.SelectedValue + "','" + comboBox3.SelectedValue + "','" + textBox3.Text + "','" + textBox6.Text + "','" + comboBox4.SelectedValue + "','" + comboBox5.SelectedValue + "','" + comboBox6.SelectedValue + "','" + textBox4.Text + "','" + textBox7.Text + "');";
 
                command.CommandText = "insert into SERVICE(Direction,Service,Fonction,Date affectation) values('" + textBox9.Text + "','" + textBox10.Text + "','" + textBox11.Text + "','" + textBox12.Text + "')";
                command.ExecuteNonQuery();       
 
                MessageBox.Show("Data Saved");
 
                connection.Close();
            }
 
            catch (Exception ex)
            {
                MessageBox.Show("Erreur" + ex);
            }
        }

SVP quelqu'un peut il m'aider
Merci!

**didate** · 18/11/2014, 09h55

Bonjour,

Pour éviter tout problème, il est vivement conseillé d'utiliser la classe OleDbParamater.

http://msdn.microsoft.com/fr-fr/libr...v=vs.110).aspx

http://www.java2s.com/Tutorial/CShar...terExample.htm

**StringBuilder** · 18/11/2014, 09h56

Utilise des requêtes paramétrées.

**Ashfor76** · 18/11/2014, 10h31

Envoyé par didate

Bonjour,

Pour éviter tout problème, il est vivement conseillé d'utiliser la classe OleDbParamater.

http://msdn.microsoft.com/fr-fr/libr...v=vs.110).aspx

http://www.java2s.com/Tutorial/CShar...terExample.htm

Ok,merci je vais essayer les requêtes paramétrées et voir ce que cela va donner

**Ashfor76** · 18/11/2014, 13h45

Envoyé par Ashfor76

Ok,merci je vais essayer les requêtes paramétrées et voir ce que cela va donner

J'ai use les requêtes paramétrées mais lors du Run,je reçois un message encore : (le paramètre @combox1 n'a pas de valeur par défaut)

**StringBuilder** · 18/11/2014, 14h04

Peut-être ton pilote ne supporte-t-il pas les "NamedParameters" ? A ce moment, il ne faut pas donner de nom à tes paramètres, et remplacer le @xxx par des "?" (attention, les paramètres doivent être passés à la collection "Parameters" dans le même ordre que dans la requête du coup !)

Ou alors tu utilises un SGBD qui utilise un autre préfixe de variable, par exemple ":" pour Oracle.

**Ashfor76** · 18/11/2014, 15h03

Envoyé par StringBuilder

Peut-être ton pilote ne supporte-t-il pas les "NamedParameters" ? A ce moment, il ne faut pas donner de nom à tes paramètres, et remplacer le @xxx par des "?" (attention, les paramètres doivent être passés à la collection "Parameters" dans le même ordre que dans la requête du coup !)

Ou alors tu utilises un SGBD qui utilise un autre préfixe de variable, par exemple ":" pour Oracle.

J'utilise Access2010 comme SGBD

**didate** · 18/11/2014, 15h13

Envoyé par Ashfor76

J'utilise Access2010 comme SGBD

Poste ton code....

**Ashfor76** · 18/11/2014, 16h00

Envoyé par didate

Poste ton 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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
 
 public partial class Form1 : Form
    {
        private OleDbConnection connection = new OleDbConnection();
 
        public Form1()
        {
            InitializeComponent();
            connection.ConnectionString = @"Provider=Microsoft.ACE.OLEDB.12.0;Data Source=C:\Users\USER\Documents\Visual Studio 2010\Projects\Perso\GA2P.mdb;
Persist Security Info=False;";
 
 
 
        }
 
 
        private void button1_Click(object sender, EventArgs e)
        {
            string marequête;
            string marequête1;
 
            try
            {
                marequête="Insert into AGENT(Matricule,Nom_Prénom,Date_naiss,Lieu_naiss,Age,Sexe,Situation_matrimon,Type_agent,Corps,Cadre,CATEGORIE,Grade,Echelon,Adresse,Personne_à_contacter) values(@textBox1,@textBox2,@dateTimePicker1,@textBox8,@textBox5,@comboBox1,@comboBox2,@comboBox3,@textBox3,@textBox6,@comboBox4,@comboBox5,@comboBox6,@textBox4,@textBox7)";
                OleDbCommand command = new OleDbCommand(marequête,connection);
 
                 command.Connection = connection;
                 comboBox1.SelectedIndex = 0;
                 command.CommandType = CommandType.StoredProcedure;
                command.Parameters.AddWithValue("?textBox1",textBox1.Text);
                command.Parameters.AddWithValue("?textBox2", textBox2.Text);
                command.Parameters.AddWithValue("?dateTimePicker1", dateTimePicker1.Value);
                command.Parameters.AddWithValue("?textBox8", textBox8.Text);
                command.Parameters.AddWithValue("?textBox5", textBox5.Text);
                command.Parameters.AddWithValue("?comboBox1", comboBox1.SelectedValue);
                command.Parameters.AddWithValue("?comboBox2", comboBox2.SelectedValue);
                command.Parameters.AddWithValue("?comboBox3", comboBox3.SelectedValue);
                command.Parameters.AddWithValue("?textBox3", textBox3.Text);
                command.Parameters.AddWithValue("?textBox6", textBox6.Text);
                command.Parameters.AddWithValue("?comboBox4", comboBox4.SelectedValue);
                command.Parameters.AddWithValue("?comboBox5", comboBox5.SelectedValue);
                command.Parameters.AddWithValue("?comboBox6", comboBox6.SelectedValue);
                command.Parameters.AddWithValue("?textBox4", textBox4.Text);
                command.Parameters.AddWithValue("?textBox7", textBox7.Text);
                command.Connection.Open();
                command.ExecuteNonQuery();
                command.Connection.Close(); 
 
                marequête1 = "insert into SERVICE(Direction,Service,Fonction,Date affectation) values(@textBox9,@ textBox10,@textBox11,@textBox12)";
                OleDbCommand command1 = new OleDbCommand(marequête1, connection);
                command1.Connection = connection;
 
                command1.Parameters.AddWithValue("@textBox9", textBox9.Text);
                command1.Parameters.AddWithValue("@textBox10", textBox10.Text);
                command1.Parameters.AddWithValue("@textBox11", textBox11.Text);
                command1.Parameters.AddWithValue("@textBox12", textBox12.Text);
 
 
                    command1.Connection.Open();
                    command1.ExecuteNonQuery();
 
                command1.Connection.Close();
 
                }
 
                catch (OleDbException ex)
            {
                MessageBox.Show("Erreur" + ex);
            }

ci dessous le code

**stolx_10** · 18/11/2014, 16h53

Bonjour

Cela me pose problème :

Code :

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

command.CommandType = CommandType.StoredProcedure;

le type de commande paramétré ici est pour les procédures stockées. Pour les requêtes "simples" il faut plutôt passer par un

Code :

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

command.CommandType = CommandType.Text;

non ?

et pour les requêtes paramétrées, essaie plutôt cela:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
marequête="Insert into AGENT(Matricule,Nom_Prénom,Date_naiss,Lieu_naiss,Age,Sexe,Situation_matrimon,
                                            Type_agent,Corps,Cadre,CATEGORIE,Grade,Echelon,Adresse,Personne_à_contacter) 
                   values(?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)";
// les paramètres de la requête, au nombre de 15, sont des ?
 
                OleDbCommand command = new OleDbCommand(marequête,connection);
                 command.Connection = connection; // pas besoin je pense, car c'est sous entendu sur la ligne juste au dessus
                 comboBox1.SelectedIndex = 0;
                 command.CommandType = CommandType.Text; // text et pas storedProcedure
 
                command.Parameters.Add("@textbox1", OleDbType.VarChar, 80).Value = textBox1.Text; //  OleDbType.VarChar, 80 est un exemple, tout dépend de la taille du champ cible
                // etc ...

C'est ce que j'ai fait dans un de mes projets via ODBC, qui appelle une procédure stockée cette fois-ci, mais c'est la même chose, juste faire attention au CommandType

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
OdbcCommand dataSQL = new OdbcCommand("{call " + C_A_Tables.GO_Site + C_A_Tables.GO_No_Societe  + ".dbo.PR_AGENTS_PREP_SELECTION(?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)}", GO_Init.conn);
 
                    dataSQL.Parameters.Add("@B_Edit", OdbcType.Char).Value = "0";
                    dataSQL.Parameters.Add("@BaseSOC", OdbcType.VarChar).Value = C_A_Tables.GO_Site + C_A_Tables.GO_No_Societe  ;
                    dataSQL.Parameters.Add("@module", OdbcType.VarChar).Value = MMain.GO_Nom_module;
                    dataSQL.Parameters.Add("@User", OdbcType.VarChar).Value = MMain.GO_Utilisateur;
                    dataSQL.Parameters.Add("@Actif", OdbcType.VarChar).Value = "0";
                    dataSQL.Parameters.Add("@Presents", OdbcType.VarChar).Value = APresents;
                    dataSQL.Parameters.Add("@Partis", OdbcType.VarChar).Value = APartis;
                    dataSQL.Parameters.Add("@Matricule", OdbcType.VarChar).Value = AMatricule;
                    dataSQL.Parameters.Add("@Nom", OdbcType.VarChar).Value = ANom;
                    dataSQL.Parameters.Add("@Car", OdbcType.VarChar).Value = ACarriere;
                    dataSQL.Parameters.Add("@TypeCar", OdbcType.VarChar).Value = ATypeCar;
                    dataSQL.Parameters.Add("@OrderBy", OdbcType.VarChar).Value = AOrderBy;
                    dataSQL.Parameters.Add("@restrict_centre", OdbcType.VarChar).Value = MMain.GA_Structure;
                    dataSQL.Parameters.Add("@GMFI_HABI", OdbcType.VarChar).Value = "";
                    dataSQL.Parameters.Add("@Date_Ref", OdbcType.VarChar).Value = "";
                    dataSQL.Parameters.Add("@CarSimple", OdbcType.VarChar).Value = ACarSimple;
                    dataSQL.Parameters.Add("@BNomJF", OdbcType.VarChar).Value = A_NomJeuneFille;

edit : dsl pour les multiples edit

**didate** · 18/11/2014, 17h46

Pour être plus bref essaye ça :

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
 
 
 private void button1_Click(object sender, EventArgs e)
        {
            string marequête;
            string marequête1;
 
            try
            {
                marequête="Insert into AGENT(Matricule,Nom_Prénom,Date_naiss,Lieu_naiss,Age,Sexe,Situation_matrimon,Type_agent,Corps,Cadre,CATEGORIE,Grade,Echelon,Adresse,Personne_à_contacter) values(@textBox1,@textBox2,@dateTimePicker1,@textBox8,@textBox5,@comboBox1,@comboBox2,@comboBox3,@textBox3,@textBox6,@comboBox4,@comboBox5,@comboBox6,@textBox4,@textBox7)";
                OleDbCommand command = new OleDbCommand(marequête,connection);
 
                 command.Connection = connection;
                 comboBox1.SelectedIndex = 0;
                 command.CommandType =CommandType.Text;
                command.Parameters.AddWithValue("@textBox1",textBox1.Text);
                command.Parameters.AddWithValue("@textBox2", textBox2.Text);
                command.Parameters.AddWithValue("@dateTimePicker1", dateTimePicker1.Value);
                command.Parameters.AddWithValue("@textBox8", textBox8.Text);
                command.Parameters.AddWithValue("@textBox5", textBox5.Text);
                command.Parameters.AddWithValue("@comboBox1", comboBox1.SelectedValue);
                command.Parameters.AddWithValue("@comboBox2", comboBox2.SelectedValue);
                command.Parameters.AddWithValue("@comboBox3", comboBox3.SelectedValue);
                command.Parameters.AddWithValue("@textBox3", textBox3.Text);
                command.Parameters.AddWithValue("@textBox6", textBox6.Text);
                command.Parameters.AddWithValue("@comboBox4", comboBox4.SelectedValue);
                command.Parameters.AddWithValue("@comboBox5", comboBox5.SelectedValue);
                command.Parameters.AddWithValue("@comboBox6", comboBox6.SelectedValue);
                command.Parameters.AddWithValue("@textBox4", textBox4.Text);
                command.Parameters.AddWithValue("@textBox7", textBox7.Text);
                command.Connection.Open();
                command.ExecuteNonQuery();
                command.Connection.Close(); 
 
                marequête1 = "insert into SERVICE(Direction,Service,Fonction,Date affectation) values(@textBox9,@ textBox10,@textBox11,@textBox12)";
                OleDbCommand command1 = new OleDbCommand(marequête1, connection);
                command1.Connection = connection;
 
                command1.Parameters.AddWithValue("@textBox9", textBox9.Text);
                command1.Parameters.AddWithValue("@textBox10", textBox10.Text);
                command1.Parameters.AddWithValue("@textBox11", textBox11.Text);
                command1.Parameters.AddWithValue("@textBox12", textBox12.Text);
 
 
                    command1.Connection.Open();
                    command1.ExecuteNonQuery();
 
                command1.Connection.Close();
 
                }
 
                catch (OleDbException ex)
            {
                MessageBox.Show("Erreur" + ex);
            }
}

Remarque : Le nom du paramètre dans la requête doit obligatoirement être identique à celui dans AddWithValue

**stolx_10** · 18/11/2014, 17h55

C'est très bien comme ça en effet.

Mais à quoi ça sert de faire des requêtes paramétrées si on ne force pas le type de valeurs lors de l'ajout des paramètres.
C'est justement l'intérêt des requêtes paramétrées, pouvoir passer en paramètre un type de donnée pour tel champ afin d'éviter des insertions de tout et n'importe quoi (injections SQL par exemple).

Ou alors le AddWithValue reconnaît directement le type de valeur ? le .Add n'est il pas mieux adapté aux requêtes paramétrées ?

J'avoue avoir proposé ma méthode qui me semble plus "contrôlable" en terme de données, mais je n'ai jamais utilisé AddWithValue, peut être est ce plus simple ?

**Ashfor76** · 19/11/2014, 11h58

Envoyé par stolx_10

C'est très bien comme ça en effet.

Mais à quoi ça sert de faire des requêtes paramétrées si on ne force pas le type de valeurs lors de l'ajout des paramètres.
C'est justement l'intérêt des requêtes paramétrées, pouvoir passer en paramètre un type de donnée pour tel champ afin d'éviter des insertions de tout et n'importe quoi (injections SQL par exemple).

Ou alors le AddWithValue reconnaît directement le type de valeur ? le .Add n'est il pas mieux adapté aux requêtes paramétrées ?

J'avoue avoir proposé ma méthode qui me semble plus "contrôlable" en terme de données, mais je n'ai jamais utilisé AddWithValue, peut être est ce plus simple ?

Merci à vous tous mais now j'ai ce problème: Nom : capture.jpg
Affichages : 184
Taille : 206,0 Ko

**didate** · 19/11/2014, 13h07

tu as oublié l'instruction suivante

Code :

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

1
2
 
command.Connection = connection;

Avant

Code :

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

 command.Connection.Open();

Oubien tu fais :

Code :

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

 command = new OleDbCommand(connection);

**katkiller** · 19/11/2014, 13h23

Envoyé par didate

tu as oublié l'instruction suivante

Le problème avec la popup d'erreur c'est qu'on suppose vu qu'on ne voit pas l'instanciation de 'OleDbCommand'

**Ashfor76** · 19/11/2014, 14h13

Envoyé par didate

tu as oublié l'instruction suivante

Code :

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

1
2
 
command.Connection = connection;

Avant

Code :

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

 command.Connection.Open();

Oubien tu fais :

Code :

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

 command = new OleDbCommand(connection);

j'ai procédé de la sorte Mamadou cette fois j'ai ce message :
Nom : capture2.png
Affichages : 179
Taille : 22,4 Ko

**katkiller** · 19/11/2014, 14h42

Tu instancie deux fois ta variable 'connection' : une fois dans le construteur (méthode 'Form1()') , et une fois au moment de ton click (méthode 'button1_Click(..)) sauf que la deuxième fois tu ne lui donne pas de valeur connectionString.