Discussion :

[labib23dz]

bonjour,

soit la classe suivante :

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
class connectionsql
     {
       SqlConnection con;
       public SqlDataReader rdr ;
       SqlCommand  cmd;
       public string req;
       string strcon = ConfigurationManager.ConnectionStrings["connection"].ConnectionString; 
 
 
       public connectionsql()
        {
           con=new SqlConnection(strcon);
           con.Open();
           cmd = new SqlCommand();
        }
 
       public void CreateReq()
       {
           cmd.CommandText = req;
           cmd.Connection = con;
       }
 
       public void AddParametre(string parametre, SqlDbType sqlDbType, object value)
       {
           cmd.Parameters.Add(parametre, sqlDbType);
           cmd.Parameters[parametre].Value = value;
       }
 
       public void ExecuteNonQuery()
       {
           cmd.ExecuteNonQuery();
       }
 
       public void ExecuteReader()
        {
          rdr=cmd.ExecuteReader();
        }
 
       public SqlCommand returnCmd()
       {
           return cmd;
       }
 
       public void Deconnection()
       {
           cmd.Dispose();
           rdr.Dispose();
           con.Close();
           con.Dispose();
       }
    //strcon contient la chaîne de connection
     }

et bout de code suivant :

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
 
private void button1_Click(object sender, EventArgs e)
{try
{
connectionsql insert = new connectionsql();
                        req = "Insert into tab_montant_bp (Code_A,Code_SC,montant,Exercice_BP,etat) values (@Code_A,@Code_SC,@montant,@Exercice_BP,@etat)";
                        insert.req = req;
                        insert.CreateReq();
                        insert.AddParametre("@Code_A", SqlDbType.Int, Convert.ToInt32(comboArticleBS.SelectedItem.ToString()));
                        insert.AddParametre("@Code_SC", SqlDbType.Int, Convert.ToInt32(comboSCBS.SelectedItem.ToString()));
                        insert.AddParametre("@montant", SqlDbType.Decimal, 0);
                        insert.AddParametre("@Exercice_BP", SqlDbType.Int, Convert.ToInt32(txtCurrentExercice.Text));
                        insert.AddParametre("@etat", SqlDbType.Bit, true);
                        insert.ExecuteNonQuery();
                        insert.Deconnection();
 
}
catch(Exception err)
{
MessageBox.show(err.Message);
]
}

en executant ce bout de code en cliquant sur le bouton, ça me génère un exception du gene :

Object reference not set to an instance of an object

cette exception est généré au niveau de la ligne

Code :

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

 rdr.Dispose();

de la classe : connectionsql.

quelqu'un saurait il d'ou peux venir le problème, et comment le résoudre ??

Merci.

[calagan99]

Bonjour,

rien de plus normal...

Avec le code que tu montres, ton SqlDataReader vaut null.
Or, tu appelles sa méthode Close() dans ta méthode Deconnection().

Tu ne peux pas faire appel à une méthode d'un objet "qui n'existe pas". Il faut que celui-ci ait été instancié (crée) pour pouvoir utiliser les propriétés/méthodes/fonctions d'un objet.

En l'occurrence, tu n'utilises pas de SqlDataReader dans ton traitement, ta méthode Deconnection ne doit donc pas essayer de le fermer.

[labib23dz]

bonsoir calagan99 , et merci pour ta réponse rapise et efficace.

en effet, tu as raison, je viens de supprimer la ligne

Code :

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

rdr.Dispose();

de la classe; maintenant je fait directement un un dispose sur l'objet "rdr" comme suit :

Code :

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

insert.rdr.Dispose()

et ça marche nikel.

Merci encore.

[kheironn]

L'idéal étant de faire les choses au bon endroit. inutile de faire du code qui va agir "à 3km de l'endroit où tu en as besoin".
D'une part ta classe connectionsql n'est pas vraiment utile vu ce qu'elle propose, de plus appeler ce genre de code dans le code behind n'est pas une bonne idée.
Je t'invite à te pencher sur l'architecture n-tiers (qui consiste à créer des couches ayant des responsabilités séparées). Au passage, inutile de créer plusieurs dll séparées s'il n'y a pas de réutilisation du code dans d'autres projets. Souvent le code est fortement imbriqué avec la fonctionnalité du projet. Si tu développes un framework, avec un excellent niveau d'abstraction, et que ton code peux être utilisé dans tous les projets, avec tous les sgdb, alors là dll à part !!!
En complément, je t'invite à te renseigner plus particulièrement sur la conception d'une Data Access Layer (je pense que tu trouveras des exemples sur le net) pour voir se qui se fait usuellement.

Après relecture, je nuance sur l'utilité de ta classe. C'est une sorte de "wrapper" (le mot n'est pas très bien choisi). Je t'invite à ne pas faire Open() dans le constructeur, mais d'y inclure au moins l'utilisation du constructeur de sqlcommand utilisant la connexion. Mieux, tu passe ta requête en paramètre du constructeur de ta classe. Utilise une propriété pour ta commande, comme ça tu utilises directement les méthodes du FW .Net...

Au final, ton exception est due à une erreur de conception fréquente chez les débutants.

bon courage pour la refactorisation.