c# Task.Run Syntax

Question

I want to run listOrders() on a separate thread and make it awaitable. I have a method of this form, that compiles and works OK

private Task<ClearedOrdersReport> listOrdersAwaitable(int a, int b)
{
    Task<ClearedOrdersReport> t = Task.Run(() =>
       listOrders(a, b));
    return t;
}

I now want to add a second line to execute before the listOrders() call. I thought it would be trivial to change the single line lambda into a block style lambda, but I cannot figure out what the correct syntax is to get it to compile ? I am probably missing something obvious, but any help would be appreciated. Thanks

Your question of "wanting to run listOrders() on a separate thread" doesn't make a lot of sense to me. If you just make it async, and await it, it automatically decides where to run it with the Task Scheduler. — Ilan Keshet
– Ilan Keshet, Commented Jul 8, 2022 at 0:37
Also, by you just calling the function (and it not being async) -- it's already running as a task — Ilan Keshet
– Ilan Keshet, Commented Jul 8, 2022 at 0:38
I think more information about the bigger picture needs to be stated, because what you are asking is not making a lot of sense. — Ilan Keshet
– Ilan Keshet, Commented Jul 8, 2022 at 0:40

Abdelkrim · Accepted Answer · 2022-07-08 01:08:27Z

3

You should use a statement lambda and return the result of listOrders.

 private Task<ClearedOrdersReport> listOrdersAwaitable(int a, int b)
 {
        Task<ClearedOrdersReport> t = Task.Run(() => {
           //add your second line here. 
           return listOrders(a, b);
        });
        return t;
 }

answered Jul 8, 2022 at 1:08

Abdelkrim

1,24011 silver badges21 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

geoffw123 Over a year ago

Thanks for all of the responses. This solved it for me, my silly oversight was that in going from a single line lambda to a block lambda you dont have the implicit return statement. I forgot to add the return statement before listOrders().

Abdelkrim Over a year ago

Don't forget to mark the answer as accepted, good luck.

tmaj · Accepted Answer · 2022-07-08 02:59:38Z

0

There are some questions about what you are trying to do with the async/await elements, but one to call two many things where there was one is to use a helper method:

listOrders(a, b));

can become

DoWork(a,b);

and then

ClearedOrdersReport DoWork(int a, int b)
{
 ...
 return listOrders(a, b); 
}

answered Jul 8, 2022 at 2:59

tmaj

35.9k12 gold badges108 silver badges143 bronze badges

Collectives™ on Stack Overflow

c# Task.Run Syntax

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related