303

I've got this code:

private async void ContextMenuForGroupRightTapped(object sender, RightTappedRoutedEventArgs args)
{
    CheckBox ckbx = null;
    if (sender is CheckBox)
    {
        ckbx = sender as CheckBox;
    }
    if (null == ckbx)
    {
        return;
    }
    string groupName = ckbx.Content.ToString();

    var contextMenu = new PopupMenu();

    // Add a command to edit the current Group
    contextMenu.Commands.Add(new UICommand("Edit this Group", (contextMenuCmd) =>
    {
        Frame.Navigate(typeof(LocationGroupCreator), groupName);
    }));

    // Add a command to delete the current Group
    contextMenu.Commands.Add(new UICommand("Delete this Group", (contextMenuCmd) =>
    {
        SQLiteUtils slu = new SQLiteUtils();
        slu.DeleteGroupAsync(groupName); // this line raises Resharper's hackles, but appending await raises err msg. Where should the "async" be?
    }));

    // Show the context menu at the position the image was right-clicked
    await contextMenu.ShowAsync(args.GetPosition(this));
}

...that Resharper's inspection complained about with, "Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call" (on the line with the comment).

And so, I prepended an "await" to it but, of course, I then need to add an "async" somewhere, too - but where?

2

3 Answers 3

468
+350

To mark a lambda async, simply prepend async before its argument list:

// Add a command to delete the current Group
contextMenu.Commands.Add(new UICommand("Delete this Group", async (contextMenuCmd) =>
{
    SQLiteUtils slu = new SQLiteUtils();
    await slu.DeleteGroupAsync(groupName);
}));
7
  • 3
    I get an error from Visual Studio that Async void methods are not supported. Commented Dec 11, 2019 at 16:01
  • 1
    @Kevin Burton: Yeah, async voids are usually only limited to event handlers. The API you're using is either not async or has an async version expecting an async Task lambda instead.
    – BoltClock
    Commented Dec 17, 2019 at 17:16
  • Can anyone help me out by giving some link to explanation how such async lambdas would execute?
    – ed22
    Commented Jul 26, 2021 at 11:00
  • @ed22: Does this help? learn.microsoft.com/en-us/dotnet/csharp/programming-guide/…
    – BoltClock
    Commented Jul 26, 2021 at 13:07
  • @BoltClock, thanks but I still don't understand how async lambda parameter that is not awaited executes...
    – ed22
    Commented Jul 26, 2021 at 14:37
61
+500

And for those of you using an anonymous expression:

await Task.Run(async () =>
{
   SQLLiteUtils slu = new SQLiteUtils();
   await slu.DeleteGroupAsync(groupname);
});
10

If you are inside a LINQ method syntax apply the async keyword right before the parameter:

 list.Select(async x =>
            {
                await SomeMethod(x);
                return true;
            });
4
  • What does the return true; represents? Commented Nov 15, 2021 at 16:08
  • @TheodorZoulias sorry for the poor explanation. The return true; represents what each value of the list should return, in my case the list was List<bool>, so from one list you are creating another list with Select() method. For the sake of simplicity I removed my logic. The important part here is that I needed to call an awaitable method inside a lambda expression and that is done by putting the async keyword. Commented Nov 17, 2021 at 13:17
  • 1
    Drilon if the list is a List<bool>, then the x is a bool. This is clear. The return true; is not clear. Why would you return a constant value instead of the result of the await SomeMethod(x), or at least the x itself? And where is this return value used? Your example does not assign the result of the list.Select to anything. Sorry, but for the time being I have to downvote the answer. You could try to improve it by editing it if you want. Commented Nov 17, 2021 at 13:35
  • You don't have to be sorry for what you are... The return depends on the awaitable method. Example. if SomeMethod returns not null , return true, else return false, and that will do a mapping of each vaule. But I did not want to provide all the code since it is irrelevant for the async keyword problem. Commented Nov 18, 2021 at 11:01

Not the answer you're looking for? Browse other questions tagged or ask your own question.