C# .NET insertion dans BD OraleRésolu

Question

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();
}

MorpionMx · Accepted Answer

Oui c'est rageant... En fait j'aurais du installer oracle dès le départ, ca t'aurais évité de perdre du temps :/  

Enfin bref, pour revenir un petit peu en arriere où tu demandais "faut-il des cotes ou autre" :  le fait d'utiliser les requetes paramétrées te permet de ne plus avoir a t'en soucier ;)

N'hésite pas a revenir poser des questions si besoin est.

a+

Mx
MVP C#

MorpionMx · Answer

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#

cs_Sabine25 · Answer

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:

private
void button1_Click(
object sender, 
EventArgs e)
{

string

 insertCmd = 
"INSERT INTO user_mpl VALUES (1,"+textBox1.Text+
","+textBox2.Text+
");";

OleDbConnection connection = 
new
OleDbConnection();

DataConnection m = 
new
DataConnection();
connection=m.DoConnection();

OleDbCommand cmd = 
new
OleDbCommand(insertCmd, connection);

int n=cmd.ExecuteNonQuery();

if (n != 1) { 
MessageBox.Show(
"insertion KO"); } 
else
MessageBox.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

class
DataConnection
{

                private
OleDbConnection conn;

                private
string connectString =
"Provider=OraOLEDB.Oracle;"
                                + 
"Data Source=*************;"
                                + 
"User ID=***********;"
                                + 
"Password=***********;";

                public DataConnection()
                {
                    conn = 
new
OleDbConnection(connectString);

                   // Le timeOut en secondes
                   //conn.ConnectionTimeout=60;
                 }

                //public void DoConnection()
                public
OleDbConnection DoConnection()
                 {

                 try
                {

                          // Connection
                          conn.Open();

                           Console.WriteLine(
"Connexion établit!!!");

                           return conn;
                 }

                 catch (
OleDbException e)
                 {

                            Console.WriteLine(
"erreur remarquee:" + e);

                            return conn;
                   }
 }

public
void DoDeconnection()
{

try
{
conn.Close();

Console.WriteLine(
"Déconnexion à la base de données");
}

catch (
Exception ex)
{

Console.WriteLine(
"erreur de deconnexion:" + ex);
}
}
}

MorpionMx · Answer

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#

MorpionMx · Answer

J'oubliais, qques infos sur les requetes paramétrées avec oledb

http://msdn2.microsoft.com/en-us/library/system.data.oledb.oledbcommand.parameters.aspx

Mx
MVP C#

cs_Sabine25 · Answer

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

ricklekebekoi · Answer

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

MorpionMx · Answer

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 :

public class Form1 : Form { public Form1() { try { Test(); } catch ( Exception e) { MessageBox.Show( "Dans catch constructeur"); } } public void Test() { try { MessageBox.Show( "Dans try test"); throw new Exception(); //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#

cs_Sabine25 · Answer

Bonjour,
j ai essayé de mettre le paramétrage mais le probleme c est que je créais mon cmd telquel:

OleDbCommand cmd = 
new
OleDbCommand(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

ricklekebekoi · Answer

Eh bah initialise le apres 

OleDbCommand cmd; 

string insertCmd = "INSERT INTO user_mpl VALUES (1, ?, ?);";

cmd = new

OleDbCommand(insertCmd, connection);

cmd.parameters.add(textBox1.Text);
cmd.parameters.add(textBox2.Text);
cmd.ExecuteNonQuery();

Eric

cs_Sabine25 · Answer

Arrrfff, j ai tenté un autre paramétrage mais apparement ca marche pas....

OleDbConnection

 conn = 
null;

private
void button1_Click(
object sender, 
EventArgs e){

try{

conn = 

new
OleDbConnection(
"Provider=OraOLEDB.Oracle;"+ 

"Data Source=********;"+ 

"User ID=***********;"+ 

"Password=***********;");conn.Open();

OleDbCommand cmd = 
new
OleDbCommand(
"INSERT INTO user_mpl VALUES (1, @user, @password);", conn);

OleDbParameter param1 = 
new
OleDbParameter();

OleDbParameter param2 = 
newOleDbParameter();param1.ParameterName "@user";param2.ParameterName 

"@password";param1.Value = textBox1.Text;

param2.Value = textBox2.Text;

cmd.Parameters.Add(param1);

cmd.Parameters.Add(param2);

cmd.ExecuteNonQuery();

}

finally{

conn.Close();

}

}

'Maintenant l erreur est : "missing expression" pour executeNonQuery()
alors là, franchement je sais pas!!!

Encore Merci
Sabine

MorpionMx · Answer

Une petite question, pourquoi utilises tu oledb et pas OracleClient

Mx
MVP C#

cs_Sabine25 · Answer

Euhh, bonne question! lol
non je l ai pas quand je tape using "System.Data...."
donc je sais pas trop
Mais je crois aue j en suis à ma 150 facons d insérer et que je réinvente le C#...rire!
merci pour ton aide
Sabine

MorpionMx · Answer

En fait il faut que tu l'ajoutes en référence a ton projet.
Clic droit sur "references" -> "Ads reference", puis tu choisis System.Data.OracleClient.

On va y arriver 

Mx
MVP C#

cs_Sabine25 · Answer

ca y est je l ai fais!
voici mon nouveau code



private



void
 button1_Click(

object
 sender, 

EventArgs
 e)



{
            OracleConnection conn = 
null;
            
conn = 
new
OracleConnection(
"Data Source=ICAODBDV_ICAODEV.ICAO.LAN;"
                         + 
"User ID=MPLDB;"
                        + 
"Password=MPLDB;"); 
            
            conn.Open();
            
OracleCommand cmd = conn.CreateCommand(); 
           cmd.CommandText = 
"INSERT INTO user_mpl (parm1,parm2,parm3) VALUES (?,?,?);" +
   
                                            "COMMIT;";
             
OracleParameter myParameter1 = 
new
OracleParameter(
"parm1", 
OracleType.Number);
             myParameter1.Value = textBox3.Text;
             
OracleParameter myParameter2 = 
new
OracleParameter(
"parm2", 
OracleType.VarChar);
             myParameter2.Value = textBox2.Text;
             
OracleParameter myParameter3 = 
new
OracleParameter(
"parm3", 
OracleType.VarChar);    
             myParameter3.Value = textBox1.Text;
             cmd.Parameters.Add(myParameter1);
             cmd.Parameters.Add(myParameter2);
             cmd.Parameters.Add(myParameter3);
 
             cmd.ExecuteNonQuery();
             cmd.Dispose();
             cmd = 
null;
             conn.Close();
}

et la j ai comme erreur "ORA-01036: illegal variable name/number", au niveau de ExecuteNonQuery,
alors que les parametres ont le bon type!!!
est cequ il faut les '?', faut t il des cotes ou autres???
je cherche mais je trouve pleins de version sur le net différentes!!!

merci merci et encore merci

Sabine

ricklekebekoi · Answer

Je n'y connais absolument rien en oracle, ni en requete parametrés, mais ce ne serait pas:

           cmd.CommandText = "INSERT INTO user_mpl (x,y,x) VALUES (parm1,parm2,parm3);" +
   
                                            "COMMIT;";

Où x,y et z sont les nom de tes colones de ta table oracle ?

Eric

MorpionMx · Answer

Essaie avec des parametres précédés de ':'
INSERT INTO user_mpl (parm1,parm2,parm3) VALUES (:p1,:p2,:p3);"  

puis

OracleParameter myParameter1 = 
new
OracleParameter(":p1", OracleType.Number);
myParameter1.Value = textBox3.Text;

Mx
MVP C#

cs_Sabine25 · Answer

Entre temps j avais trouvé ta solution, mais ca marche toujours pas

toujours "Invalid character"

j ai créé ma table user_mpl comme ceci:
SQL> desc user_mpl
 Name                                      Null?    Type
 ----------------------------------------- -------- -----------------------
 ID_USER                                   NOT NULL NUMBER(38)
 LOGIN                                              VARCHAR2(30)
 PASSWORD                                           VARCHAR2(30)

est ce que les types sont appropriés??? oui je pense!!!

emsuite dans le LOAD j ai rien mis, c est ok?
InitializeComponent se fait dans mon constructeur de Formulaire!!

Bref je sais plus!!!
Merci encore
------------------------------------------------------
OracleCommand
 cmd conn.CreateCommand();cmd.CommandText 

"INSERT INTO user_mpl (p1,p2,p3) VALUES (:parm1,:parm2,:parm3);" + 
"COMMIT;";

OracleParameter myParameter1 = 
new
OracleParameter(
":parm1", 
OracleType.Number);myParameter1.Value = textBox3.Text;

OracleParameter myParameter2 = 
new
OracleParameter(
":parm2", 
OracleType.VarChar,30);myParameter2.Value = textBox2.Text;

OracleParameter myParameter3 = 
new
OracleParameter(
":parm3", 
OracleType.VarChar,30);myParameter3.Value = textBox1.Text;

MorpionMx · Answer

Bon, j'installe Oracle et je vois ce que ca dit 

Mx
MVP C#

MorpionMx · Answer

Ok on y est Il s'agit simplement du ';' dans ta requete.... ce qui donne, par rapport a la structure de ta table

OracleCommand cmd = conn.CreateCommand(); cmd.CommandText = "INSERT INTO user_mpl (ID_USER, LOGIN, PASSWORD) VALUES (:parm1,:parm2,:parm3)"; OracleParameter myParameter1 = new OracleParameter( ":parm1", OracleType.Number); myParameter1.Value = textBox1.Text OracleParameter myParameter2 = new OracleParameter( ":parm2", OracleType.VarChar, 30); myParameter2.Value = textBox2.Text; OracleParameter myParameter3 = new OracleParameter( ":parm3", OracleType.VarChar, 30); myParameter3.Value = textBox3.Text cmd.Parameters.Add(myParameter1); cmd.Parameters.Add(myParameter2); cmd.Parameters.Add(myParameter3); cmd.ExecuteNonQuery();

Mx MVP C#

cs_Sabine25 · Answer

J ai tenté de faire un SELECT! 
et ca marche correctement!
donc voila je comprends toujours pas pour le INSERT!!!
Donc je continue a chercher de mon coté!!!

Sab







private






void
 button1_Click(

object
 sender, 

EventArgs
 e){
         


OracleConnection
 conn = 

null
; 


         conn = 

new



OracleConnection
(

"Data Source=ICAODBDV_ICAODEV.ICAO.LAN;" 

                                         + 

"User ID=MPLDB;"
                                         
+ 

"Password=MPLDB;"
); 
         conn.Open();
         

OracleCommand
 cmd = conn.CreateCommand();
         

string
 sql = 

"SELECT login, password FROM user_mpl"
;
         cmd.CommandText = sql;


         OracleDataReader
 reader = cmd.ExecuteReader();
         

while
 (reader.Read())
        {        


                     string
 user_login = (

string
)reader[

"login"
];
                     

string
 user_password = (

string
)reader[

"password"
];
                     

Console
.WriteLine(

"User Login: {0} User password: {1}"
,
                    user_login, user_password); 
         }
        

// clean up 

        reader.Close();
        reader = 

null
;
        cmd.CommandText = sql;
        cmd.ExecuteNonQuery();
        cmd.Dispose();
        cmd = 

null
;
         conn.Close();
}

MorpionMx · Answer

Ma réponse précédente ne fonctionne pas ?

Mx
MVP C#

cs_Sabine25 · Answer

Ouiiii ca marche, tout marche!!!
trop cool, merciii!
juste pour un petit " ; ", alors ça c'est rageant...lol
en tout cas merci!

Je pense que tu entendra reparler de moi ;-)
Sabine

cs_Sabine25 · Answer

Ok, super, merci pour tout!!!


en fait j ai un projet à faire en 3 mois et c'est mission impossible, en plus on me l a demandé en C# que je connaissais pas, donc je vais avoir besoin de conseil!!! ;-)


@+
Sab

cs_Bidou · Answer

Et n'hésites pas à valider les postes qui t'ont aidés, surtout que ce brave Mx à même installé Oracle pour toi !
Quel bon type  lol

<hr />
-Blog-

cs_Sabine25 · Answer

Bonjour bonjour,
un nouveau probleme me prend un peu la tete!
je veux fermer un form et ouvrir un autre:
voici mon code

this.Close();

// open the main menu

Menu myMenu = 
new
Menu();myMenu.ShowDialog();

myMenu.Activate();

Sauf que ca ferme mais ouvre rien! et si je mets le close apres le "activate", ca laisse tout ouvert!!!
donc je sais pas

merciii

Sabine

MorpionMx · Answer

Salut,

Tu devrais plutot ouvrir un nouveau topic, pour un sujet différent ;)

Le this.Close(), c'est la fermeture de ta Form principale ?
Si c'est le cas, c'est normal que tout se ferme. Si tu ne veux vraiment plus la voir a l'écran, il faut plutot la cacher (méthode Hide()). Et dans ce cas, autant utiliser la méthode Show() que ShowDialog(); sur ton instance de 'myMenu'

Mx
MVP C#

cs_coq · Answer

Salut,

Parfait, ça répond à la question que je me posais depuis le début de la lecture du sujet : est ce que le provider Oracle intégré se comporte de la même manière avec les ';' en .NET 2.0 que .NET 1.1, visiblement la réponse est oui :/
Si possible, passez par le provider livré par Oracle.

/*
coq
MVP Visual C#
CoqBlog
*/

Matroks · Answer

tiens sabine on dirait qu"on a le meme projet :)
donc voila, moi j'ai un autre prob concernant justement ce projet...
quand j'essaye de creer un user dans ma base oracle, ca plante avec cette fameuse erreur :
ORA-01036 numéro/nom de variable interdit

mon code est le suivant :

public void AddUser(string name, string pass)
        {
            OracleCommand cmd = _oraConnection.CreateCommand();
            cmd.CommandText =
                "CREATE USER :name IDENTIFIED BY :pass" ;
            OracleParameter p1 = new OracleParameter(":name", OracleDbType.Varchar2, 30);
            OracleParameter p2 = new OracleParameter(":pass", OracleDbType.Varchar2, 30);
            p1.Value = name;
            p2.Value = pass;
            System.Windows.Forms.MessageBox.Show(cmd.CommandText);
            cmd.Parameters.Add(p1);
            cmd.Parameters.Add(p2);

            cmd.ExecuteNonQuery();
        }

si qqun pourrait m'expliquer...

cs_Sabine25 · Answer

Je n ai pas rencontré cette erreur, mais je pense qu il faudrait que tu fasses apparaitre des "" quelque part comme p1.Value= ""+name+"";
ca veut dire que tu concatene avec des quotes compte tenu que tu as du String comme type de parametres!
pour voi si l erreur vient de là, change ton type par du "int", juste pour vérifier, parceque pour du Int t as pas besoin de quotes!!!
public void AddUser(int name, int pass)

si c est pas ca, je vois pas autre chose!!
Sabine

Matroks · Answer

c bon, j'ai la solution :-p enfin on m'a eclairé lol
en fait create user est une requete ddl et non sql, donc on ne peut pas utiliser les requetes parametrées avec des ddl...du coup il va falloir que je revienne a la methode concaténation de chaine (de préférence avec des tests de sécurité ;-)
Merci quand meme :)
Dim

C# .NET insertion dans BD Orale

31 réponses

Votre réponse

Discussions similaires