SignalR Hub 中应该使用异步方法还是非异步方法？

Question

我在SignalR 部分中有以下Hub 类，在这里我定义了与连接相关的所有方法：

public override Task OnConnected()
{
    // here I cannot call this, and need to convert this method async
     await AddToGroup("stockGroup");
    //

    string name = Context.User.Identity.Name;
    _connections.Add(name, Context.ConnectionId);
    return base.OnConnected();
}

public async Task AddToGroup(string groupName)
{
    await Groups.Add(Context.ConnectionId, groupName);
    await Clients.Group(groupName).SendAsync("Send", $"{Context.ConnectionId} joined");
}

我查看了许多关于这些连接方法（和其他 Hub 方法）的不同示例，发现其中一些使用 async 方法，而另一些则不使用。在上面的示例中，我需要将 OnConnected() 方法转换为 async 以便调用 AddToGroup() 方法。当然相反的情况也可以，但我不确定哪个更好。那么，对于Hub 中的所有方法，我应该使用async 方法还是非异步方法？任何帮助，将不胜感激。

更新 1： 转换方法（到异步）。

public override async Task OnConnected()
{        
    await AddToGroup("stockGroup");

    string name = Context.User.Identity.Name;
    _connections.Add(name, Context.ConnectionId);
    return base.OnConnected();
}

更新 2：

public override async Task OnConnected()
{
    // #1 There is no async method in "Microsoft.AspNet.SignalR" library. 
    //await Groups.AddToGroupAsync(Context.ConnectionId, "SignalR Users");  

    // #2 I just have sync version of "AddToGroupAsync()" and used it
    await Groups.Add(Context.ConnectionId, "SignalR Users");

    /* #3 I think there is no need to use this custom method in the Hub. 
    Because the same method is already exist in the IGroupManager interface */
    //await AddToGroup("jiraGroup");        

    string name = Context.User.Identity.Name;
    _connections.Add(name, Context.ConnectionId);

    // #4 Here also the same problem and I used sync version of OnConnected()
    //await base.OnConnectedAsync();
    await base.OnConnected();
}

Answer 1

如果您需要 await 在方法中，只需添加 async。 async 是编译器生成异步状态机的标记。

should I use async- 如果需要等待，请使用。

如果您不需要await，只需返回任务本身而不需要async。在这种情况下，您可以避免创建状态机。

也请阅读 Stephen Toub 的Async/Await FAQ：

当你用“async”关键字标记一个方法时，你真的在​​告诉 编译器两件事：

1. 您是在告诉编译器您希望能够使用 方法中的“await”关键字（如果 并且仅当它所在的方法或 lambda 被标记为异步时）。这样做 所以，你告诉编译器使用状态来编译方法 机器，这样该方法将能够暂停然后恢复 在等待点异步。

2. 您是在告诉编译器“提升” 方法的结果或可能发生的任何异常 返回类型。对于返回 Task 或 Task 的方法，此 表示任何返回值或异常在 该方法存储在结果任务中。对于返回的方法 void，这意味着任何异常都会传播到调用者的 通过当时当前的任何“SynchronizationContext”上下文 方法的初始调用。

Answer 2

您应该将 Async 方法与 Await 一起使用并返回 Task。

异步是病毒式的，所以你应该避免 Async Void

不好

 public async void MethodAsync()
    {
        var result = await SendAsync();
        DoSomething(result);
    }

不错

 public async Task MethodAsync()
    {
        var result = await SendAsync();
        DoSomething(result);
    }

@davidfowl here 提供了一些很棒的异步指南

更新：删除返回

public override async Task OnConnected()
{        
    await AddToGroup("stockGroup");

    string name = Context.User.Identity.Name;
    _connections.Add(name, Context.ConnectionId);
    await base.OnConnected();
}