exeption c# NullReferenceException

Version imprimable

Bonjour, je suis actuellement sur un projet d'étude d'une station de pompage d'eau pour les agriculteur, pour cela je simule une arriver d'eau avec débit pression etc.. Je lance le débit grace a une scrool_bar mais lors de l'assignation de celle ci a une jauge (outils iocomp) une exception ce créée du type NullReferenceException
voici le code :

Code:

public List<Compteur> listeCompteur; // Liste créee dans la classe ControleCompteur
Code:

1 2 3 4 lesCompteurs.Add(new Compteur(lesCanalisations[0],"1234")); lesCompteurs.Add(new Compteur(lesCanalisations[1],"7654")); lesCompteurs.Add(new Compteur(lesCanalisations[2],"6789")); // ajout de compteur dans la liste lesCOmpteur dans la classe Simulateur
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 public string numCompteur; public string NumCompteur { get { return numCompteur; } set { numCompteur = value; } } public Compteur(Canalisation canal, string numCompteur) { this.canal = canal; this.numCompteur = numCompteur; debitPrec = 0; monTimer = new Timer(); monTimer.Tick += new EventHandler(timer_Tick); monTimer.Interval = 1000; monTimer.Start(); } // classe compteur
On retourne dans la classe simulateur ou l’exception ce passe :
Code:

1 2 3 4 5 6 7 8 9 private void trackBar1_Scroll(object sender, EventArgs e) { if (this.monCompteur.NumCompteur == "1234") // ici { maCentrale.getGauge2Compteur().Value = trackBar1.Value; } }
voila j’espère que je ne vous est pas trop embrouillé ^^ merci d'avance de votre aide je retourne me creusé les méninges :)

27/01/2013, 22h13
infosam76
Bonjour
Code:

1 2 if (this.monCompteur.NumCompteur == "1234") // ici
si votre erreur se passe sur cette ligne , ou la variable monCompteur est null, ou le NumCompteur n'est pas initialisée
28/01/2013, 00h34
morgand

Peux tu envoyer la stack d'erreur souvent l'explication est dedans :)
28/01/2013, 09h28
Amiral62
Citation:
Envoyé par infosam76

Bonjour

Code:

1 2 if (this.monCompteur.NumCompteur == "1234") // ici

si votre erreur se passe sur cette ligne , ou la variable monCompteur est null, ou le NumCompteur n'est pas initialisée
Bonjour, Merci de votre réponse, monCompteur est bien null au debeugage pourtant l'agrégation est bien présente si non je n'aurais pas accès à la variable numCompteur ?
28/01/2013, 09h40
infosam76

Au vu de l'exception et de votre code c'était un ou l'autre mais je penchais plus effectivement pour monCompteur. Je suppose que votre NumCompteur est un type de base numérique donc lui ne devrais pas poser de problème.
28/01/2013, 10h39
Amiral62

Citation:

Envoyé par morgand

Peux tu envoyer la stack d'erreur souvent l'explication est dedans :)

Bonjour, merci de votre réponse le stack me répond ça : " La référence d'objet n'est pas définie à une instance d'un objet."

ma référence ( monCompteur) a la classe ControlCompteur est null pourtant j'ai accès a ces variables 8O
28/01/2013, 10h41
Amiral62

Citation:

Envoyé par infosam76

Au vu de l'exception et de votre code c'était un ou l'autre mais je penchais plus effectivement pour monCompteur. Je suppose que votre NumCompteur est un type de base numérique donc lui ne devrais pas poser de problème.

Oui je pense aussi numCopteur est un string retournée en get set et passer en paramètre d'une fonction Compteur pour ensuite crée 3 compteur avec des nom aléatoire que je chargerais dans un fichier xml mais le problème ici c'est que je ne comprend pas pourquoi ma référence a ma classe est null avez vous une idée ?
28/01/2013, 12h36
infosam76

Il serait intéressant de poster les 2 classes complètes car avec seulement les quelques morceaux de votre premier post, il est difficile de dire ce qui se passe entre l'instanciation et la fonction ou vous avez l'exception.

Citation:

Envoyé par infosam76

Il serait intéressant de poster les 2 classes complètes car avec seulement les quelques morceaux de votre premier post, il est difficile de dire ce qui se passe entre l'instanciation et la fonction ou vous avez l'exception.

voilà ;)

Classe simulateur

Code:

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
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
 
//Classe simulateur 
 
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 Iocomp.Instrumentation.Standard;
using System.Threading;
 
 
namespace Compteurs
{
    public partial class FormSimulateur : Form 
    {
        Canalisation maCanalisation;
        Compteur monCompteur;
        ControleCompteurs monControle;
        CentraleAcquisition maCentrale;
 
 
 
        internal Compteur MonCompteur
        {
            get { return monCompteur; }
            set { monCompteur = value; }
        }
 
 
        List<Compteur> lesCompteurs = new List<Compteur>();  //Création d'une Liste de Compteurs
 
        public List<Compteur> LesCompteurs
        {
            get { return lesCompteurs; }
            set { lesCompteurs = value; }
        }
 
        List<Canalisation> lesCanalisations = new List<Canalisation>(); //Création d'une Liste de Canalisations
 
 
 
 
        public FormSimulateur()
        {
            InitializeComponent();
 
            lesCanalisations.Add(new Canalisation(this.pipe1));
            lesCanalisations.Add(new Canalisation(this.pipe2));
            lesCanalisations.Add(new Canalisation(this.pipe3));
 
            lesCompteurs.Add(new Compteur(lesCanalisations[0],"1234"));
            lesCompteurs.Add(new Compteur(lesCanalisations[1],"7654"));
            lesCompteurs.Add(new Compteur(lesCanalisations[2],"6789"));
 
 
            maCentrale = new CentraleAcquisition(this);
            monControle = new ControleCompteurs(maCentrale, this.LesCompteurs);
            maCentrale.Hide();
            this.btn_AccesCentrale.Text = "Afficher la centrale";
 
 
            maCanalisation = new Canalisation(this.pipe1);
 
 
        }
 
     /*   public void Attente()
        {
          
          form1.getGauge2Compteur().Invoke(new TrackBarHandler(Affichage));
        }
 
        public void Affichage()
        {
            if (form1.getLabel8().Text == "1")
            {
                form1.getGauge2Compteur().Value = trackBar1.Value;
            }
            if (form1.getLabel8().Text == "2")
            {
                form1.getGauge2Compteur().Value = trackBar2.Value;
            }
            if (form1.getLabel8().Text == "3")
            {
                form1.getGauge2Compteur().Value = trackBar3.Value;
            }
        
        }
      */
 
        private void trackBar1_ValueChanged(object sender, EventArgs e)
        {
 
           lesCanalisations[0].Debit = trackBar1.Value;
 
        }
 
       public void btn_AccesCentrale_Click(object sender, EventArgs e)
        {
            if (this.maCentrale.Visible == false)
            {
                maCentrale.label8.Text = monControle.listeCompteur[monControle.indexCompteur_InfoCompteur].NumCompteur.ToString();
                this.maCentrale.Show();
                this.btn_AccesCentrale.Text = "Masquer la centrale";
 
            }
            else {
 
                this.maCentrale.Hide();
                this.btn_AccesCentrale.Text = "Afficher la centrale";
 
            }
        }
 
       private void trackBar1_Scroll(object sender, EventArgs e)
       {
 
           if (this.monCompteur.NumCompteur == "1234")
           {
               maCentrale.getGauge2Compteur().Value = trackBar1.Value;
           }
 
       }
 
       private void trackBar2_Scroll(object sender, EventArgs e)
       {
           if (monCompteur.NumCompteur == "7654")
           {
               maCentrale.getGauge2Compteur().Value = trackBar2.Value;
           }
       }
 
       private void trackBar3_Scroll(object sender, EventArgs e)
       {
           if (monCompteur.NumCompteur == "6789")
           {
               maCentrale.getGauge2Compteur().Value = trackBar3.Value;
           }
 
       }
 
       private void FormSimulateur_Load(object sender, EventArgs e)
       {
 
       }
 
 
 
 
 
    }
}
 
Classe Compteur

Code:

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
59
60
61
62
63
64
65
66
67
 
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using Iocomp.Instrumentation.Standard;
using System.Xml.Linq;
using System.IO;
using System.Web;
using System.Windows.Forms;
 
 
namespace Compteurs
{
    public class Compteur 
    {
        private Canalisation canal;
        private double debitActuel;
        private double debitPrec;
        private double consommationTotalisee = 0;
 
        public double ConsommationTotalisee1
        {
            get { return consommationTotalisee; }
 
        }
        public string numCompteur;
 
        public string NumCompteur
        {
            get { return numCompteur; }
            set { numCompteur = value; }
        }
 
 
 
        public double ConsommationTotalisee
        {
            get { return consommationTotalisee; }
            set { consommationTotalisee = value; }
        }
 
        private Timer monTimer;
 
 
        public Compteur(Canalisation canal, string numCompteur) {
 
            this.canal = canal;
            this.numCompteur = numCompteur;
            debitPrec = 0;
            monTimer = new Timer();
            monTimer.Tick += new EventHandler(timer_Tick);
            monTimer.Interval = 1000;
 
            monTimer.Start();
 
        }
 
         private void timer_Tick(object sender, EventArgs e)
        {
            this.debitActuel = this.canal.Debit;
 
            this.consommationTotalisee += (((this.debitPrec + this.debitActuel) / 2) * 1)*3600;
            this.debitPrec = this.debitActuel;
        }
    }
}

28/01/2013, 13h58
micka132

dans ton simulateur il n'y a nul part où tu instancies MonCompteur. Normal donc que ca ne fonctionne pas!
Je ne sais pas bien quel est ton besoin, mais tu peux en creer un dans le constructeur de ta form.
28/01/2013, 13h59
Amiral62

Je pense que cela viens des liste la référence qu'utilise ma condition fait appel a rien juste comme il n'est pas instancié donc il faut que je récupère ces info depuis la list j'avait pensé a un list.get mais ça n’existe pas dans ce conteste :/
28/01/2013, 14h02
micka132

Citation:

Envoyé par Amiral62

Je pense que cela viens des liste la référence qu'utilise ma condition fait appel a rien juste comme il n'est pas instancié donc il faut que je récupère ces info depuis la list j'avait pensé a un list.get mais ça n’existe pas dans ce conteste :/

Rien compris:roll:
MonCompteur est censé faire partie de la liste lesCompteurs ?
30/01/2013, 12h08
alucia
micka132 a raison, pour corriger l'erreur il faut que tu fasses un
Code:

1 2 monCompteur = new Compteur(...);
avant d'essayer d'accéder à monCompteur.NumCompteur
Tu as bien créé des nouveaux compteurs dans ta liste lesCompteurs; mais monCompteur n'est jamais instancié

Si tu souhaites récupérer un des compteurs de ta liste, il existe des méthodes :

First() => renvoie le 1er élément
Last() => envoie le dernier élément

Tu peux aussi faire une recherche précise de cette manière :
(exemple si tu veux récupérer celui qui a NumCompteur = 1234)
Code:

1 2 monCompteur = lesCompteurs.Find(p => p.NumCompteur = "1234");
31/01/2013, 19h07
Amiral62
Bonjour, merci de vos réponse, j'ai trouvé le probléme il vennais du fait que les association crée :
Code:

1 2 maCentrale = new CentraleAcquisition(this); monControle = new ControleCompteurs(maCentrale, this.LesCompteurs);
été mal placé on faisais appel a une référence d'objet qui n'été pas encore crée.

la création d'un compteur n'été pas obligatoire comme ils sont crée dans les liste :

Code:

lesCompteurs.Add(new Compteur(lesCanalisations[0],"1234"));

voila je vous re merci pour votre attentions à bientôt :ccool: