Multicast-Delegat mit Rückgabewert: Wie das Überschreiben von Ergebnissen verhindern?

Hallo zusammen,

ich bin in einem unserer Projekte gerade auf folgenden Code gestoßen:

public delegate bool MyDelegate(Control page1, Control page2);
...
public event MyDelegate ValidateQuantity;
...


var valid = true;
var handler = ValidateQuantity;
if (handler != null) {
    valid = handler(page1, page2);
}
return valid;

Das kann/darf man doch so nicht schreiben, oder? Man weiß doch überhaupt nicht wieviele Handler sich registriert haben und von daher überschreibt wohlmöglich ein Handler-Rückgabewert einen anderen. Oder sehe ich das falsch?

MfG
wax

Hallo Wax,

das siehst du schon richtig. Wenns wie hier um den Rückgabewert geht*, so sollten die Delegaten in einer Schleife abgearbeitet werden. Mittels GetInvocationList kannst du dir die Delegaten holen und dann abarbeiten.

* ich würde das nicht per Event lösen, sondern per normalen Methoden-Aufruf um z.B. genau dieses Problem von vornherein zu vermeiden

mfG Gü

Hallo Wax,

im Framework wird denn auch bei Events (die ja per se die Möglichkeit von Mulitcasts in Betracht ziehen / ziehen müssen) statt eines Rückgabewerts eine Property in den EventArgs verwendet, z.B. CancelEventArgs.Cancel.

Aber auch da kommt es auf die Kooperationsbereitschaft der EventHandler an, damit die sich den Wert nicht gegenseitig überschreiben. Also z.B. ein EventHandler nicht einfach am Anfang Cancel auf false "initialisiert" und damit möglicherweise das schon gesetzte true eines anderen EventHandlers überschreibt.

Erschwerend kommt hinzu, dass die Reihenfolge der Abarbeitung der EventHandler nicht definiert/spezifiziert ist ... und selbst wenn sie es wäre, sollte man von der Reihenfolge nichts abhängig machen.

Wenn das event-auslösende Objekt die Kontrolle darüber behalten will, welches der Ergebnisse der einzelnen EventHandler gewinnt bzw. wie die Teilergebnisse zu einem Gesamtergebnis zusammengefasst werden und man das nicht der Kooperation der EventHandler überlassen will, könnte man entsprechenden Code in den Setter der Property in den EventArgs einfügen. Oder man verwendet eine Methode (die den Wert als Parameter bekommt) statt einer Property, damit klarer wird, dass noch eine Logik dahinter steckt.

herbivore

Hallo,

schau Dir auch mal Multicast-Delegaten: Auswertung aller Rückgabewerte und Abbruchmöglichkeit an. Damit kommt man an alle Rückgabewerte aller registrierten Handler, und auch an die entsprechenden Handler die die Werte zurückgeben.

Ist für Deinen Fall nur möglicherweise eine Lösung, in diesem Zusammenhang aber zumindest interessant.

Gruß, MarsStein