c# textboxに出力できない

Question

using System;
using System.Windows.Forms;

namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        private void button1_Click(object sender, EventArgs e)
        {
            //Processオブジェクトを作成
            System.Diagnostics.Process p = new System.Diagnostics.Process();

            //出力をストリームに書き込むようにする
            p.StartInfo.UseShellExecute = false;
            p.StartInfo.RedirectStandardOutput = true;
            //OutputDataReceivedイベントハンドラを追加
            p.OutputDataReceived += p_OutputDataReceived;

            p.StartInfo.FileName =
                System.Environment.GetEnvironmentVariable("ComSpec");
            p.StartInfo.RedirectStandardInput = false;
            p.StartInfo.CreateNoWindow = true;
            p.StartInfo.Arguments = @"/c コマンド";

            //起動
            p.Start();

            //非同期で出力の読み取りを開始
            p.BeginOutputReadLine();

            p.WaitForExit();
            p.Close();

            Console.ReadLine();
        }

        //OutputDataReceivedイベントハンドラ
        //行が出力されるたびに呼び出される
        static void p_OutputDataReceived(object sender, System.Diagnostics.DataReceivedEventArgs e)
        {
            //出力された文字列を表示する
            Console.WriteLine(e.Data);
            textbox.Text = (e.Data);
        }
    }
}

↑のコードをコンパイルすると"textbox.Text = (e.Data);"のところにエラーが出ます

エラー内容

静的でないフィールド、メソッド、またはプロパティ 'WindowsFormsApplication1.Form1.textbox' で、オブジェクト参照が必要です

参考にしたコード(エラーコード&Visual Studio 2013が生成したコード以外コピペです)一応理解しているつもりです
http://dobon.net/vb/dotnet/process/standardoutput.html#section2
助けてくださいよろしくお願いします(_ _)

Answer 1

staticメソッドで使用しているからですね。

staticメソッド内で参照できるのはstaticなメンバやプロパティに限られます。

ざっと見た感じp_OutputDataReceivedはprivateで問題なさそうですが、
staticにしてある理由があれば追記してください。

追記

今回はFormアプリなのでDispatcherクラスではなく、ContorolクラスのInvoke,BeginInvokeを使用します。
間違った情報失礼しました。

まず、C#がマルチスレッドで動作することはご存知と思います。マルチスレッド
Formアプリでは、アプリを起動すると必ずUIスレッドというものがただ1つだけ起動します。
UIスレッドは、ユーザからの入力（TextBoxへの入力やButtonクリックの処理など）の受付や出力（TextBoxへのTextの設定など）を制御するスレッドです。
逆にいうと、UIスレッド以外からUIの更新をすることはできません。

今回の場合、画面入出力を制御するUIスレッドとは別のスレッドでDOMコマンドの処理を行うことで、UIがフリーズしないようにしているわけです。

ただ、
p.WaitForExit();
とあるように、プロセスが終了するまでUIスレッドをロックしたまま待機していますので、実はフリーズしているんですが。。。

では、どのようにUIスレッドとは別のスレッドからUIの更新を行うかといいますと、
ここで、Invoke,BeginInvokeメソッドが登場するわけです。

先述した通り、実はUIスレッドはロックされているので、今回はInvoke（同期）は使うことはできません。
BeginInvoke(非同期)で説明します。

BeginInvokeメソッドを理解するうえで、delegate(デリゲート）の理解が必須になってきます。処理を委譲するdelegate
こちらについても、調べてみてください。

さて、BeginInvokeメソッドの使い方ですが、BeginInvokeメソッドにUIスレッドで実行したい処理のdelegateを引数に渡します。
今回の場合、「textBox.Text = e.Data」をUIスレッド上で実行してもらいたいので、以下のようにします。
(目的の動作となるよう、少し処理を変えています。TextBoxをMultiLineにして、十分に大きくしておいてください。dirコマンドの出力結果をTextBoxに出力します。）

public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
    }

    private void Form1_Load(object sender, EventArgs e)
    {

    }

    private void button1_Click(object sender, EventArgs e)
    {
        //Processオブジェクトを作成
        System.Diagnostics.Process p = new System.Diagnostics.Process();

        //出力をストリームに書き込むようにする
        p.StartInfo.UseShellExecute = false;
        p.StartInfo.RedirectStandardOutput = true;
        //OutputDataReceivedイベントハンドラを追加
        p.OutputDataReceived += p_OutputDataReceived;

        p.StartInfo.FileName =
            System.Environment.GetEnvironmentVariable("ComSpec");
        p.StartInfo.RedirectStandardInput = false;
        p.StartInfo.CreateNoWindow = true;
        p.StartInfo.Arguments = @"/c dir";

        //起動
        p.Start();

        //非同期で出力の読み取りを開始
        p.BeginOutputReadLine();

        p.WaitForExit();
        p.Close();

    }

    //OutputDataReceivedイベントハンドラ
    //行が出力されるたびに呼び出される
    private void p_OutputDataReceived(object sender, System.Diagnostics.DataReceivedEventArgs e)
    {
        //出力された文字列を表示する
        Console.WriteLine(e.Data);
        
        // Delegateを作成
        Action act = () =>
            {
                if (string.IsNullOrEmpty(e.Data) == false)
                {
                    textbox.AppendText(e.Data);
                }

                textbox.AppendText(Environment.NewLine);
            };

        BeginInvoke(act);
    }
}

スレッドやデリゲートなんかはイメージを掴むまでが大変です。
めげずに頑張りましょう。

Comments

[wakuwaku]

ちなみに今回の場合、 p_OutputDataReceivedはUIスレッドとは別のスレッドで実行されますので、
直接textbox.Text = e.Dataのように値を設定することはできません。

DispatcerクラスのInvokeメソッドなどでUIスレッドに処理を委譲する必要がありますので注意してください。

[come25136]

staticにしている理由は質問に乗せているURLから取ってきたものだからです
staticからprivateにしたところコメントに書いてくださっている通りエラーが出ました
コメントに書いてくださっている言葉はまだ僕には難しいので解決方法をできるだけ簡単に教えて下さい(_ _)

[wakuwaku]

追記しました。

余談ですが、サンプルはコンソールアプリケーション向けのサンプルです。
今回はFormアプリケーションなので、Console.WriteLineやConsole.Readは使いません。

例えばVisual Studioの出力うウィンドウにデバック出力する場合なんかは、
Syatem.Dignostics.Debug.WriteLine
を使います。

[come25136]

わかりやすい回答ありがとうございます(_ _)
おかげさまで無事解決出来ました(^o^)／

[come25136]

解決したと思っていたのですが実行時間が長いコマンドを実行する予定だったのを忘れていましてUIがフリーズしてしまいましたもう一つ質問をしたので回答していただけないでしょうか？よろしくお願いします(_ _)
https://teratail.com/questions/13120

※追伸　解決しました

Answer 2

実行時エラーでなく、コンパイルエラーのように見えます。

Form1にtextboxという名前のコントロールを配置してますか？

Comments

[come25136]

はい、textboxは配置してあります
あとお書きになっている通り、実行時のエラーではなくコンパイルエラーです。