WPF buttonをクリックしたら非同期でTextBoxへ書き込みを行いたいです。

Question

現在、WPFにて画面にButtonとTextBoxを配置しております。

Buttonをクリックしましたら、TextBoxへの書き込み処理を非同期で行いたいのですが、様々な方法を調べて試してみても画面が固まってしまい非同期での処理が成功しておりません。

private async void AppendEndButton_Click(object sender, RoutedEventArgs e)
        {
            await AppendEnd();
        }

        public async Task AppendEnd()
        {
            await Task.Run(() => 
            {
                this.Dispatcher.Invoke(() =>
                {
                    for (int i = 0; i < BeforeText.LineCount; i++)
                    {
                        AfterText.Text += BeforeText.GetLineText(i) + "　" + AppendText.Text + Environment.NewLine;
                    }
                });
            });
            
        }

どこが間違えているのか、ご教授頂けますと幸いです。

Answer 1

this.Dispatcher.Invokeの引数にデリゲートを渡すことで、UIスレッド（WPFのコントロールの状態を操作するスレッド）に処理を委譲できます。

画面が固まってしまい非同期での処理が成功しておりません

UIスレッドにはコントロールにアクセスする以外の処理は行わせないでください。
forでループしている部分や文字列を結合する処理はUIスレッドで行わず、下記のようにAfterText.Text の値を上書きするだけに留めてください。

        public async Task AppendEnd ()
        {
            await Task.Run (() => {
                var text = string.Empty;
                for (int i = 0; i < BeforeText.LineCount; i++) {
                    text += BeforeText.GetLineText (i) + "　" + AppendText.Text + Environment.NewLine;
                }
                this.Dispatcher.Invoke (() => {
                    AfterText.Text = text;
                });
            });
        }

Comments

[fama]

アドバイスを頂きましてありがとうございました。
こちらの方法に加え、　text += BeforeText.GetLineText (i) + "　" + AppendText.Text + Environment.NewLine;　を　this.Dispatcher.Invoke (() => {});　で囲む事で解決が出来ました。

非常に助かりました。ありがとうございます。

[BluOxy]

確かにBeforeTextを参照している部分もDispatcherで囲わないとInvalidOperationExceptionが発生しそうです。

その方法で解決したのであれば恐らく何度もAfterText.Textを上書きする処理が重かったのかもしれません。

[BluOxy]

いや、何度も文字列を結合する処理が重かったのかもしれません。余談にはなりますが、StringBuilderを使うと文字列の結合が+=よりも高速になります。