Discussion :

[kakashi05]

Bonjour

En fait j'ai une winform et j'aimerai inserer des donnees dans ma base
j'utilise le mode deconnecté je vous montre mon code la precision la partie SELECT marche bien c'est l'INSERT qui ne marche pas

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
 
 
 private void button_ajouter_Click(object sender, EventArgs e)
        {
            //on va utiliser un Insert
            myDa.InsertCommand = new SqlCommand();
            myDa.InsertCommand.Connection = connection;
            myDa.InsertCommand.CommandText = "INSERT INTO T_AVION(AV_MODELE, AV_CONSTRUCTEUR) VALUES('" + txt_model + "','" + txt_construc + "')";
 
 
            myDa.Update(myDs, "T_AVION");
            dataGridView1.DataSource =myDs.Tables["T_AVION"];
 
 
 
 
private void button_affichage_Click(object sender, EventArgs e)
        {
            //on va utiliser la fonction select
            myDa.SelectCommand = new SqlCommand();
            myDa.SelectCommand.Connection = connection;
            myDa.SelectCommand.CommandText = "SELECT * FROM T_AVION";
 
            myDa.Fill(myDs, "T_AVION");
            dataGridView1.DataSource = myDs.Tables["T_AVION"];
 
        }
 
Si quelqu'un à une idee merci

[kakashi05]

je viens de remarquer que cette ligne provoque un bug

Code :

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

1
2
 
myDa.Update(myDs, "T_AVION");

donc voila le nouveau 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
 
private void button_ajouter_Click(object sender, EventArgs e)
        {
            //on va utiliser un Insert
            myDa.InsertCommand = new SqlCommand();
            myDa.InsertCommand.Connection = connection;
            myDa.InsertCommand.CommandText = "INSERT INTO T_AVION(AV_MODELE, AV_CONSTRUCTEUR) VALUES('" + txt_model + "','" + txt_construc + "')";
 
            MessageBox.Show("information saisie");
 
            dataGridView1.DataSource =myDs.Tables["T_AVION"];
 
 
 
 
private void button_affichage_Click(object sender, EventArgs e)
        {
            //on va utiliser la fonction select
            myDa.SelectCommand = new SqlCommand();
            myDa.SelectCommand.Connection = connection;
            myDa.SelectCommand.CommandText = "SELECT * FROM T_AVION";
 
            myDa.Fill(myDs, "T_AVION");
            dataGridView1.DataSource = myDs.Tables["T_AVION"];
 
        }

[dormilon]

Il te manque le SqlCommandBuilder, avec lui tu ne t'embêtes pas à créer le INSERT il le fera tout seul.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
using( SqlConnection connection = new SqlConnection( stringConnection.ConnectionString ) )
{
                SqlDataAdapter adapter = new SqlDataAdapter( "SELECT * FROM T_AVION", connection );
                SqlCommandBuilder cmd = new SqlCommandBuilder( adapter );
                DataTable tableAvion = new DataTable();                
                adapter.Fill(  tableAvion );              
}
 
//Remplissage du DataGridView
dataGridView1.DataSource = tableAvion;
 
//Mise à jour de la base par rapport au info entrées dans le DataGridView
adapter.Update( tableAvion );

[kakashi05]

Merci pour ta reponse mais lorsque je remplis mes textboxs et lorsque je clique sur le bouton il m'affiche juste le contenu de ma datagrivew sans nouvelle donnee


en fait je m'explique un peu c'est une winform avec 2 textbox qui represent le Modele et le Constructeur de l'avion et un bouton ajouter et afficher pour ajouter les donnees dans la datagrivew j'utilise le mode deconnecté pour faire le mes requets dans la base

voila l'integralité du 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
 
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Data.SqlClient;
namespace Test11
{
    public partial class Form1 : Form
    {
        public SqlConnection connection = new SqlConnection();
        public SqlDataAdapter myDa = new SqlDataAdapter();
        public DataSet myDs = new DataSet();
 
        public Form1()
        {
            InitializeComponent();
        }
 
        private void Form1_Load(object sender, EventArgs e)
        {
            //connection à la base
            connection.ConnectionString = "Data Source=pc-de-sangare\\sqlexpress;Initial Catalog=SaberatourSQL;Integrated Security=True";
 
        }
 
        private void button_ajouter_Click(object sender, EventArgs e)
        {
 
 
            //On reprend tout ce passage
 
            //on va utiliser un Insert
 
           myDa.InsertCommand = new SqlCommand();
 
           myDa.InsertCommand.Connection = connection;
           myDa.InsertCommand.CommandText = "INSERT INTO T_AVION(AV_MODELE, AV_CONSTRUCTEUR) VALUES('" + txt_model + "','" + txt_construc + "')";
 
           dataGridView1.DataSource= myDs.Tables["T_AVION"];
 
        }
 
private void button_affichage_Click(object sender, EventArgs e)
        {
            //on va utiliser la fonction select
            myDa.SelectCommand = new SqlCommand();
            myDa.SelectCommand.Connection = connection;
            myDa.SelectCommand.CommandText = "SELECT * FROM T_AVION";
 
            myDa.Fill(myDs, "T_AVION");
            dataGridView1.DataSource = myDs.Tables["T_AVION"];
 
        }

surtout n'hesité pas si vous avez besoin de precision

[dormilon]

As tu initialisé la propriété DataPropertyName dans les colonnes créées pour le DataGridView? Elle doit avoir le même nom que le nom de ta colonne dans la base, donc AV_MODELE et AV_CONSTRUCTEUR.

[kakashi05]

je l'ai pas fait et je vois pas vraiment comment le faire c'est ca le gros probleme

[dormilon]

Regarde si dans la méthode InitializeComponent tu as des new DataGridViewTextBoxColumn ou new DataGridViewXXColumn avec XX egal à ComboBox ou TextBox ou autre.


Sinon voila comment je créer mes colonnes si ça peut aider.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
addColumn( new DataGridViewTextBoxColumn(), "Titre de la colonne"," Nom dans la base" );
 
private void addColumn( DataGridViewColumn viewColumn, string name, string dbName )
        {
            viewColumn.AutoSizeMode = DataGridViewAutoSizeColumnMode.Fill;
            viewColumn.DataPropertyName = dbName;
            viewColumn.Name = name;
            viewColumn.HeaderText = name;
            dataGridView.Columns.Add( viewColumn );
        }

[kakashi05]

Merci à dormilon pour ton aide j'ai finalement pu ecrire dans la base directement sans passé par la datagridview

voila le code que j'ai utiliser pour le INSERT

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
SqlConnection cnt = new SqlConnection();
            SqlCommand cmd = new SqlCommand();
            cnt.ConnectionString = connection.ConnectionString;
            cmd.Connection = cnt;
            cmd.CommandType = CommandType.Text;
 
            cmd.CommandText = "INSERT INTO T_AVION(AV_ID,AV_MODELE,AV_CONSTRUCTEUR) values('" + txt_type.Text.ToString() + "','" + txt_model.Text.ToString() + "','" + txt_construc.Text.ToString() + "')";
            if (cnt.State == ConnectionState.Closed)
                cnt.Open();
            cmd.ExecuteNonQuery();
            if (cnt.State == ConnectionState.Open)
                cnt.Close();

[Louis-Guillaume Morand]

Code :

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

 if (cnt.State == ConnectionState.Closed)

ça tu pourrais t'en passer.
vu que tu ne l'ouvres pas, elle est forcément fermée la connexion.

Ensuite, tu peux utiliser le mot-clé using pour gérer la destruction propre de la connexion

Code :

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

1
2
3
4
5
 using (SqlConnection connection = new SqlConnection(connectionString))
    {
        connection.Open();
      blablabla
    }

le code fermera la connexion tout seul comme un grand


Par contre, ta technique d'INSERT est très très très mauvaise. si je tape

');DELETE FROM T_AVION;--

dans la case numéro 2
1- le programme ne va pas planter
2- ca va supprimer toutes les données de ta base
*le code est pas exactement cela mais avec plusieurs essais, c'est très facile de deviner le format de la requete d'origine et de l'attaquer

bref, tu es une porte ouverte à l'injection SQL. Utilise des requêtes paramétrées même si c'est pour un petit projet, il faut prendre des bonnes habitudes dès le départ

[kakashi05]

Merci toute ses infos