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 常见问题解答：

当您使用“async”关键字标记方法时，您实际上是在告诉编译器两件事：

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

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

Answer 2

您应该将Async方法与await一起使用并return Task 。

异步是病毒式的，所以你应该避免async void 。

坏的：

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

好的：

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

@davidfowl 这里有一些很棒的异步指南。

更新：删除return ：

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

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