这是用异步/等待回调包装方法的正确方法吗？

Question

我工作的，可以发送结构化消息来回在网络库。该库有一个Send方法，需要将Action作为回调。当调用Send时，消息被发送到服务器，当收到该消息的响应时，存储的回调将被检索并执行。

这一切都完美的工作，但我想添加一个SendAsync方法，包装Send，以便可以使用异步/等待，而不是直接传递Action。我已经得到它的工作，但我不知道如果我正确地做，或者如果我以后会遇到问题。特别有一条线真的困扰着我。

Send看起来是这样的：

public void Send(Packet packet, Action<Response> callback) 
{ 
    if (packet is Request) 
    { 
     RegisterResponseCallback(packet, callback); 
    } 
    Send(packet); 
} 

异步SendAsync看起来是这样的：

public Task<Response> SendAsync(Request request) 
{ 
    var tcs = new TaskCompletionSource<Response>(); 
    Send(request, (response) => { tcs.TrySetResult(response); }); 
    return tcs.Task; 
} 

我打电话SendAsync这个代码的一些时间，我也得到了正确的反应早在与正确的时间正确的订单：

private async Task Connected() 
{ 
    Console.WriteLine((await SendAsync(new EchoRequest() { Text = "Test 1" })).Message); 
    Console.WriteLine((await SendAsync(new EchoRequest() { Text = "Test 2" })).Message); 
    Console.WriteLine((await SendAsync(new EchoRequest() { Text = "Test 3" })).Message); 

困扰我的线路是那个叫Connected的线路。这是一种方法，我不想标记async由于各种原因。

protected override void StatusChanged(NetIncomingMessage message) 
{ 
    switch (message.SenderConnection.Status) 
    { 
     case NetConnectionStatus.Connected: 
      _serverConnection = new ServerConnection(this, message.SenderConnection); 
      (new Task(async() => { await Connected(); })).Start(); 
      break; 
    } 
} 

具体来说，读取(new Task(async() => { await Connected(); })).Start();行，感觉很“臭”给我，但是这是我能想出获得Visual Studio中停止给我警告的唯一途径。

有我包Send正确的异步/ AWAIT，或将任何东西我还这里做使我的问题了线？有没有更好的方式给我打电话Connected？

Answer 1

我觉得StatusChanged是事件处理程序？因此你需要async void。

protected override async void StatusChanged(NetIncomingMessage message) 
{ 
    //use await here 
} 

编辑因为它不是一个事件处理程序，你有各种理由不将其标记为async，我能给的唯一建议是：使用Task.Run代替new Task，它可能是这里如果最合适的解决方案你不想改变方法的签名。

Task.Run(async() => await Connected()); 

new Task vs Task.Factory.StartNew

Task.Run vs Task.Factory.StartNew