C# .NET insertion dans BD Orale

Bonjour,
je souhaiterai faire cette insertion mais j ai une erreur comme quoi "ExecuteNonQuery: Connection property has not been initialized."
est ce que quelqu un peu me donner une réponse clair, parceque je débute et c'est pas évident encore!!!

Merci 1000 fois

Sabine

Voici mon code:

private

void button1_Click( object sender, EventArgs e)

{
        

string insertCmd = "INSERT INTO user VALUES (seq_user.NEXTVAL,textBox1.Text,textBox2.Text);";
         DataConnection m = new DataConnection ();
         OleDbCommand cmd = new OleDbCommand (insertCmd);
         m.DoConnection();
         cmd.ExecuteNonQuery();
         m.DoDeconnection();
}

Salut,

Ce n'est pas parce que tu appelles DoConnection() que ton Command sait qu'il faut qu'il utilise cette connexion.
Il faut que tu affectes une instance d'OleDbConnection à la propriété Connection de ton OleDbCommand. Il faut donc que ta méthode DoConnection retourne ta connexion et dire a ton OleDbcommand de l'utiliser.

Mx
MVP C# 

bonjour,
J'ai une erreur comme quoi au niveau de ExecuteNonQuery, j ai un "invalid character", meme avec le prog en commentaire!
j essaye un peu tout et je fouille sur le net mais la j avous que je bloque!

merci pour votre aide
Sabine
voici mon code:

privatevoid button1_Click(object sender, EventArgs e)
{

string

insertCmd = "INSERT INTO user_mpl VALUES (1,"+textBox1.Text+","+textBox2.Text+");";
OleDbConnection connection = newOleDbConnection();
DataConnection m = newDataConnection();
connection=m.DoConnection();
OleDbCommand cmd = newOleDbCommand(insertCmd, connection);
int n=cmd.ExecuteNonQuery();
if (n != 1) { MessageBox.Show("insertion KO"); } elseMessageBox.Show("insertion GOOOOOD!");
m.DoDeconnection();

// String connect = "Provider=OraOLEDB.Oracle;"
//                                + "Data Source=***********;"
//                                + "User ID=*********;"
//                                 + "Password=***********;";

// OleDbConnection con = new OleDbConnection(connect);
// con.Open();
// OleDbCommand cmd = con.CreateCommand();
// cmd.CommandText = "INSERT INTO user_mpl VALUES (1,'blabla','joker');";
// cmd.ExecuteNonQuery();
// con.Close();

}

et voici ma connection:

public

classDataConnection
{
                privateOleDbConnection conn;
                privatestring connectString ="Provider=OraOLEDB.Oracle;"
                                + "Data Source=*************;"
                                + "User ID=***********;"
                                + "Password=***********;";
                public DataConnection()
                {
                    conn = newOleDbConnection(connectString);
                   // Le timeOut en secondes
                   //conn.ConnectionTimeout=60;
                 }
                //public void DoConnection()
                publicOleDbConnection DoConnection()
                 {
                 try
                {
                          // Connection
                          conn.Open();
                           Console.WriteLine("Connexion établit!!!");
                           return conn;
                 }
                 catch (OleDbException e)
                 {
                            Console.WriteLine("erreur remarquee:" + e);
                            return conn;
                   }
 }

publicvoid DoDeconnection()
{
try
{
conn.Close();
Console.WriteLine("Déconnexion à la base de données");
}
catch (Exception ex)
{
Console.WriteLine("erreur de deconnexion:" + ex);
}
}
}

Salut,

Plutot que de construire ta requete comme tu le fais, avec des concaténations, tu devrais plutot utiliser des requetes paramétrées (ce qui en plus d'etre plus propre et sécurisé, va certainement résoudre ton probleme)

En gros ca doit donner ca :
string insertCmd = "INSERT INTO user_mpl VALUES (1, ?, ?);";
cmd.parameters.add(textBox1.Text);
cmd.parameters.add(textBox2.Text);
cmd.ExecuteNonQuery();

De plus, il faut absolument que tu fermes ta connexion dans une clause finally (qui s'executera quoiqu'il arrive dans ton code, exception ou non).

try
{
    // Ton Code
}
catch{} // Gestion des exception
finally
{
   connection.Dispose();
   connection = null;
}

Ceci est valable aussi pour les fermeture de fichiers, ...

Mx
MVP C# 

J'oubliais, qques infos sur les requetes paramétrées avec oledb

[ Lien ]

Mx
MVP C# 

Merciiii beaucoup pour ton aide!
je teste tout ca!
Sabine

Tiens, un peu hors contexte mais quand même .. a propos du finally ... qu'apporte-t-il de plus ? Je veux dire, les 2 bloc d'instructiosn suivants font la même choses non ?
========================
BLOC A

try
{
    // Ton Code
}
catch{} // Gestion des exception
finally
{
   connection.Dispose();
   connection = null;
}
=====================
BLOCB

try
{
    // Ton Code
}
catch{} // Gestion des exception

connection.Dispose();
connection = null;

===============
Eric

Héhé, la question avait déja été posée au semblable ;)
Tel que tu l'as écrit, oui la connection sera libérée. Mais dans la réalité, tu ne fais pas un simple catch{ }. Tu peux tout a fait relancer l'exception au code appelant, voire ne pas catcher la bonne exception.
Tu peux aussi avoir un 'return qqchose' dans ton try ou ton catch. Le finally sera appelé, mais pas le code dans ton Bloc B

Simple code de test  :

---

publicclassForm1 : Form
{
   public Form1()
   {
      try  
     {
         Test();
      }
      catch (Exception e) { MessageBox.Show("Dans catch constructeur"); } 
   }
   
   publicvoid Test()
   {
      try 
      {
         MessageBox.Show("Dans try test");
         thrownewException();
         //return;
      }
      catch (Exception)
      {
         MessageBox.Show("Dans catch test");
         throw;
        //return;
      }
      finally { MessageBox.Show("Dans finally"); }
      MessageBox.Show("Apres finally");
   }
}

---

Tu verras que le "Apres finally" ne montre jamais le bout de son nez. Si tu modifies les throw par des return, idem.
D'ailleurs, pour s'en rendre compte plus facilement, Resharper me met la ligne du "apres finally" en gris.

Mx
MVP C# 

Bonjour,
j ai essayé de mettre le paramétrage mais le probleme c est que je créais mon cmd telquel:

OleDbCommand cmd = newOleDbCommand(insertCmd, connection);
et donc faire:
string insertCmd = "INSERT INTO user_mpl VALUES (1, ?, ?);";
cmd.parameters.add(textBox1.Text);
cmd.parameters.add(textBox2.Text);
cmd.ExecuteNonQuery();

n est pas possible compte tenu que je passe en parametre 'InsertCmd', donc je peux pas l initialiser avant d ajouter les parametres!!!
enfin je comprends pas vraiment l utilité des paramètres par rapport à ma concaténation, sachant que cést une requete pas compliquée!

donc j en suis toujours au meme point, je sais pas mon erreur du ExecuteNonQuery!!!

merci encore pour vos réponses!

Sabine

Eh bah initialise le apres

OleDbCommand cmd;
string insertCmd = "INSERT INTO user_mpl VALUES (1, ?, ?);";

cmd = newOleDbCommand(insertCmd, connection);
cmd.parameters.add(textBox1.Text);
cmd.parameters.add(textBox2.Text);
cmd.ExecuteNonQuery();

Eric