Discussion :

[agh]

Bonsoir à tous,

Mon problème est simple, et pourtant je bloque :

J'ai un thread principal, qui lance un autre thread :

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
public event EventHandler usbLecture;
 
public Usb()
{
   threadDetection = new Thread(new ThreadStart(lire));
   threadDetection.Start();
 
   // ...
}
 
private void lire()
{
   while (Thread.CurrentThread.IsAlive)
   {
      Thread.Sleep(500);
 
       // traitement
 
       OnUsbLecture(new EvtUsb (EventArgs.Empty));
   }
}

Le thread se lance bien, le traitement s'effectue, mais au moment de déclencher l'évenement que voici :

Code :

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

1
2
3
4
protected virtual void OnUsbLecture(EventArgs e)
{
   SensChange(this, e);
}

J'ai un NullReferenceException au niveau de SensChange (this, e)

Pourriez-vous m'aider ?
J'imagine que ça a avoir avec les thread, parce que j'utilise tout le temps les évenements, mais au sein d'un même thread...

Je vous remercie beaucoup.
Alexis

[StormimOn]

Ca a juste avoir avec le fait que tu n'es pas abonné à l'événement et que tu ne déclenches pas l'événement comme il faut. Regarde cet exemple de la MSDN pour voir comment bien procéder

[tomlev]

Ca a juste avoir avec le fait que tu n'es pas abonné à l'événement et que tu ne déclenches pas l'événement comme il faut. Regarde cet exemple de la MSDN pour voir comment bien procéder

Effectivement... Pour compléter : s'il n'y a aucun handler abonné à l'évènement SensChange, alors le delegate SensChange vaut null, donc tu obtiens une NullReferenceException si tu essaies de l'invoquer.

La technique habituelle pour déclencher un évènement est la suivante :

Code :

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

1
2
3
4
5
6
7
8
public event EventHandler MyEvent;
 
protected virtual void OnMyEvent()
{
    EventHandler handler = MyEvent;
    if (handler != null)
        handler(this, EventArgs.Empty);
}

(pour une explication sur la variable temporaire "handler", regarde cet article)

Sinon, juste une petite remarque : fais attention aux noms que tu donnes à tes méthodes et évènements, parce que là on y comprend pas grand chose ... Tu déclares un évènement usbLecture, mais la méthode OnUsbLecture déclenche un autre évènement qui s'appelle SensChange

[StormimOn]

Histoire de compléter aussi, une autre façon de faire et d'utiliser une méthode anonyme :

Code :

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

1
2
3
4
5
6
public event EventHandler MyEvent = delegate { };
 
protected virtual void OnMyEvent()
{
    MyEvent (this, EventArgs.Empty);
}

MyEvent ne sera jamais null de cette manière.

[agh]

Merci pour vos réponses.

Ca marche.

Et oui, désolé, je m'étais trompé dans mon message il fallait bien sur lire :

Code :

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

1
2
3
4
protected virtual void OnUsbLecture(EventArgs e)
{
   usbLecture (this, e);         
}

...
Merci encore !

Alexis