質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.61%

  • C#

    6891questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • .NET Framework

    447questions

    .NET Framework は、Microsoft Windowsのオペレーティングシステムのために開発されたソフトウェア開発環境/実行環境です。多くのプログラミング言語をサポートしています。

  • 非同期処理

    105questions

    非同期処理とは一部のコードを別々のスレッドで実行させる手法です。アプリケーションのパフォーマンスを向上させる目的でこの手法を用います。

WinFormsのFormClosingイベントハンドラをasyncにすると画面が閉じません

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 733

ry18847

score 1

asyncなFormClosingイベントハンドラで、以下の実装をしたところFormClosedイベントが起きず、フォームが閉じません。
ブレークポイントを置いて実行したところ、e.Cancel = false; は実行されるのですが、なぜ画面が閉じないのでしょうか?

該当のソースコード

        private async void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            e.Cancel = true;
            await Task.Delay(1000); //重たい処理
            e.Cancel = false;
         }

試したこと

以下のコードの場合、閉じたり閉じなかったりします。
コントロールボックスの×ボタンを連打するとそのうち閉じるのですが、なぜこんな動作をするんでしょうか?
(Taskが即座に終わるかどうかが関係している?)

        private async void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            e.Cancel = true;
            await Task.Run(() =>
            {
                return;
            });
            e.Cancel = false;
        }

補足情報(言語/FW/ツール等のバージョンなど)

.Net Framework 4.5
VS2012

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

await で一度イベントハンドラが終了します。その時 e.Cancel は true です。
ですのでフォームは閉じません。

しかし e.Cancel が評価される前にタスクが終了して e.Cancel が書き換えられるとフォームは閉じます。

追記

わかりにくかったかもしれないので順を追って説明します。

これは System.Windows.Forms.Form のソースです。
Form.cs

OnFormClosing(e);
if (e.Cancel)
{
    dialogResult = DialogResult.None;
}
else
{
    // we have called closing here, and it wasn't cancelled, so we're expecting a close
    // call again soon.
    CalledClosing = true;
}

上記の OnFormClosing(e); で FormClosing イベントハンドラを(あれば)呼びます。
ユーザーはハンドラの中で e.Cancel を設定します。
イベントハンドラが次のようになっている場合

private async void Form1_FormClosing(object sender, FormClosingEventArgs e)
{
    e.Cancel = true;
    await Task.Run(() =>
    {
        return;
    });
    e.Cancel = false;
}

await によって一度ハンドラが終了するので、次のようなハンドラを指定したのと同じことになります。

private async void Form1_FormClosing(object sender, FormClosingEventArgs e)
{
    e.Cancel = true;
}

そして同時に次の非同期処理が登録されます。
(実際に実行されるのは同時とは限りません)

Task.Run(() =>
{
    return;
});
e.Cancel = false;

この非同期処理は return のみのタスクを実行し、それが終了すると e.Cancel =  false というタスクを実行しますので、整理すると次の内容とほぼ同じです。

Task.Run(() =>
{
    e.Cancel = false;
});

さてこの非同期処理がいつ行われるかですが、それは実行してみるまでわかりません。
この処理によって e.Cancel が書き換えられるわけですが、それが Form.cs 中の

if (e.Cancel)

この部分より前に行われると、e.Cancel は false となって else ブロックが実行されます。
また、後に行われたならば真のブロックが実行されます。

重たい処理の場合はおそらく間に合わないので e.Cancel は true となります。
二番目の例の場合は間に合うか間に合わないかは状況次第です。

以上が処理の流れとなります。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/01/09 12:34

    await はその名前から「待つ」と理解しがちですが、「非同期タスクが終わった後の処理を予約して終了する」と考えた方が理解しやすいです。

    キャンセル

  • 2018/01/09 13:20

    回答ありがとうございます。
    FormClosingイベントハンドラの処理がすべて完了してからFormClosedが起きると思っていましたが、違うのですね。

    ということは、FormClosingイベントで非同期に重い処理をしたい場合は、重たい処理以降で必要に応じてthis.Close();し直す(この場合は重たい処理が再度走らないようにフラグ管理等する)必要があるという認識でいいでしょうか?
    (そもそも重い処理を終わらせてからthis.Close(); したほうがいいのでしょうけど・・・)

    キャンセル

  • 2018/01/09 13:22

    フォームが閉じるのに必要な処理なら非同期にせず同期処理で待たせましょう。
    その間のループ中に `Application.DoEvents()` を繰り返し呼んでメッセージを処理するといいと思います。

    キャンセル

  • 2018/01/09 13:54

    ご丁寧な解説をいただき、ありがとうございます。
    同期で待たせてApplication.DoEvents()で処理することを検討してみたいと思います。

    最後に1点、今回例にした重い処理の正体は画面を閉じるとき以外にも実行する非同期通信処理メソッドでして、Closingイベント中ではループがありません。
    この場合はどのようにDoEvents()を繰り返し実行させればよいでしょうか?
    (Application.DoEvents();を無限ループ処理するTaskを作ってWhenAny()を使う?)

    キャンセル

  • 2018/01/09 13:58 編集

    その方法が簡単だと思います。
    どの程度重いのかにもよりますが、ShowDialog() でプログレスダイアログを出してその Shown イベントで非同期処理を行い、処理が終わってダイアログが閉じられるのを待つという手もあります。

    キャンセル

  • 2018/01/09 14:03

    ご回答ありがとうございました!
    イベントハンドラおよび非同期処理については引き続き勉強していきたいと思います。

    キャンセル

  • 2018/01/09 14:19

    書くのを忘れていました。無限ループを使う場合には非同期処理が始まらない場合があるので、その時には非同期処理を別スレッドに追い出す必要があります。TaskScheduler を調べてください。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • C#

    6891questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • .NET Framework

    447questions

    .NET Framework は、Microsoft Windowsのオペレーティングシステムのために開発されたソフトウェア開発環境/実行環境です。多くのプログラミング言語をサポートしています。

  • 非同期処理

    105questions

    非同期処理とは一部のコードを別々のスレッドで実行させる手法です。アプリケーションのパフォーマンスを向上させる目的でこの手法を用います。