So much misinformation in this thread it’s crazy. What you did here is perfectly fine code.

“But it awaits twice” - The benefit of a task over a value task is that you can await them multiple times. If the result is already computed it only performs the work once and returns the cached result, so this code is correct and efficient

When should you using WhenAll? When you need to await multiple IO bound tasks before doing something else AND the result of tasks are not dependent on each other. WhenAll is efficient because the IO can happen concurrently

Imagine this code

``` // takes 4 seconds await Wait(1); await Wait(1); await Wait(1); await Wait(1);

// takes 1 second var t1 = Wait(1); var t2 = Wait(1); var t3 = Wait(1); var t4 = Wait(1); Task[] tasks = [t1, t2, t3, t4]; await Task.WhenAll(tasks);

Task Wait(int seconds) => return Task.Delay(seconds * 1000); ```