I want to use TaskCompletionSource
to wrap MyService
which is a simple service:
public static Task ProcessAsync(MyService
The issue here is that the Completed
event is raised on each action but the TaskCompletionSource
can only be completed once.
You can still use a local TaskCompletionSource
(and you should). You just need to unregister the callback before completing the TaskCompletionSource
. That way this specific callback with this specific TaskCompletionSource
will never be called again:
public static Task ProcessAsync(MyService service, int parameter)
{
var tcs = new TaskCompletionSource();
EventHandler callback = null;
callback = (sender, e) =>
{
service.Completed -= callback;
tcs.SetResult(e.Result);
};
service.Completed += callback;
service.RunAsync(parameter);
return tcs.Task;
}
This will also solve the possible memory leak that you have when your service keeps references to all these delegates.
You should keep in mind though that you can't have multiple of these operations running concurrently. At least not unless you have a way to match requests and responses.