WPF: 非同期にProcessの終了をチェックする

Question

WPFで非同期にProcessの終了を取りたいです。概要、以下のようなコードです。

Formではなく、WPFだと、上手く動かないように思います。Dispatcherとかが関係しているのでしょうか。不勉強なところ、大変恐縮ですが、ご教示頂けますと幸いです。

//some_project.xaml
<UserControl:ToggleSwitch x:Name="SomeControl" Checked="ToggleSwitch_Checked"/>
<MediaElement x:Name="LoadingIcon"/>

//some_project.xaml.cs
private void LoadingStart(){
    //LoadingIconの開始処理
}

private void LoadingFinish(){
    //LoadingIconの終了処理
}

private void ToggleSwitch_Checked(object sender, RoutedEventArgs e){
    var process = new Process()
    Directory.SetCurrentDirectory(directory_path);
    process.StartInfo.WindowStyle = ProcessWindowStyle.Hidden;
    process.StartInfo.FileName = "cmd";
    process.StartInfo.Arguments = string.Format("/c \"{0} < {1}\"", @"test.exe", test_input_file_path);
    process.EnableRaisingEvents = true;
    process.Exited += (sender, e) => 
    {
        SomeControl.IsChecked = false;
    };
    process.Start();
}

Comments

[Zuishin]

どうなったらうまく動いたことになりますか？

[5REva]

ありがとうございます。
ボタンをクリックすると、あるコントロールのチェックがtrueになり、上記のプロセスが動いて、上記プロセスが完了すると、そのコントロールのチェックがfalseになると、うまく動いた事になります。

[Zuishin]

現在はどうなりますか？

[5REva]

プロセスが終了しても、チェックがtrueのままです

[Zuishin]

SomeControl.IsChecked = false; にブレークポイントを置くとそこで停止しますか？

[5REva]

停止しません。

[Zuishin]

ProcessStartInfo と Process が関連付けられていないようですが、Process のコンストラクタで渡すとどうなりますか？

[5REva]

ありがとうございます。試してみます。

Answer 1

最終的に、Dispatcher.BeginInvokeを介入させる方法で、うまく動きました。こちらが参考になりました。
https://stackoverflow.com/questions/4821265/updating-parent-process-gui-on-receiving-process-exited-event-wpf

some_project.xaml
<UserControl:ToggleSwitch x:Name="SomeControl" Checked="ToggleSwitch_Checked"/>
<MediaElement x:Name="LoadingIcon"/>

some_project.xaml.cs
private void LoadingStart(){
    //LoadingIconの開始処理
}

private void LoadingFinish(){
    //LoadingIconの終了処理
}

private void ToggleSwitch_Checked(object sender, RoutedEventArgs e){
    LoadingStart();

    Directory.SetCurrentDirectory(directory_path);
    Process process = new Process();
    process.StartInfo.WindowStyle = ProcessWindowStyle.Hidden;
    process.StartInfo.FileName = "cmd";
    process.StartInfo.Arguments = string.Format("/c \"{0} < {1}\"", @"test.exe", test_input_file_path);

    process.Exited += process_Exited;
    process.EnableRaisingEvents = true;

    process.Start();
}

private void process_Exited(object sender, EventArgs e){
    this.Dispatcher.BeginInvoke(
        new Action(() => {
                LoadingFinish();
                SomeControl.IsChecked = false;
            }));
}

Answer 2

process.StartInfo.WindowStyle = ProcessWindowStyle.Hidden;

ここでコマンドラインをあえて見えなくしていると思いますが、
表示してみて、cmdプロセスがしっかり終了している事を確認していますか？

Comments

[5REva]

ありがとうございます。表示してみます。
WaitForExitで同期で止めて、SomeControl.IsChecked = falseとすると、ちゃんと動きます。他のメソッドの関係で、ここを非同期でやる必要があるのですが、うまく行きません。

StackOverFlowとかみると、Dispatcherを介しているみたいで、こっちも試したんですが、うまく行かず（正直、私があまりDispatcherをわかっていない気がしています。）
https://stackoverflow.com/questions/4821265/updating-parent-process-gui-on-receiving-process-exited-event-wpf

[takabosoft]

質問欄で「SomeControl.IsChecked = false;」箇所にブレークポイントを貼っても停止しないという回答がありますので、Dispatchar以前の問題かと思われます。他の方の回答（GCなど）の検証もお願いいたします。

[5REva]

私の情報が不足してたように思います。情報を追加し、うまく動いたコードも共有させて頂きました。どうもありがとうございました。

Answer 3

processの参照を残してなくて、GCに回収されちゃっているに一票入れときます。

追記：process又はイベントハンドラのどちらかをフィールドなどに保持しておいてください。

追記その2：↑と書きましたがイベントハンドラを保持するほうが確実ですね。
processの中身を知る必要がないので。

Comments

[5REva]

どうもありがとうございます。確認してみます。

StackOverFlowとかみると、Dispatcherを介しているみたいで、こっちも試したんですが、うまく行かず（正直、私があまりDispatcherをわかっていない気がしています。）
https://stackoverflow.com/questions/4821265/updating-parent-process-gui-on-receiving-process-exited-event-wpf

[hihijiji]

別スレッドからコントロールを操作すると例外が発生します。例外が出ないならそれ以前の問題です。
イベントを発生するオブジェクトとそのイベントハンドラの両方の参照を捨てているならいつかはGCに回収されます。
Windows.Formsで同じようにして動いていたなら、それは偶々回収されずに動いていただけです。

[5REva]

私の情報が不足してたように思います。情報を追加し、うまく動いたコードも共有させて頂きました。どうもありがとうございました。