web apiコントローラ（.netコア）でasync/awaitまたはtaskを使用

Question

返す集約オブジェクトを構築するコントローラを持つ.netコアAPIがあります。 作成するオブジェクトは、3つのメソッド呼び出しから得られたデータから成ります。これらはすべて互いに独立しており、互いに独立して実行できます。 現在、このコントローラーのパフォーマンスを改善するためにタスクを使用しています。現在のバージョンは次のようになりますタスクを使用すると、ここで最高のソリューションである場合、私は疑問に思って...しかし

[HttpGet] 
public IActionResult myControllerAction() 
{  
    var data1 = new sometype1(); 
    var data2 = new sometype2(); 
    var data3 = new List<sometype3>(); 

    var t1 = new Task(() => { data1 = service.getdata1(); }); 
    t1.Start(); 

    var t2 = new Task(() => { data2 = service.getdata2(); }); 
    t2.Start(); 

    var t3 = new Task(() => { data3 = service.getdata2(); }); 
    t3.Start(); 

    Task.WaitAll(t1, t2, t3); 

    var data = new returnObject 
    { 
     d1 = data1, 
     d2 = data2, 
     d2 = data3 
    }; 

    return Ok(data); 
} 

これがうまく機能しますか？ async/awaitを使用する方が良いアイデアであり、より受け入れられる方法でしょうか？

たとえば、コントローラが非同期としてマークされ、サービスメソッドへの呼び出しごとに待機する必要がありますか？

Answer 1

これは私はしかし、うまく機能タスクを使用するのが最善の解決策であるのだろうか？ async/awaitを使用する方が良いアイデアであり、より受け入れられる方法でしょうか？

はい、絶対に。 ASP.NETで並列処理を行うと、要求ごとに複数のスレッドが使用されるため、スケーラビリティに重大な影響を与える可能性があります。非同期処理はI/Oにとってはるかに優れています。

asyncを使用するには、まずサービス内のどこかで最低レベルのコールから始めます。おそらく、ある時点でHTTPコールを行っている可能性があります。非同期HTTP呼び出しを使用するように変更してください（例：HttpClient）。それから、asyncをそこから自然に生育させてください。

最終的に、あなたは、getdata1Async非同期getdata2Async、そのように同時に消費することができgetdata3Async方法、になってしまいます：3つのサービス・コールが進行中であるが、このアプローチで

[HttpGet] 
public async Task<IActionResult> myControllerAction() 
{ 
    var t1 = service.getdata1Async(); 
    var t2 = service.getdata2Async(); 
    var t3 = service.getdata3Async(); 
    await Task.WhenAll(t1, t2, t3); 

    var data = new returnObject 
    { 
    d1 = await t1, 
    d2 = await t2, 
    d3 = await t3 
    }; 

    return Ok(data); 
} 

、myControllerActionの代わりに、の代わりにのスレッドを使用します。

Answer 2

[HttpGet] 
public async Task<IActionResult> GetAsync() 
{  
    var t1 = Task.Run(() => service.getdata1()); 
    var t2 = Task.Run(() => service.getdata2()); 
    var t3 = Task.Run(() => service.getdata3()); 

    await Task.WhenAll(t1, t2, t3); 

    var data = new returnObject 
    { 
     d1 = t1.Status == TaskStatus.RanToCompletion ? t1.Result : null, 
     d2 = t2.Status == TaskStatus.RanToCompletion ? t2.Result : null, 
     d3 = t3.Status == TaskStatus.RanToCompletion ? t3.Result : null 
    }; 

    return Ok(data); 
} 

1. あなたがタスクを待っているとき、あなたの行動のスレッドが現在ブロックされています。待ち受け可能なTaskオブジェクトを返すには、TaskWhenAllを使用します。したがって、非同期メソッドを使用すると、スレッドをブロックするのではなく、タスクを待つことができます。
2. ローカル変数を作成してタスクに割り当てる代わりに、Task<T>を使用して必要な型の結果を返すことができます。代わりに作成してタスクを実行しているの
3. が、私はアクション名の規則を使用することをお勧めしますTask<TResult>.Run方法
4. を使用する - それが必要がGet
5. 次で始まる名前だ、アクションが要求をGET受け入れる場合は、タスクが正常に完了したかどうか確認する必要があります。これは、タスクの状態をチェックすることによって行われます。私のサンプルでは、​​一部のタスクが正常に完了しなかった場合、戻りオブジェクトのプロパティにnullの値を使用しました。別のアプローチを使用することができます。一部のタスクが失敗した場合はエラーを返します。

Answer 3

私が理解しているように、あなたはこれを並列に実行したいので、あなたのコードに間違いはないと思います。ガブリエルが述べたように、あなたはタスクの仕上げを待つことができます。

[HttpGet] 
public async IActionResult myControllerAction() 
{  
    var data1 = new sometype1(); 
    var data2 = new sometype2(); 
    var data3 = new List<sometype3>(); 

    var t1 = Task.Run(() => { data1 = service.getdata1(); }); 
    var t2 = Task.Run(() => { data2 = service.getdata2(); }); 
    var t3 = Task.Run(() => { data3 = service.getdata3(); }); 

    await Task.WhenAll(t1, t2, t3); // otherwise a thread will be blocked here 

    var data = new returnObject 
    { 
     d1 = data1, 
     d2 = data2, 
     d2 = data3 
    }; 

return Ok(data); 
} 

あなたはまた、全体的に「より良い」（コメントを参照）のコードのいくつかの行を保存し、コードを作成するタスクの結果を使用することができます：

[HttpGet] 
public async IActionResult myControllerAction() 
{  
    var t1 = Task.Run(() => service.getdata1()); 
    var t2 = Task.Run(() => service.getdata2()); 
    var t3 = Task.Run(() => service.getdata3()); 

    await Task.WhenAll(t1, t2, t3); // otherwise a thread will be blocked here 

    var data = new returnObject 
    { 
     d1 = t1.Result, 
     d2 = t2.Result, 
     d2 = t3.Result 
    }; 

return Ok(data); 
}