is returning an empty static task in TPL a bad practice?

Question

There are cases that I would want to run a task conditionally. I use some sort of extension method like this:

public static class MyTaskExtension{
  private stat         


        

Answer 1

It's perfectly acceptable to return an already completed task in some contexts. It's not something that is done particularly often, but it is done.

There's also nothing wrong at all with just using a single static completed task. There is no need to have a whole bunch of different tasks that are all identical, given that once they're completed, and if they have no result, there's nothing wrong with reusing them.

Note that if you want to return an already completed task you can use Task.FromResult to generate one with less overhead than what you're doing now, as you won't be creating an empty method, scheduling it, waiting for it to be started, and then have it finish right away. Just returning Task.FromResult(false) will give you an already completed task.

If you are using .NET 4.0 you can create your own FromResult easily enough:

public static Task FromResult(T result)
{
    var tcs = new TaskCompletionSource();
    tcs.SetResult(result);
    return tcs.Task;
}