Winforms async/awaitタスクが返されないようにする

Question

私はWinforms exeを持っており、メニューからは、タスクとして低速実行プロセスを起動します。データを取得してからダイアログを表示するまでには約30秒かかります。通常、それはもはや返されません。私は例外をキャッチし、ログに何も表示されないので、それが正常に実行されていることがわかります。フォームは決して表示されず、CPU時間がかかっていないようです。しかし、私はデバッガでそれを実行し、コードをステップ実行し、正常に動作します。時々、より高速のPCで動作するように見えます。何が起こっている？

private async void inPlayRecordToolStripMenuItem_Click(object sender, EventArgs e) 
    { 
     if (!GetClient()) return; 
     { 
      await Task.Run(() => 
      { 
       LaunchForm(); 
      }); 
     } 
    } 

    private async void LaunchForm() 
    { 
     try 
     { 
      { 
       var inPlayView = new InPlayView(); 
       await inPlayView.GetData(); 
       inPlayView.ShowDialog(); 
      } 
     } 
     catch (Exception ex) 
     { 
      Logger.LogMessage(ex.ToString()); 
     } 
    } 

Answer 1

代わりにこれを行います。

private async void inPlayRecordToolStripMenuItem_Click(object sender, EventArgs e) 
{ 
    if (!GetClient()) return; 

    await LaunchForm(); 
} 

private async Task LaunchForm() 
{ 
    try 
    { 
     var inPlayView = new InPlayView(); 
     await inPlayView.GetData(); 
     inPlayView.ShowDialog(); 
    } 
    catch (Exception ex) 
    { 
     Logger.LogMessage(ex.ToString()); 
    } 
} 

あなたは既に非同期メソッドのTask.Run()をしたくない、と一般的なルールとして、async voidので、LaunchForm()イベントハンドラのみ大丈夫ではありません方法。

また、コメントとして指摘すると、Task.Run()はThreadPoolにタスクをキューイングするので、UIスレッドから終了します。

Answer 2

自分のプロジェクトでasync/awaitを使用しましたが、タスクでShowDialogを実行する理由が考えられません。これがうまくいくかどうかはわかりませんが、あなたの流れを少し変更したいかもしれません。これにより、より一貫性があり、デバッグが容易になるはずです。

private async void inPlayRecordToolStripMenuItem_Click(object sender, EventArgs e) { 
    if (!GetClient()) { 
     return; 
    } 

    var playView = await LaunchForm(); 

    if (playView != null) { 
     playView.ShowDialog(); 
    } 
} 

private async Task<InPlayView> LaunchForm() { 
    try { 
     var inPlayView = new InPlayView(); 
     await inPlayView.GetData(); 

     return inPlayView; 
    } catch (Exception ex) { 
     // do cleanup of view if needed 
     Logger.LogMessage(ex.ToString()); 
     return null; 
    } 
}