views:

218

answers:

3

I am doing a small multithreaded app that uses asynchronous TCP sockets, but I will get to the point: I am using a custom event to read a value from a form and the delegate used by the event returns a string when finished.

My question here is: is that correct? is it Ok to return values from the events? or is there a best way to do this? (like using a simple delegate to the form to read the values)

Thanks in advance,

+6  A: 

The closest example I can think of is the FormClosing event in WinForms. It lets the form cancel the event by setting the eventArgs.Cancel property to true. For you to do something similar, you would define your own event args class with the return value as a property on that class. Then pass an event args object whenever you raise the event. Whoever raised the event can inspect the event args object for the return value. Others who are receiving the event can also inspect or change the event args object.

Update: I just ran across the AppDomain.AssemblyResolve event, and it appears to be an event that returns a value. It seems you just need to declare a delegate type that returns a value, and then define your event with that delegate type. I haven't tried creating my own event like this, though. One advantage to using a property on the event argument is that all subscribers to the event can see what previous subscribers have returned.

Don Kirkby
+4  A: 

It's often awkward to return values from events. In practice, I've found it much easier to include a writable property on a set of custom EventArgs that is passed to the event, and then checked after the event fires -- similar to Cancel property of the WinForms FormClosing event.

Dustin Campbell
+3  A: 

I don't think it's a good idea... events are basically multicast delegates, so there can be multiple handlers. Which return value will you take in that case ?

Thomas Levesque