Problème exception C#

Version imprimable

Bonjour,
Je suis actuellement en réalisation de projet dans le cadre de mon BTS ( IRIS )
Explication du projet : Gestion de capteurs d'une chaîne de production à l'aide d'une carte µChameleon.

voici mon problème ( ne pas faire attention au message de l'antivirus :lol: ):

http://img26.imageshack.us/img26/5660/22805205.png

Code source :

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
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
 
//********CSerie.cs*******//
 
using System;
using System.Collections.Generic;
using System.Text;
using System.IO.Ports;
using System.Windows.Forms;
 
 
namespace TestChameleon
{
    class CSerie
    {
        SerialPort Com;
        int Vitesse;
        public CSerie(SerialPort Port, int Vitesse)
        {
 
            Com = Port;
            this.Vitesse = Vitesse;
            Com.BaudRate = Vitesse;
            Com.StopBits = StopBits.One;
            Com.DataBits = 8;
            Com.Parity = Parity.None;
            Com.Handshake = Handshake.None;
            Com.ReadTimeout = 1000;
 
 
        }
 
        public CSerie()
        {
 
            SerialPort Com = new SerialPort();
            // TODO: Complete member initialization
        }
        //-------------Ouverture du port série-------------//
        public bool Ouvrir()
        {
            bool EtatConnexion; 
            if (Com.IsOpen == false)
            {
                Com.Open();
                EtatConnexion = true;
            }
            else
            {
                MessageBox.Show("Port utilisé", "Erreur", MessageBoxButtons.OK, MessageBoxIcon.Stop);
                EtatConnexion = false;
            }
           return EtatConnexion;
        }
 
        //-------------Envoi d'un octet-------------//
        public void Envoi_Octet(byte[] Octet)
        {
            Com.Write(Octet, 0, 1);
        }
 
        //-------------Envoi d'une trame-------------//
        public void Envoi_Trame(byte[] Octet,int Taille)
        {
            Com.Write(Octet, 0, Taille);
        }
 
        //-------------Envoi d'une chaine-------------//
        public void Envoi_Trame(string Chaine)
        {
            Com.WriteLine(Chaine);
        }
 
        //----------Recevoir d'une chaine-------------//
        public string Recevoir_Trame()
        {
            return Com.ReadLine();
        }
        //-------------Réception d'une trame-------------//
        public byte[] Recevoir_DonneeByte()
        {
            byte[] BY = null;
            int Count = Com.BytesToRead;
            if (Count > 0)
            {
                BY = new byte[Count];
                for (int i = 0; i < Count; i++)
                {
                    BY[i] = (byte)Com.ReadByte();
                }
            }
            return BY;
        }
 
        //-------------Vidage du Buffer-------------//
        public void Vider_Buffer()
        {
              Com.ReadExisting();          
        }
 
        public bool Fermer()
        {
            bool EtatConnexion;
            if (Com.IsOpen == true)
            {
                Com.Close();
                EtatConnexion = false;
            }
            else
            {
                //MessageBox.Show("Port utilisé", "Erreur", MessageBoxButtons.OK, MessageBoxIcon.Stop);
                EtatConnexion = true;
            }
            return EtatConnexion;
        }
 
    }
}
 
//*************CChameleon.cs*******************//
 
using System;
using System.Collections.Generic;
using System.Text;
 
namespace TestChameleon
{
    class CChameleon
    {
        CSerie ComUSB;
        Pin[] Bornier = new Pin[100];
 
        struct Pin{
            public int Config;
            public int Valeur;
        };
 
        public enum ConfigPin {  
           Entree = 0, 
           Sortie = 1           
        }  
 
        public CChameleon()
        {
            ComUSB = null;
        }
 
        public CChameleon(CSerie Ref)
        {
            ComUSB = Ref;
            this.InitialisationDefaut();
        }
 
        /*************************************************************/
        /********Permet de configurer une pin ou un ensemble *********/
        /************** de pins consécutives en entrée ***************/
        /*************************************************************/
        public void ConfigurerEntreeTOR(int NumeroPin, int NombrePin)
        {
            string Trame;
            int Indice = NumeroPin;
 
            while (Indice < (NumeroPin + NombrePin))
            {
                Trame = "pin " + Indice + " in";
                ComUSB.Envoi_Trame(Trame);
                Trame = "pin " + Indice + " pullup on";
                ComUSB.Envoi_Trame(Trame);
                this.Bornier[Indice -1].Config = (int)ConfigPin.Entree;
                Indice++;
            }
 
        }
        /*************************************************************/
        /********Permet de configurer une pin ou un ensemble *********/
        /************** de pins consécutives en sortie ***************/
        /*************************************************************/
 
        public void ConfigurerSortieTOR(int NumeroPin, int NombrePin)
        {
            string Trame;
            int Indice = NumeroPin;
 
            while (Indice < (NumeroPin + NombrePin))
            {
                Trame = "pin " + Indice + " out";
                ComUSB.Envoi_Trame(Trame);
                this.Bornier[Indice - 1].Config = (int)ConfigPin.Sortie;
                Trame = "pin " + Indice + " lo";
                ComUSB.Envoi_Trame(Trame);
                this.Bornier[Indice - 1].Valeur = 0;
                Indice++;
            }
        }
 
        /*************************************************************/
        /********Mettre au niveau haut ou bas une sotie **************/
        /*************************************************************/
 
        public void EcrireSortieTOR(int NumeroPin,int Valeur,int NombrePin)
        {
            string Trame;
            int Indice = NumeroPin;
 
 
            while (Indice < (NumeroPin + NombrePin)){
 
                if (this.Bornier[Indice - 1].Config == 1){
 
                    if (Valeur != this.Bornier[Indice - 1].Valeur){
 
                        if (Valeur == 1)
                        {
                            Trame = "pin " + Indice + " hi";
                            this.Bornier[Indice - 1].Valeur = 1;
                        }
                        else
                        {
                            Trame = "pin " + Indice + " lo";
                            this.Bornier[Indice - 1].Valeur = 0;
                        }
                        ComUSB.Envoi_Trame(Trame);
                    }
                    Indice++;
                }
            }
        }
        /*************************************************************/
        /******************* Ecrire sur une sortie *******************/
        /*************************************************************/
 
        public void EcrireSortieTOR(int NumeroPin,int Valeur) {
 
            EcrireSortieTOR(NumeroPin, Valeur, 1);
        }
 
        /*************************************************************/
        /******** Envoyer un octet sur 8 sorties consécutives ********/
        /*************************************************************/
 
        public void EcrireOctet(byte Octet, int NumPinD0 ){
            int Indice = 0;
            byte Masque = 0x01;
            byte ValeurEnvoyee = Octet;
 
            while (Indice < 8)
            {
                if ((this.Bornier[NumPinD0 + Indice - 1].Valeur) != (ValeurEnvoyee & Masque))
                {
                    if ((ValeurEnvoyee & Masque) == 1)
                    {
                        this.EcrireSortieTOR((NumPinD0 + Indice), 1);
                    }
                    else
                    {
                        this.EcrireSortieTOR((NumPinD0 + Indice), 0);
                    }
                }
                Indice++;
                ValeurEnvoyee = (byte)(ValeurEnvoyee >> 1);
            }
        }
 
        /*************************************************************/
        /******************** Allumer la led *************************/
        /*************************************************************/
 
        public void AllumerLED() {
 
            ComUSB.Envoi_Trame("led on");
        }
 
        /*************************************************************/
        /*********************** Eteindre la LED *********************/
        /*************************************************************/
 
        public void EteindreLED()
        {
            ComUSB.Envoi_Trame("led off");
        }
 
        /*************************************************************/
        /********************** Initialiser tout en entree *************/
        /*************************************************************/
 
        public void InitialisationDefaut()
        {
            ConfigurerEntreeTOR(1,18);
        }
 
        /*************************************************************/
        /***************** Allumer toute les led *********************/
        /*************************************************************/
 
        public void AllumerTouteLesLed()
        {
            EcrireSortieTOR(5, 1);
            EcrireSortieTOR(6, 1);
            EcrireSortieTOR(7, 1);
            EcrireSortieTOR(8, 1);
            EcrireSortieTOR(9, 1);
            EcrireSortieTOR(18, 1);
 
        }
 
        /*************************************************************/
        /***************** Eteindre toute les led ********************/
        /*************************************************************/
 
        public void EteindreTouteLesLed()
        {
            EcrireSortieTOR(5, 0);
            EcrireSortieTOR(6, 0);
            EcrireSortieTOR(7, 0);
            EcrireSortieTOR(8, 0);
            EcrireSortieTOR(9, 0);
            EcrireSortieTOR(18, 0);
 
 
 
        }
 
        /*************************************************************/
        /********************** allumer le buzzer ********************/
        /*************************************************************/
 
        public void AllumerBuzzer(int tempsEnSeconde)
        {
            EcrireSortieTOR(12, 1);
            tempsEnSeconde = tempsEnSeconde * 1000;
            System.Threading.Thread.Sleep(tempsEnSeconde);
            EcrireSortieTOR(12, 0);
 
        }
 
        /*************************************************************/
        /***************** Lire l'état d'une entre TOR ***************/
        /*************************************************************/
 
        public int LireEntreeTOR(int NumeroPin)
        {
            string Trame;
            string TrameReponse;
            int Etat = -1;
 
            CSerie ComUSB = new CSerie();
 
            if (this.Bornier[NumeroPin - 1].Config == 0) /* Si configurer en entree */
            {  
                Trame = "pin " + NumeroPin + " state";
                ComUSB.Envoi_Trame(Trame);
                TrameReponse = ComUSB.Recevoir_Trame();
                Etat = this.DecoderEtatEntreeTOR(TrameReponse);
                this.Bornier[NumeroPin - 1].Valeur = Etat;
            }
            return Etat;
         }
 
        private int  DecoderEtatEntreeTOR (string Reponse){
            string[] SousChaine ;
            SousChaine = Reponse.Split(' ');    // Cherche le caractére espace est divise la chaine dans le tableau
            return Convert.ToInt16(SousChaine[2].ToString());
 
        }
     }
}
 
 
//*************Form1.cs*****************//
 
 
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 System.IO.Ports;
using System.Threading;
using TestChameleon;
 
namespace TestChameleon
{
    public partial class Form1 : Form
    {
 
        private CSerie Serie;
        private bool EtatConnexion;
        private CChameleon CarteChameleon;
 
        public Form1()
        {
            InitializeComponent();
            CSerie Serie = new CSerie();
            CChameleon CarteChameleon = new CChameleon();
            Thread th = new Thread(Lire);
            th.Start();
        }
 
       public void Lire()
       {
            int i = 1;
            int retour;
            int GestionCapteurs = 1;
            CChameleon CarteChameleon = new CChameleon();
 
 
            retour = CarteChameleon.LireEntreeTOR(i);
 
            if (retour == 1)
            {
                switch (GestionCapteurs)
                {
                    case 1:
                        i++;
                        this.label9.Text = "OK";
                        break;
                    case 2:
                        i++;
                        this.label10.Text = "OK";
                        break;
                    case 3:
                        i++;
                        this.label11.Text = "OK";
                        break;
                    case 4:
                        i++;
                        this.label12.Text = "OK";
                        break;
                    case 5:
                        i++;
                        this.label13.Text = "OK";
                        break;
                    case 6:
                        i++;
                        this.label14.Text = "OK";
                        break;
                    case 7:
                        i++;
                        this.label15.Text = "OK";
                        break;
                    case 8:
                        i++;
                        this.label16.Text = "OK";
                        break;
 
                }
            }
        } 
 
        private void Form1_Load(object sender, EventArgs e)
        {
            EtatConnexion = false;
            Serie = null;
 
            string[] ports = SerialPort.GetPortNames();
 
            foreach (string port in ports)
            {
                this.comboBox1.Items.Add(port);
            }
            this.comboBox1.SelectedIndex = 0;
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
 
 
 
            if(this.EtatConnexion == false)
            {
                this.serialPort1.PortName = this.comboBox1.SelectedItem.ToString();
                if (Serie == null)
                {
                    Serie = new CSerie(serialPort1, 9600);
                    try
                    {
                        EtatConnexion = Serie.Ouvrir();
                        this.CarteChameleon = new CChameleon(Serie);
                        this.button1.Text = "Deconnecter";
                        toolStripStatusLabel1.Text = "Etat : Connecté";
 
 
 
                        /*************************************************************/
                        /*************Configuration des pin de la carte***************/
                        /*************************************************************/
                        /*************************************************************/
 
 
                        CarteChameleon.ConfigurerSortieTOR(18, 1); // led de reponse carte
                        CarteChameleon.ConfigurerEntreeTOR(3, 2); //inter alarm
                        CarteChameleon.ConfigurerSortieTOR(5, 5); // led alarm + led chauffage + led volet
                        CarteChameleon.ConfigurerEntreeTOR(10, 1); //inter capteur lumiere
                        CarteChameleon.ConfigurerSortieTOR(12, 1); // buzzer alarm
 
                        /*************************************************************/
                        /*************initialisation des pins de la carte*************/
                        /*************************************************************/
                        /*************************************************************/
 
                        CarteChameleon.EcrireSortieTOR(18, 0);
                        CarteChameleon.EcrireSortieTOR(5, 0);
                        CarteChameleon.EcrireSortieTOR(6, 0);
                        CarteChameleon.EcrireSortieTOR(7, 0);
                        CarteChameleon.EcrireSortieTOR(8, 0);
                        CarteChameleon.EcrireSortieTOR(9, 0);
                        CarteChameleon.EcrireSortieTOR(12, 0);
 
 
 
 
                    }
                    catch
                    {
                         toolStripStatusLabel1.Text = "Etat : Erreur de connection";
                         MessageBox.Show("Port utilisé", "Erreur", MessageBoxButtons.OK, MessageBoxIcon.Stop);
                         Serie = null;
                    }
 
                }
            }
            else
            {
                if(EtatConnexion == true)
                {
                    EtatConnexion = Serie.Fermer();
                    Serie = null;
                    this.button1.Text = "Connecter";
                    toolStripStatusLabel1.Text = "Etat : Déconnecté";
                }
 
            }
 
        }
 
        private void button2_Click(object sender, EventArgs e)
        {
 
            CarteChameleon.AllumerLED();
 
        }
 
        private void button3_Click(object sender, EventArgs e)
        {
 
            CarteChameleon.EteindreLED();
        }
 
    //**********************************GESTION CAPTEURS**********************************//
 
 
 
 
 
 
    }
}

18/01/2012, 11h49
Bluedeep

Bonjour

Visiblement Com est à null.
18/01/2012, 11h56
GRUMLY

Bonjour,

Tu ne semble pas instancier ta classe Com.
Cela vient du fait que tu n'utilises pas le constructeur CSerie(SerialPort Port, int Vitesse).
18/01/2012, 11h57
Amiral62

Bonjour Bluedeep,

Oui Com est à null mais je ne vois pas comment il puisse l'être :/
18/01/2012, 12h04
Amiral62

Quand je l'instancie il me fait une autre erreur toujours sur la même ligne mais avec le message

Citation:

Le port est fermé.
18/01/2012, 12h04
Bluedeep

Citation:

Envoyé par Amiral62

Bonjour Bluedeep,

Oui Com est à null mais je ne vois pas comment il puisse l'être :/

Met un breakpoint quelque part et exécute pas à pas (F11) pour voir si tu passes ou pas par le constructeur de Com.
18/01/2012, 12h05
synolog

Le constructeur CSerie(SerialPort Port, int Vitesse) est instancié lors de l'appui sur Button1.
Met un point d'arrêt pour voir si celui-ci s’exécute bien
18/01/2012, 12h09
Amiral62

Oui visiblement je ne passe pas dedans.
18/01/2012, 12h20
synolog

Enlève déjà le this à ta ligne 471.
Tu peut conclure que c'est une de tes 2 conditions qui bloquent (malgrès que leurs valeurs correspondent à celle que tu charges dans ta form)
18/01/2012, 15h31
Amiral62

Merci pour ta reponse bugg.
J'essaye ça en rentrant et je te dis quoi ;)
18/01/2012, 17h59
Amiral62

Re,
J'ai supprimé le this de la ligne 471 et essayé de bidouiller les deux conditions mais rien ne fait... avec un point d’arrêt il ne passe pas par là, après avoir lancer le Thred il lance le run ==>

Code:

Application.Run(new Form1());

et bloque sur le fameux Com. =(

Voilà le Message d'erreur et le code actualisé :

Message d'erreur :

http://img88.imageshack.us/img88/2924/pb2o.png

Code :

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
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
 
 
//********************CSerie.cs******************//
 
using System;
using System.Collections.Generic;
using System.Text;
using System.IO.Ports;
using System.Windows.Forms;
 
 
namespace TestChameleon
{
    class CSerie
    {
        SerialPort Com = new SerialPort() ;
        int Vitesse;
        public CSerie(SerialPort Port, int Vitesse)
        {
 
            Com = Port;
            this.Vitesse = Vitesse;
            Com.BaudRate = Vitesse;
            Com.StopBits = StopBits.One;
            Com.DataBits = 8;
            Com.Parity = Parity.None;
            Com.Handshake = Handshake.None;
            Com.ReadTimeout = 1000;
 
 
        }
 
        public CSerie()
        {
 
 
            // TODO: Complete member initialization
        }
        //-------------Ouverture du port série-------------//
        public bool Ouvrir()
        {
            bool EtatConnexion; 
            if (Com.IsOpen == false)
            {
                Com.Open();
                EtatConnexion = true;
            }
            else
            {
                MessageBox.Show("Port utilisé", "Erreur", MessageBoxButtons.OK, MessageBoxIcon.Stop);
                EtatConnexion = false;
            }
           return EtatConnexion;
        }
 
        //-------------Envoi d'un octet-------------//
        public void Envoi_Octet(byte[] Octet)
        {
            Com.Write(Octet, 0, 1);
        }
 
        //-------------Envoi d'une trame-------------//
        public void Envoi_Trame(byte[] Octet,int Taille)
        {
            Com.Write(Octet, 0, Taille);
        }
 
        //-------------Envoi d'une chaine-------------//
        public void Envoi_Trame(string Chaine)
        {
            Com.WriteLine(Chaine);
        }
 
        //----------Recevoir d'une chaine-------------//
        public string Recevoir_Trame()
        {
            return Com.ReadLine();
        }
        //-------------Réception d'une trame-------------//
        public byte[] Recevoir_DonneeByte()
        {
            byte[] BY = null;
            int Count = Com.BytesToRead;
            if (Count > 0)
            {
                BY = new byte[Count];
                for (int i = 0; i < Count; i++)
                {
                    BY[i] = (byte)Com.ReadByte();
                }
            }
            return BY;
        }
 
        //-------------Vidage du Buffer-------------//
        public void Vider_Buffer()
        {
              Com.ReadExisting();          
        }
 
        public bool Fermer()
        {
            bool EtatConnexion;
            if (Com.IsOpen == true)
            {
                Com.Close();
                EtatConnexion = false;
            }
            else
            {
                //MessageBox.Show("Port utilisé", "Erreur", MessageBoxButtons.OK, MessageBoxIcon.Stop);
                EtatConnexion = true;
            }
            return EtatConnexion;
        }
 
    }
}
 
//**************CChameleon.cs*******************//
 
using System;
using System.Collections.Generic;
using System.Text;
 
namespace TestChameleon
{
    class CChameleon
    {
        CSerie ComUSB;
        Pin[] Bornier = new Pin[100];
 
        struct Pin{
            public int Config;
            public int Valeur;
        };
 
        public enum ConfigPin {  
           Entree = 0, 
           Sortie = 1           
        }  
 
        public CChameleon()
        {
            ComUSB = null;
        }
 
        public CChameleon(CSerie Ref)
        {
            ComUSB = Ref;
            this.InitialisationDefaut();
        }
 
        /*************************************************************/
        /********Permet de configurer une pin ou un ensemble *********/
        /************** de pins consécutives en entrée ***************/
        /*************************************************************/
        public void ConfigurerEntreeTOR(int NumeroPin, int NombrePin)
        {
            string Trame;
            int Indice = NumeroPin;
 
            while (Indice < (NumeroPin + NombrePin))
            {
                Trame = "pin " + Indice + " in";
                ComUSB.Envoi_Trame(Trame);
                Trame = "pin " + Indice + " pullup on";
                ComUSB.Envoi_Trame(Trame);
                this.Bornier[Indice -1].Config = (int)ConfigPin.Entree;
                Indice++;
            }
 
        }
        /*************************************************************/
        /********Permet de configurer une pin ou un ensemble *********/
        /************** de pins consécutives en sortie ***************/
        /*************************************************************/
 
        public void ConfigurerSortieTOR(int NumeroPin, int NombrePin)
        {
            string Trame;
            int Indice = NumeroPin;
 
            while (Indice < (NumeroPin + NombrePin))
            {
                Trame = "pin " + Indice + " out";
                ComUSB.Envoi_Trame(Trame);
                this.Bornier[Indice - 1].Config = (int)ConfigPin.Sortie;
                Trame = "pin " + Indice + " lo";
                ComUSB.Envoi_Trame(Trame);
                this.Bornier[Indice - 1].Valeur = 0;
                Indice++;
            }
        }
 
        /*************************************************************/
        /********Mettre au niveau haut ou bas une sotie **************/
        /*************************************************************/
 
        public void EcrireSortieTOR(int NumeroPin,int Valeur,int NombrePin)
        {
            string Trame;
            int Indice = NumeroPin;
 
 
            while (Indice < (NumeroPin + NombrePin)){
 
                if (this.Bornier[Indice - 1].Config == 1){
 
                    if (Valeur != this.Bornier[Indice - 1].Valeur){
 
                        if (Valeur == 1)
                        {
                            Trame = "pin " + Indice + " hi";
                            this.Bornier[Indice - 1].Valeur = 1;
                        }
                        else
                        {
                            Trame = "pin " + Indice + " lo";
                            this.Bornier[Indice - 1].Valeur = 0;
                        }
                        ComUSB.Envoi_Trame(Trame);
                    }
                    Indice++;
                }
            }
        }
        /*************************************************************/
        /******************* Ecrire sur une sortie *******************/
        /*************************************************************/
 
        public void EcrireSortieTOR(int NumeroPin,int Valeur) {
 
            EcrireSortieTOR(NumeroPin, Valeur, 1);
        }
 
        /*************************************************************/
        /******** Envoyer un octet sur 8 sorties consécutives ********/
        /*************************************************************/
 
        public void EcrireOctet(byte Octet, int NumPinD0 ){
            int Indice = 0;
            byte Masque = 0x01;
            byte ValeurEnvoyee = Octet;
 
            while (Indice < 8)
            {
                if ((this.Bornier[NumPinD0 + Indice - 1].Valeur) != (ValeurEnvoyee & Masque))
                {
                    if ((ValeurEnvoyee & Masque) == 1)
                    {
                        this.EcrireSortieTOR((NumPinD0 + Indice), 1);
                    }
                    else
                    {
                        this.EcrireSortieTOR((NumPinD0 + Indice), 0);
                    }
                }
                Indice++;
                ValeurEnvoyee = (byte)(ValeurEnvoyee >> 1);
            }
        }
 
        /*************************************************************/
        /******************** Allumer la led *************************/
        /*************************************************************/
 
        public void AllumerLED() {
 
            ComUSB.Envoi_Trame("led on");
        }
 
        /*************************************************************/
        /*********************** Eteindre la LED *********************/
        /*************************************************************/
 
        public void EteindreLED()
        {
            ComUSB.Envoi_Trame("led off");
        }
 
        /*************************************************************/
        /********************** Initialiser tout en entree *************/
        /*************************************************************/
 
        public void InitialisationDefaut()
        {
            ConfigurerEntreeTOR(1,18);
        }
 
        /*************************************************************/
        /***************** Allumer toute les led *********************/
        /*************************************************************/
 
        public void AllumerTouteLesLed()
        {
            EcrireSortieTOR(5, 1);
            EcrireSortieTOR(6, 1);
            EcrireSortieTOR(7, 1);
            EcrireSortieTOR(8, 1);
            EcrireSortieTOR(9, 1);
            EcrireSortieTOR(18, 1);
 
        }
 
        /*************************************************************/
        /***************** Eteindre toute les led ********************/
        /*************************************************************/
 
        public void EteindreTouteLesLed()
        {
            EcrireSortieTOR(5, 0);
            EcrireSortieTOR(6, 0);
            EcrireSortieTOR(7, 0);
            EcrireSortieTOR(8, 0);
            EcrireSortieTOR(9, 0);
            EcrireSortieTOR(18, 0);
 
 
 
        }
 
        /*************************************************************/
        /********************** allumer le buzzer ********************/
        /*************************************************************/
 
        public void AllumerBuzzer(int tempsEnSeconde)
        {
            EcrireSortieTOR(12, 1);
            tempsEnSeconde = tempsEnSeconde * 1000;
            System.Threading.Thread.Sleep(tempsEnSeconde);
            EcrireSortieTOR(12, 0);
 
        }
 
        /*************************************************************/
        /***************** Lire l'état d'une entre TOR ***************/
        /*************************************************************/
 
        public int LireEntreeTOR(int NumeroPin)
        {
            string Trame;
            string TrameReponse;
            int Etat = -1;
 
            CSerie ComUSB = new CSerie();
 
            if (this.Bornier[NumeroPin - 1].Config == 0) /* Si configurer en entree */
            {  
                Trame = "pin " + NumeroPin + " state";
                ComUSB.Envoi_Trame(Trame);
                TrameReponse = ComUSB.Recevoir_Trame();
                Etat = this.DecoderEtatEntreeTOR(TrameReponse);
                this.Bornier[NumeroPin - 1].Valeur = Etat;
            }
            return Etat;
         }
 
        private int  DecoderEtatEntreeTOR (string Reponse){
            string[] SousChaine ;
            SousChaine = Reponse.Split(' ');    // Cherche le caractére espace est divise la chaine dans le tableau
            return Convert.ToInt16(SousChaine[2].ToString());
 
        }
     }
}
 
 
//*******************Form1.cs************************//
 
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 System.IO.Ports;
using System.Threading;
using TestChameleon;
 
namespace TestChameleon
{
    public partial class Form1 : Form
    {
 
        private CSerie Serie;
        private bool EtatConnexion;
        private CChameleon CarteChameleon;
 
        public Form1()
        {
            InitializeComponent();
            CSerie Serie = new CSerie();
            CChameleon CarteChameleon = new CChameleon();
            Thread th = new Thread(Lire);
            th.Start();
        }
 
       public void Lire()
       {
            int i = 1;
            int retour;
            int GestionCapteurs = 1;
            CChameleon CarteChameleon = new CChameleon();
 
 
            retour = CarteChameleon.LireEntreeTOR(i);
 
            if (retour == 1)
            {
                switch (GestionCapteurs)
                {
                    case 1:
                        i++;
                        this.label9.Text = "OK";
                        break;
                    case 2:
                        i++;
                        this.label10.Text = "OK";
                        break;
                    case 3:
                        i++;
                        this.label11.Text = "OK";
                        break;
                    case 4:
                        i++;
                        this.label12.Text = "OK";
                        break;
                    case 5:
                        i++;
                        this.label13.Text = "OK";
                        break;
                    case 6:
                        i++;
                        this.label14.Text = "OK";
                        break;
                    case 7:
                        i++;
                        this.label15.Text = "OK";
                        break;
                    case 8:
                        i++;
                        this.label16.Text = "OK";
                        break;
 
                }
            }
        } 
 
        private void Form1_Load(object sender, EventArgs e)
        {
            EtatConnexion = false;
            Serie = null;
 
            string[] ports = SerialPort.GetPortNames();
 
            foreach (string port in ports)
            {
                this.comboBox1.Items.Add(port);
            }
            this.comboBox1.SelectedIndex = 0;
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
 
 
 
            if(EtatConnexion == false)
            {
                this.serialPort1.PortName = this.comboBox1.SelectedItem.ToString();
                if (Serie == null)
                {
                    Serie = new CSerie(serialPort1, 9600);
                    try
                    {
                        EtatConnexion = Serie.Ouvrir();
                        this.CarteChameleon = new CChameleon(Serie);
                        this.button1.Text = "Deconnecter";
                        toolStripStatusLabel1.Text = "Etat : Connecté";
 
 
 
                        /*************************************************************/
                        /*************Configuration des pin de la carte***************/
                        /*************************************************************/
                        /*************************************************************/
 
 
                        CarteChameleon.ConfigurerSortieTOR(18, 1); // led de reponse carte
                        CarteChameleon.ConfigurerEntreeTOR(3, 2); //inter alarm
                        CarteChameleon.ConfigurerSortieTOR(5, 5); // led alarm + led chauffage + led volet
                        CarteChameleon.ConfigurerEntreeTOR(10, 1); //inter capteur lumiere
                        CarteChameleon.ConfigurerSortieTOR(12, 1); // buzzer alarm
 
                        /*************************************************************/
                        /*************initialisation des pins de la carte*************/
                        /*************************************************************/
                        /*************************************************************/
 
                        CarteChameleon.EcrireSortieTOR(18, 0);
                        CarteChameleon.EcrireSortieTOR(5, 0);
                        CarteChameleon.EcrireSortieTOR(6, 0);
                        CarteChameleon.EcrireSortieTOR(7, 0);
                        CarteChameleon.EcrireSortieTOR(8, 0);
                        CarteChameleon.EcrireSortieTOR(9, 0);
                        CarteChameleon.EcrireSortieTOR(12, 0);
 
 
 
 
                    }
                    catch
                    {
                         toolStripStatusLabel1.Text = "Etat : Erreur de connection";
                         MessageBox.Show("Port utilisé", "Erreur", MessageBoxButtons.OK, MessageBoxIcon.Stop);
                         Serie = null;
                    }
 
                }
            }
            else
            {
                if(this.EtatConnexion == true)
                {
                    EtatConnexion = Serie.Fermer();
                    Serie = null;
                    this.button1.Text = "Connecter";
                    toolStripStatusLabel1.Text = "Etat : Déconnecté";
                }
 
            }
 
        }
 
        private void button2_Click(object sender, EventArgs e)
        {
 
            CarteChameleon.AllumerLED();
 
        }
 
        private void button3_Click(object sender, EventArgs e)
        {
 
            CarteChameleon.EteindreLED();
        }
 
    //**********************************GESTION CAPTEURS**********************************//
 
 
 
 
 
 
    }
}

18/01/2012, 18h22
synolog

Si je comprend quand tu test tu appui sur le Button1 et CSerie ne s’instancie pas! 8O

Ce n'est pas normal! Met le BreakPoint sur le Button1 et dis nous ce qu'il en est!
S'il rentre dans la première condition??
S'il rentre dans la seconde??
Sinon instancie le chargement et test...
18/01/2012, 18h34
Amiral62

Quand je met le BreakPoint sur le Button1 il me met directement à l'erreur... :/
18/01/2012, 18h36
synolog

Et il y a autre chose, ta classe Cserie ne peut pas connaitre le serialPort ni la vitesse puisque tu ne lui déclare pas que c'est pour une méthode :?
A ta ligne 476 essai plutôt:

Code:

Serie.CSerie(serialPort1, 9600);

Cela devrait mieux marcher
18/01/2012, 18h48
Amiral62

Exactement ! Comme quoi on ne peut faire confiance aux profs :s lol
Par contre il me dit qu'il y a une référence manquante :

http://img11.imageshack.us/img11/8959/pb3pb.png
18/01/2012, 18h51
synolog

Change le Nom de ta méthode, il y a confusion!
18/01/2012, 19h01
BB-69

Bonjour,

Avant de corriger trop de choses à droite et à gauche pas forcement utile, il faudrait peut être commencer par démarrer le thread qui fait tourner 'Lire' après avoir appuyé sur le bouton et initialisé le port au lieu de le lancer lorsque la form est créée.
18/01/2012, 19h06
Bluedeep

Citation:

Envoyé par bugg7111

Et il y a autre chose, ta classe Cserie ne peut pas connaitre le serialPort ni la vitesse puisque tu ne lui déclare pas que c'est pour une méthode :?
A ta ligne 476 essai plutôt:

Code:

Serie.CSerie(serialPort1, 9600);

Cela devrait mieux marcher

8O

Appeler un constructeur comme une méthode d'instance,non, ça ne marche pas !
18/01/2012, 19h12
Amiral62

Bonsoir,
J'ai changer le nom de ma méthode, ça passe mais toujours la même erreur;
Ensuite j'ai lancer le thread après avoir appuyer sur le bouton mais quand tu dis " initialisé le port au lieu de le lancer lorsque la form est créée" tu parle de :

Code:

string[] ports = SerialPort.GetPortNames();

?
En tout cas merci pour vos réponses ;)
18/01/2012, 19h18
synolog

@Bluedeep
Ce n'est pas le constructeur, mais le nom de l'une des méthode que comprend la classe. :?

D'où la confusion de nom!
18/01/2012, 19h31
Amiral62

Le programme ce lance ! mais il y a des confusions avec les changements que l'ont a fait ;) je vais essayer de voir ça
18/01/2012, 19h42
BB-69

Si vous ne démarrez pas le thread, avez vous un message d'erreur ?
Vous devriez pouvoir utiliser votre bouton 1 pour connecter normalement.

Même en ayant le port connecté en fait, 'Lire' ne va pas faire grand chose à part une nouvelle erreur si le thread essaye de changer le valeur d'un label.

Le code vous à été fourni et vous devez faire afficher l'état des LED sur un écran ?
Précisez le code fourni et celui que vous avez ajouté ainsi que le but, cela permettra de voir plus clair.
18/01/2012, 19h46
Amiral62

Voilà ce que j'ai :

Je me connecte au port COM9 ou ce situ ma µChameleon

http://img846.imageshack.us/img846/162/pb5.png

Je clique sur le Bouton1...

http://img196.imageshack.us/img196/314/pb6y.png

Et voilà l'erreur :

http://img840.imageshack.us/img840/5846/pb4m.png
18/01/2012, 19h56
Amiral62

Citation:

Envoyé par BB-69

Si vous ne démarrez pas le thread, avez vous un message d'erreur ?
Vous devriez pouvoir utiliser votre bouton 1 pour connecter normalement.

Même en ayant le port connecté en fait, 'Lire' ne va pas faire grand chose à part une nouvelle erreur si le thread essaye de changer le valeur d'un label.

Le code vous à été fourni et vous devez faire afficher l'état des LED sur un écran ?
Précisez le code fourni et celui que vous avez ajouté ainsi que le but, cela permettra de voir plus clair.

Une partie du code a été fournis par le professeur pour un projet antérieur j'ai donc récupéré celui-ci affin de garder les fonctions relier à la µChameleon qui me permet de lire les entrées des Pins, le but de cette partie du projet et de géré 8 capteurs indépendants ( tout ou rien ).

Ici je voudrais voir les états des capteurs " en live " pour trouver un éventuelle mal fonctionnement du système : voir tout simplement quels capteurs sont actif ou non.

Les parties de code que j'ai moi même crée et/ou changer sont : la création du thread ainsi que sa methode Lire() , la méthode

Code:

public int LireEntreeTOR(int NumeroPin)

et j'ai ensuite changer le tableau suivant

Code:

Pin[] Bornier = new Pin[100];

avant Pin étais à 18 au lieu de 100 puis quelque référence concernant ComUSB.
18/01/2012, 20h06
BB-69

Revenez sur votre code d'origine en désactivant le thread déjà, les modifications faites sur une partie du code que le programme n'atteignait jamais pose surement problème.
18/01/2012, 21h04
Amiral62

Alors, j'ai repris le code à part j'ai réalisé quelque modifications :
Serie n'avais rien a faire dans le code, erreur de ma part Serie = Com en faite.
Maintenant quand je lance l'application ( sans le thread ) ça tourne je peut donc me connecter au port COM9 et vérifier si la carte µchameleon répond grâce a la led.
mais une fois le thread en place la même erreur surgit

Citation:

Le port est fermé

ça dois donc venir de mon thread, je vais éplucher ça ;)
18/01/2012, 21h08
Amiral62

peut être du au faite que j'ai mis la méthode du thread :

Code:

public void Lire()

en public et non en static ?
mais quand je la passe en static il ne reconnait pas

Code:

this

dans l'instruction :

Code:

this.label9.Text = "OK";
18/01/2012, 21h20
BB-69

Pour ce qui est de votre code dans un premier temps:

- Un petit tour sur les tuto. à propos des threads en particulier sur les methodes pour modifier un label (ou autre) dans la form.
- Votre fonction Lire doit tester à intervale régulière les led ? il va falloir revoir la conception car elle le fait pas.
- Dans 'LireEntreeTOR' vous n'utilisez pas le port ouvert fait par votre bouton de connexion, ce qui ne pourra donc pas fonctionner.

Etes vous certains de l'utilité de passer le test dans un thread séparé ?
18/01/2012, 21h26
Amiral62

ma fonction Lire dois afficher "OK" dans le label "Pas de Capteur" quand il y a présence d'un capteur (TOR, comme un interrupteur en faite )
je vais me renseigner pour modifier un label car j'ai un doute aussi ;)
18/01/2012, 21h32
BB-69

Citation:

Envoyé par Amiral62

peut être du au faite que j'ai mis la méthode du thread :

Code:

public void Lire()

en public et non en static ?

Non, cela ne vient pas de la déclaration de Lire qui peut rester private dans votre cas.

Je vous conseil de lire un cours sur le fonctionnement de public, private, static, etc . . .
Un petit coup d'oeil au passage sur 'this'.
Suffisamment d'information sur le site pour cela.
18/01/2012, 21h43
Bluedeep

Citation:

Envoyé par bugg7111

@Bluedeep
Ce n'est pas le constructeur, mais le nom de l'une des méthode que comprend la classe. :?

Pas du tout. Sa classe comporte deux constructeurs et aucune méthode avec le nom de la classe(d'ailleurs C# ne permet pas d'avoir une méthode qui a le nom de la classe, que ce soit une méthode d'instance ou statique).
19/01/2012, 08h53
Amiral62

Bonjour,
ou conseillez vous de mettre mon thread ?
19/01/2012, 09h32
synolog

Citation:

Envoyé par Bluedeep

Pas du tout. Sa classe comporte deux constructeurs et aucune méthode avec le nom de la classe(d'ailleurs C# ne permet pas d'avoir une méthode qui a le nom de la classe, que ce soit une méthode d'instance ou statique).

D'accord! Comme quoi on en apprend tout les jours :)

Le thread est en fait le dernier de vos soucis, avant de vouloir l'utiliser il va falloir revoir vos bases.
Vous ne semblez pas comprendre la différence entre les variables locales et globales. Avant de poursuivre je vous invite à reprendre vos cours sur le c# pour comprendre ce qui ne va pas et revenir avec quelques correction sur votre code.
Quelques annotations pour piste.

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
public partial class Form1 : Form
    {
        // Variable Globale
        private CSerie Serie;
        private bool EtatConnexion;
        private CChameleon CarteChameleon;
 
        public Form1()
        {
            InitializeComponent();
            CSerie Serie = new CSerie(); // Inutile de créer une variable locale ici que vous n'utilisez pas.
            CChameleon CarteChameleon = new CChameleon(); // Idem Inutile de créer une variable locale ici que vous n'utilisez pas.
            Thread th = new Thread(Lire); // Toujours pas certains de l'intéret de passer par un thread, mais bon c'est juste un avis. A voir une fois le reste terminé.
            th.Start();
        }
 
public void Lire()
       {
            int i = 1;
            int retour;
            int GestionCapteurs = 1;
            CChameleon CarteChameleon = new CChameleon(); // Pourquoi créer encore une variable locale ? Sans l'initialiser correctement pour une connexion en plus.
 
 
 
            retour = CarteChameleon.LireEntreeTOR(i); // Utilisation de la variable locale !?  
 
            if (retour == 1)
            {
                switch (GestionCapteurs)
                {
                    case 1:
                        i++; // Pourquoi incrémenter i que vous n'utilisez jamais ?
                        this.label9.Text = "OK";
                        break;
                    case 2:
                        i++;
                        this.label10.Text = "OK";
                        break;
                    case 3:
                        i++;
                        this.label11.Text = "OK";
                        break;
                    case 4:
                        i++;
                        this.label12.Text = "OK";
                        break;
                    case 5:
                        i++;
                        this.label13.Text = "OK";
                        break;
                    case 6:
                        i++;
                        this.label14.Text = "OK";
                        break;
                    case 7:
                        i++;
                        this.label15.Text = "OK";
                        break;
                    case 8:
                        i++;
                        this.label16.Text = "OK";
                        break;
 
                }
            }
        }
}

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
class CChameleon
    {
        CSerie ComUSB;
        Pin[] Bornier = new Pin[100];
 
        struct Pin{
            public int Config;
            public int Valeur;
        };
 
        public enum ConfigPin
        {  
           Entree = 0, 
           Sortie = 1           
        }  
 
        public CChameleon()
        {
            ComUSB = null;
        }
 
        public CChameleon(CSerie Ref)
        {
            ComUSB = Ref;
            this.InitialisationDefaut();
        }
 
public int LireEntreeTOR(int NumeroPin)
        {
            string Trame;
            string TrameReponse;
            int Etat = -1;
 
            CSerie ComUSB = new CSerie(); // A nouveau, pourquoi créer encore une variable locale ? Sans l'initialiser correctement pour une connexion en plus.
 
            if (this.Bornier[NumeroPin - 1].Config == 0) /* Si configurer en entree */
            {  
                Trame = "pin " + NumeroPin + " state";
                ComUSB.Envoi_Trame(Trame); // Utilisation de la variable locale !?
                TrameReponse = ComUSB.Recevoir_Trame(); // Utilisation de la variable locale !?
                Etat = this.DecoderEtatEntreeTOR(TrameReponse);
                this.Bornier[NumeroPin - 1].Valeur = Etat;
            }
            return Etat;
         }
}