75

After asking this question, I am wondering if it is possible to wait for an event to be fired, and then get the event data and return part of it. Sort of like this:

private event MyEventHandler event;
public string ReadLine(){ return event.waitForValue().Message; }
...
event("My String");
...elsewhere...
var resp = ReadLine();

Please make sure whatever solution you provide returns the value directly rather than getting it from something else. I'm asking if the method above is available in some way. I know about Auto/ManuelResetEvent, but I don't know that they return the value directly like I did above.

Update: I declared an event using MyEventHandler (which contains a Message field). I have a method in another thread called ReadLine waiting for the event to fire. When the event fires the WaitForValue method (part of the event handling scene) returns the event args, which contains the message. The message is then returned by ReadLine to whatever had called it.

The accepted answer to that question I asked was what I did, but it just doesn't feel quite right. It almost feels like something could happen to the data between the ManuelResetEvent firing and the program retrieving the data and returning it.

Update: The main problem with the Auto/ManualResetEvent is that it is too vulnerable. A thread could wait for the event, and then not give enough time for anyone else to get it before changing it to something else. Is there a way to use locks or something else? Maybe using get and set statements.

12
  • what about a while loop: while (someGlobalvar); cahnge someGlobalvar in a function you asign to the event.
    – elyashiv
    Oct 5, 2012 at 12:04
  • 12
    it's active waiting, and it's a very bad solution Oct 5, 2012 at 12:07
  • 3
    Except that this question predates that one :) Jun 7, 2017 at 13:15
  • 1
    @Vahid, With 7 more years of programming experience since I asked this question, I would say that if you have to do this, you're probably doing it wrong. It's better to specify a callback. If you're working with old systems that don't allow anything else, such as COM, then just have code wait on a ManualResetEvent and set a second one to protect writing until all the threads are done reading, I guess. Feb 13, 2020 at 3:54
  • 1
    I would also recommend you check out reactivex.io. Feb 13, 2020 at 3:55

5 Answers 5

63

If the current method is async then you can use TaskCompletionSource. Create a field that the event handler and the current method can access.

    TaskCompletionSource<bool> tcs = null;

    private async void Button_Click(object sender, RoutedEventArgs e)
    {
        tcs = new TaskCompletionSource<bool>();
        await tcs.Task;
        WelcomeTitle.Text = "Finished work";
    }

    private void Button_Click2(object sender, RoutedEventArgs e)
    {
        tcs?.TrySetResult(true);
    }

This example uses a form that has a textblock named WelcomeTitle and two buttons. When the first button is clicked it starts the click event but stops at the await line. When the second button is clicked the task is completed and the WelcomeTitle text is updated. If you want to timeout as well then change

await tcs.Task;

to

await Task.WhenAny(tcs.Task, Task.Delay(25000));
if (tcs.Task.IsCompleted)
    WelcomeTitle.Text = "Task Completed";
else
    WelcomeTitle.Text = "Task Timed Out";
8
  • 3
    You don't need async to use the TaskCompletionSource
    – Felix Keil
    Oct 11, 2016 at 6:02
  • 2
    That's true, but you do need it if you want to await the Task without blocking the UI.
    – Adam
    Oct 12, 2016 at 18:04
  • Can a TaskCompletionSource be reset? Apr 2, 2018 at 21:19
  • 1
    @KyleDelaney No.In my example you would replace the tcs with a new instance
    – Adam
    Apr 3, 2018 at 0:01
  • How can you make this with event that is not a button click? Like is there a way to invoke an event wait for the handler to finish executing and then at the end call TrySetResult(true)?
    – Muhannad
    Mar 18, 2021 at 3:55
46

You can use ManualResetEvent. Reset the event before you fire secondary thread and then use the WaitOne() method to block the current thread. You can then have secondary thread set the ManualResetEvent which would cause the main thread to continue. Something like this:

ManualResetEvent oSignalEvent = new ManualResetEvent(false);

void SecondThread(){
    //DoStuff
    oSignalEvent.Set();
}

void Main(){
    //DoStuff
    //Call second thread
    System.Threading.Thread oSecondThread = new System.Threading.Thread(SecondThread);
    oSecondThread.Start();

    oSignalEvent.WaitOne(); //This thread will block here until the reset event is sent.
    oSignalEvent.Reset();
    //Do more stuff
}
1
  • This doesn't return the value directly. I already tried that in the other question. Isn't there anything like I'm asking about? Oct 5, 2012 at 12:24
4

A very easy kind of event you can wait for is the ManualResetEvent, and even better, the ManualResetEventSlim.

They have a WaitOne() method that does exactly that. You can wait forever, or set a timeout, or a "cancellation token" which is a way for you to decide to stop waiting for the event (if you want to cancel your work, or your app is asked to exit).

You fire them calling Set().

Here is the doc.

2
  • 1
    But that doesn't return any kind of value. Oct 5, 2012 at 12:25
  • 1
    Can you edit the qu and add an overview of the different components and their interactions? It will help us to see better what you are trying to achieve and hopefully lead to an improved design.
    – Mike
    Oct 5, 2012 at 12:29
1

If you're happy to use the Microsoft Reactive Extensions, then this can work nicely:

public class Foo
{
    public delegate void MyEventHandler(object source, MessageEventArgs args);
    public event MyEventHandler _event;
    public string ReadLine()
    {
        return Observable
            .FromEventPattern<MyEventHandler, MessageEventArgs>(
                h => this._event += h,
                h => this._event -= h)
            .Select(ep => ep.EventArgs.Message)
            .First();
    }
    public void SendLine(string message)
    {
        _event(this, new MessageEventArgs() { Message = message });
    }
}

public class MessageEventArgs : EventArgs
{
    public string Message;
}

I can use it like this:

var foo = new Foo();

ThreadPoolScheduler.Instance
    .Schedule(
        TimeSpan.FromSeconds(5.0),
        () => foo.SendLine("Bar!"));

var resp = foo.ReadLine();

Console.WriteLine(resp);

I needed to call the SendLine message on a different thread to avoid locking, but this code shows that it works as expected.

-1

Try it : e.Handled = true; It works to prevent KeyEventArgs, for example.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.