Currently I have a code like this:
bool task1Result = await RunTask1(data);
if(!task1Result)
return false;
bool task2Result = await RunTask2(data);
if(!task2Result)
return false;
bool task3Result = await RunTask3(data);
if(!task3Result)
return false;
bool task4Result = await RunTask4(data);
if(!task4Result)
return false;
Added sample:
private async Task<bool> RunListOfTasks() {
CancellationTokenSource cts = new CancellationTokenSource();
CancellationToken ct = cts.Token;
var tasks = new List<Task<bool>> { RunTask1(data, ct), RunTask2(data, ct), RunTask3(data, ct), RunTask4(data, ct) };
while (tasks.Any())
{
var currentTask = await Task.WhenAny(tasks);
if (!await currentTask)
{
ct.Cancel();
return false;
}
tasks.Remove(currentTask);
}
return true;
}
Is it possible to run all of them in parallel and if one of them fails (like result is false), then stop processing the rest and return. Thanks
2
Answers
The
Task.WhenAny
-in-a-loop is generally considered an antipattern, because of its O(n²) complexity. The preferred approach is to wrap your tasks in another set of tasks, that will include the functionality of canceling theCancellationTokenSource
when the result isfalse
. Thenawait
the wrapper tasks instead of the initial tasks, and propagate their result.An easy way to wrap the tasks is the
Select
LINQ operator:Microsoft’s Reactive Framework does this in a very nice way:
It returns all of the results that come in right up to the point one of the tasks returns a
false
.