Ajouter des données dans la base de donner SQL et C#

Question

Bonsoir 

je suis un débutant en programmation et je suis entrait de faire une petite application en C# mais là je suis bloc pour faire l'ajouter des données dans la base de donnée je ne sais vraiment ce quoi l'erreur aide moi SVP voila les code dont j'utilise pour faire l'ajoute:

private void bAjouter_Click(object sender, EventArgs e)
        {
            if(cnx.State==0)
            cnx.Open();
            //affecté les données dans les colonne du datagrid
            string numE = Numéro.HeaderText;
            string nom = Column1.HeaderText;
            string P = Column2.HeaderText;
            string sal=Column3.HeaderText;
            string rq = @"INSERT INTO EMPLOYE(numE,nom,prenom,salaire) values('" + numE + "'," + nom + ",'" + P + "',"+sal+"')";
        SqlCommand cmd = new SqlCommand(rq, cnx);
        int resultat = cmd.ExecuteNonQuery();
        if (resultat > 0)
            MessageBox.Show("L'Ajouter Effectuée");
        cnx.Close();
        
        }
dans mon interface il y a juste un datagride et trois bouton Ajouter,Modifie et Supprimer


bonne fête de fin d'année à tous

cs_jopop · Answer

Salut,

avec le code erreur, le nom de l'exception ou sa description complète, se serait bien plus simple de t'aider ;)

PS : le nom de la propriété HeaderText me laisse supposer qu'elle ne contient pas les valeurs que tu souhaites.

Tupad · Answer

On peut remarquer que dans ta requête "INSERT INTO EMPLOYE(numE,nom,prenom,salaire) values('" + numE + "'," + nom + ",'" + P + "',"+sal+"')", il manque des ' autour de certains champs. 

De plus je te conseille d'utiliser les paramètres des commandes SQL pour toutes entrées utilisateurs, ca t'évitera bien des problèmes.

Pense aussi à placer l'exécution de la requête dans un try catch pour éviter de planter l'application suite à une erreur SQL.

Il existe également une autre solution si tu utilises une dataset ou une datatable pour afficher tes données dans ton datagrid. Ces objets ont une method AcceptChanges qui modifient directement les données dans la BD. Plus d'informations sur le lien suivant : http://msdn.microsoft.com/fr-fr/library/ceab2k93.aspx

mandess · Answer

l'ajouter dont je veux faire avec le datagride ça ne marche pas toujours je ne sais pas ce ne marche avec ma requête aide moi SVP 

merci

Tupad · Answer

Tu peux faire une méthode du style :

public void bAjouter_Click(object sender, EventArgs e) 
{ 
bool ok = true;
foreach (DataGridViewRow row in this.dataGrid.Rows)
{
if(row.IsNewRow)
ok &= this.AddRecord(row);
}
if(ok)
MessageBox.Show("Rows inserted);
}

public bool AddRecord(DataGridViewRow row)
{
bool ok = false;
try
{
if(cnx.State==0) 
cnx.Open(); 

string rq = @"INSERT INTO EMPLOYE(numE,nom,prenom,salaire) values(@numE, @nom, @P, @sal)"; 
SqlCommand cmd = new SqlCommand(rq, cnx); 
cmd.Parameters.AddWithValue("@numE", row.Cells[0]); 
cmd.Parameters.AddWithValue("@libelle", row.Cells[1]); 
cmd.Parameters.AddWithValue("@libelle", row.Cells[2]); 
cmd.Parameters.AddWithValue("@libelle", row.Cells[3]); 

int resultat = cmd.ExecuteNonQuery(); 
ok = (resultat > 0);
}
catch(SQLException ex)
{
// Logger l'exception sql
}
finally
{
cnx.Close(); 
}
return ok;
}


Le code n'est absolument pas testé, mais la logique y est.

mandess · Answer

Bonsjoi je viens de test avec ce code ton tu ma donnée ça ne marche pas toujours ça revoie une message d'erreur comme "aucun mappage existe à partir du type d'objet systeme.windows.form.datadrideviewstexteboxcll ver un type inconnu de fournisseur natif managé"

merci pour l'aide

Tupad · Answer

Tu peux copier le code que tu utilises?

mandess · Answer

voila le code que j'utilise:

 private void bAjouter_Click(object sender, EventArgs e)
        {
            bool ok = true;
foreach (DataGridViewRow row in this.dgv.Rows)
{
if(row.IsNewRow)
ok &= this.AddRecord(row);
}
if(ok)
MessageBox.Show("Rows inserted");

}

public bool AddRecord(DataGridViewRow row)
{
bool ok = false;
try
{
if(cnx.State==0) 
cnx.Open(); 

string rq = @"INSERT INTO EMPLOYE(numE,nom,prenom,salaire) values(@numE, @nom, @Prenom, @salaire)"; 
SqlCommand cmd = new SqlCommand(rq, cnx); 
cmd.Parameters.AddWithValue("@numE", row.Cells[0]); 
cmd.Parameters.AddWithValue("@libelle", row.Cells[1]); 
cmd.Parameters.AddWithValue("@libelle", row.Cells[2]); 
cmd.Parameters.AddWithValue("@libelle", row.Cells[3]); 

int resultat = cmd.ExecuteNonQuery(); 
ok = (resultat > 0);
}
catch(Exception ex)
{
// Logger l'exception sql
    MessageBox.Show("erreur "+ ex.Message, "erreur", MessageBoxButtons.OK, MessageBoxIcon.Error);
}
finally
{
cnx.Close(); 
}
return ok;

Tupad · Answer

Commence par changer

cmd.Parameters.AddWithValue("@libelle", row.Cells[1]);
cmd.Parameters.AddWithValue("@libelle", row.Cells[2]);
cmd.Parameters.AddWithValue("@libelle", row.Cells[3]); 


par 


cmd.Parameters.AddWithValue("@nom", row.Cells[1]);
cmd.Parameters.AddWithValue("@Prenom", row.Cells[2]);
cmd.Parameters.AddWithValue("@salaire", row.Cells[3]); 


et reessaie.

Tu peux nous donner la structure de la table SQL dans laquelle tu essaies d'insérer tes données? Le nom des champs ainsi que leur type

mandess · Answer

Bonjour

la structure de la table est la suivant:

numE type int 
nom type varchar
prenom type varchar
salaire type money

le code dont j'ai modifier toujours la mm erreur

BarthOlivier · Answer

Salut, je suis peu habitué à SQL mais tu peux faire comme sous mysql ? (je l'ai adapté a ton code) :

      StringBuilder SB = new StringBuilder();
      SB.Append("INSERT INTO EMPLOYE(numE,nom,prenom,salaire) values(");
      SB.Append(row.Cells[0].ToString() + ",");
      SB.Append(row.Cells[1].ToString() + ",");
      SB.Append(row.Cells[2].ToString() + ",");
      SB.Append(row.Cells[3].ToString() + ",");
      SB.Append(")");

      SqlCommand cmd = new SqlCommand(SB.ToString(), cnx); 
      int resultat = cmd.ExecuteNonQuery();

MisterMok

BarthOlivier · Answer

Modif -> SB.Append(row.Cells[3].ToString()) pour la dernière ligne

MisterMok

Tupad · Answer

Oui mais dans ce cas, je vois deux problèmes.
- Tu as oublié les guillemets pour les champs de type varchar (ici les lignes avec les paramètres 1 et 2)
- Il vaut mieux utiliser les paramètres des commandes SQL pour éviter les problème de sécurité par des injections SQL.

Sinon tant que ca fonctionne...

Ajouter des données dans la base de donner SQL et C#

12 réponses

Votre réponse

Discussions similaires