EventHandler C#

[CsharpUser (Gast)]

Guten Morgen,

ich benötige einen EventHandler. Dazu habe ich zunächst einen 
EventHandler definiert:

public event EventHandler<bool> Connected;

In dieser Funktion wird der bool Wert vom EventHandler auf true gesetzt.

public void Test()
{
            var p = this.Connected;
            if (p != null)
            {
                p(this, true);
            }
}

Folgende Fehlermeldung erscheint beim übersetzen:
Der Typ "bool" kann nicht als Typparameter "TEventArgs" im generischen 
Typ oder in der generischen Methode "System.EventHandler<TEventArgs>" 
verwendet werden. Es ist keine Boxing-Konvertierung von "bool" in 
"System.EventArgs" vorhanden.

Was mache ich hier möglicheriweise falsch?

[Borislav B. (boris_b)]

Versuchs doch mal mit

public event Action<object, bool> Connected;

Das Auslösen des Events kannst du auch noch etwas kompakter formulieren:

if(Connected != null) Connected(this, true);

[Andreas R. (andreasr)]

> Das Auslösen des Events kannst du auch noch etwas kompakter formulieren:
>

> if(Connected != null) Connected(this, true);
>

Bitte nicht machen, das ist nicht thread safe!

[Borislav B. (boris_b)]

Andreas R. schrieb:
> Bitte nicht machen, das ist nicht thread safe!

So hat  CsharpUser  das ja bisher auch gemacht, also bin ich davon 
ausgegangen, dass es das ist, was er will. Ist ja nicht unbedingt 
schlimm.

Alternativ könnte man auch ein Invoke(...) verwenden.

[Andreas R. (andreasr)]

>So hat  CsharpUser  das ja bisher auch gemacht

Nein hat er nicht, er hat den Event-Handler vor der null-Abfrage einer 
lokalen Variable zugewiesen. Das macht hier den Unterschied aus.

Der eigentliche Fehler liegt darin, dass bool nicht als 
Template-Argument genommen werden kann.
Es muss entweder EventArgs oder eine ableleitete Klasse von EventArgs 
sein.
Wenn man einen boolean versenden möchte muss man den also in eine Klasse 
einpacken. z.B. so

class MyEventArgs : EventArgs
{
    public bool Data { get; set;}
}

public event EventHandler<MyEventArgs> Connected;

[R. (Gast)]

Andreas R. schrieb:
> Nein hat er nicht, er hat den Event-Handler vor der null-Abfrage einer
> lokalen Variable zugewiesen. Das macht hier den Unterschied aus.

Das erscheint mir auch nicht ganz sauber. Wenn in einem zweiten Thread 
zeitlich zwischen der p-Zuweisung und p-Invoke ein Handler an das Event 
an- oder davon abgehängt wird, wird der Handler nicht oder noch 
ausgeführt. Das ist auch nicht thread safe, wenn auch keine 
NullReferenceException ausgelöst werden kann.
IMHO muss das eh synchronisiert werden, weshalb ich die Lösung von Boris 
P. so auch verwende. Sorry für OT!

Da wir in unserer etwas grösseren Solution öfter mal solche EventArgs 
mit unterschiedlichen Typen brauchen, haben wir eine generische Klasse 
erstellt:

public class EventArgs<T> : EventArgs
{
    private T item;

    public EventArgs(T item)
    {
        this.item = item;
    }

    public T Item
    {
        get { return item; }
    }
}

[Frank (Gast)]

Auch wenn das Hauptproblem des TO gelöst sein dürfte, hier ein paar 
Links zum Thema Events und Threadsicherheit:

U.a. Eric Lipperts Sichtweise zum "Stale Handler Problem":

http://blogs.msdn.com/b/ericlippert/archive/2009/04/29/events-and-races.aspx

Reihe zu Events in C# 4 und höher:

http://blogs.msdn.com/b/cburrows/archive/2010/03/05/events-get-a-little-overhaul-in-c-4-part-i-locks.aspx

http://blogs.msdn.com/b/cburrows/archive/2010/03/08/events-get-a-little-overhaul-in-c-4-part-ii-semantic-changes-and.aspx

http://blogs.msdn.com/b/cburrows/archive/2010/03/18/events-get-a-little-overhaul-in-c-4-part-iii-breaking-changes.aspx

http://blogs.msdn.com/b/cburrows/archive/2010/03/30/events-get-a-little-overhaul-in-c-4-afterward-effective-events.aspx

C# 6 "Elvis operator" bei Events sowie Volatile.Read in .NET 4.5 und 
höher:

h*tp://codeblog.jonskeet.uk/2015/01/30/clean-event-handlers-invocation-w 
ith-c-6/

(t = * wegen Spamschutz)

Nicht vergessen, auch die Kommentare zumindest zu überfliegen.