別のクラスからRichTextBoxに文字列を追加する方法が知りたい。

Question

実現したいこと

RichTextBoxの表示位置やサイズなどを作成したクラスとは別のクラスで、でRichTextBoxのUIにAppendし、追加した文字列をUIに表示させたいです。

何故別のクラスである必要があるのか、についてですが、別のクラスからではないといけないと思い込んでいるからで、本当は、別のクラスからでなくとも良いのかもしれないです。
ただ、何ができて何ができないのかなど、C＃・プログラミング言語に明るくないので、ひとまず、別のクラスから文字列を追加する方法を知りたいです。

発生している問題・分からないこと

public void B()内のDebug.WriteLineの結果は、test:System.Windows.Forms.RichTextBox, Text: void BHello, World!のようになっています。

しかし、UIには文字列が追加されていません。

該当のソースコード

//Fomr1Designer.cs
namespace NewApp
{
    partial class Form1
    {
        private System.ComponentModel.IContainer components = null;
</param>
        protected override void Dispose(bool disposing)
        {
            if (disposing && (components != null))
            {
                components.Dispose();
            }
            base.Dispose(disposing);
        }
        #region Windows Form Designer generated code
        private void InitializeComponent()
        {
            AutoScaleDimensions = new SizeF(7F, 15F);
            AutoScaleMode = AutoScaleMode.Font;
            ClientSize = new Size(800, 450);
            Name = "Form1";
            Text = "Form1";
            ResumeLayout(false);
            this.classA = new  NewApp.ClassA();
            this.Controls.Add(this.classA);
        }

        #endregion
        private NewApp.ClassA classA ;
    }
}

//Form1.cs
using System.Diagnostics;
using System.Windows.Forms;
using static System.Net.Mime.MediaTypeNames;
namespace NewApp
{
    public partial class Form1 : Form
    {
        //private Form2 _form2;
        private ClassA _classA;
        //public Form1(Form2 form2)
        public Form1(ClassA classA)
        {
            _classA = classA;
            InitializeComponent();
            B();
        }
        public void B()
        {
            RichTextBox richTextBox = _classA.A();
            richTextBox.AppendText("void B");
            _classA.A().AppendText("Hello, World!");
            _classA.A().Refresh();
            Debug.WriteLine("test" + _classA.A());
        }
    }
}

//ClassA.Designer.cs
using System;
using System.Windows.Forms;
namespace NewApp
{
    partial class ClassA
    {
        private System.ComponentModel.IContainer components = null;

        protected override void Dispose(bool disposing)
        {
            if (disposing && (components != null))
            {
                components.Dispose();
            }
            base.Dispose(disposing);
        }
        #region Windows Form Designer generated code
        private void InitializeComponent()
        {
            richTextBox = new RichTextBox();
            TabPagerich = new TabPage();
            tabControl1 = new TabControl();
            button1 = new Button();
            TabPagerich.SuspendLayout();
            tabControl1.SuspendLayout();
            SuspendLayout();
            // 
            // richTextBox
            // 
            richTextBox.Dock = DockStyle.Fill;
            richTextBox.Location = new Point(3, 3);
            richTextBox.Name = "richTextBox";
            richTextBox.Size = new Size(386, 266);
            richTextBox.TabIndex = 0;
            richTextBox.Text = "";
            // 
            // TabPagerich
            // 
            TabPagerich.Controls.Add(richTextBox);
            TabPagerich.Location = new Point(4, 24);
            TabPagerich.Name = "TabPagerich";
            TabPagerich.Padding = new Padding(3);
            TabPagerich.Size = new Size(392, 272);
            TabPagerich.TabIndex = 0;
            TabPagerich.Text = "RichText";
            TabPagerich.UseVisualStyleBackColor = true;
            // 
            // tabControl1
            // 
            tabControl1.Controls.Add(TabPagerich);
            tabControl1.Location = new Point(12, 12);
            tabControl1.Name = "tabControl1";
            tabControl1.SelectedIndex = 0;
            tabControl1.Size = new Size(400, 300);
            tabControl1.TabIndex = 0;
            // 
            // ClassA
            // 
            AutoScaleDimensions = new SizeF(7F, 15F);
            AutoScaleMode = AutoScaleMode.Font;
            Controls.Add(button1);
            Controls.Add(tabControl1);
            Margin = new Padding(4, 3, 4, 3);
            Name = "ClassA";
            Size = new Size(495, 372);
            TabPagerich.ResumeLayout(false);
            tabControl1.ResumeLayout(false);
            ResumeLayout(false);
        }
        private System.Windows.Forms.RichTextBox richTextBox;
        private System.Windows.Forms.TabPage TabPagerich;
        private System.Windows.Forms.TabControl tabControl1;
        #endregion
        private Button button1;
    }
}

//ClassA.cs
using System;
using System.Diagnostics;
using System.Windows.Forms;
using static System.Net.Mime.MediaTypeNames;

namespace NewApp
{
    public partial class ClassA : UserControl
    {
        public ClassA()
        {
            InitializeComponent();
        }
        public RichTextBox A()
        {
            return richTextBox;
        }
    }
}

//Program.cs
namespace NewApp
{
    internal static class Program
    {
        [STAThread]
        static void Main()
        {
            ApplicationConfiguration.Initialize();
            ClassA classA = new ClassA();
            Application.Run(new Form1(classA));
            Application.Run();
        }
    }
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

stackoverflowでCan't add to a RichTextBox from another class?というページや、そのほか慣例ページを見ました。
自分に起きている問題と同じ、或いは似ていると思うのですが、コードの詳細が簡略化されており、こちら側で応用することが叶いませんでした。
今のところ漠然と、インスタンスの作成の仕方や、処理の順番のによる問題なのかなと思っています。

ChatGPT3.5には、私の伝え方がなっていないためか、見当違いと思われる、たくさんの返答をいただきました。

補足

プログラミング言語自体全くの初心者です。
インスタンスに関しての理解は、用意した構造体を複製したモノ、という程度の理解しかありません。

上記のコードは実際に扱っているコードを、今回の問題に関係ありそうな部分を取り出して、できるだけ同じ構造になるように単純化したものです。

提示した、上記コードはVisual Studio2022を使用していますが、実際には
Visual Studio2019、.NET Framework ver 4.0を使用しています。

実際に扱っているコードはオープンソースのものです。
深い理由はないですが、何を扱っているかのURLは提示しないつもりです。複雑なことに対して、私が混乱しそうなので。

実際の環境では諸事情で、デザイナーズファイルを使用して、コードを追加するということが出来なくなってしまいました。

Comments

[YAmaGNZ]

Mainを変更されているようですが何か目的があって変更されているのでしょうか？
通常であればこのような変更は行わないので目的があって変更しているのであれば教えていただきたいと思います。
なんとなくこの変更のせいでインスタンスのことも理解できていない状態でこんがらがっているように思えます。

[hatte]

コメントありがとうございます。
Mainを変更している目的はありません。
提示したコードに至るまでに紆余曲折があり、ChatGptと二人三脚でやっていたので、その過程で変更を加えていました。
また、なぜそのような変更をしたのかやコードに関してですが、理屈は理解できていません。一先ず、おまじないを唱えるような認識です。

上記のコードであれば、本来どうなっているべきなのでしょうか？また、変更に目的がない場合、Main関数の修正が、私の抱えている問題の解決につながりますでしょうか？

また、「なんとなくこの変更のせいでインスタンスのことも理解できていない状態でこんがらがっているように思えます」とのことですが、もし可能であれば、その辺りのこと、何故そう直感？されたのか、お聞きしたいです。

期待通りには動いたものの、何故、最初のコードではダメだったのか、修正したらなぜうまくいったのかが、理解にはまだ及んでいないので（勿論、時間をかけて勉強を試みるつもりではいます）、ぜひお聞きしたいのです。

よろしくお願いします。

[hatte]

すみません、コメントをする場所が適切ではなかったかもしれません。

[fana]

> 提示したコードに至るまでに紆余曲折があり、ChatGptと二人三脚でやっていたので、その過程で変更を加えていました。
> また、なぜそのような変更をしたのかやコードに関してですが、理屈は理解できていません。一先ず、おまじないを唱えるような認識です。

個人的な感覚でしかない話で恐縮ですが……
なんというか，ひとまず ChatGpt とやらとは早々に手を切り，最低限の基礎的な部分に関してはより適切な手段を用いて習得するのが良いのではなかろうか，とか思いました．

Answer 1

何も目的がないのであればVisualStudioにて自動生成される部分に変更を加えるのはお勧めしません。
なのでProgram.csやDesigner.csを変更するべきではありません。

提示されている例ですとClassAというユーザーコントロールを作成し、フォームデザイナーにてForm1に追加するだけでいいはずです。
そしてBというメソッドを作成し

        public void B()
        {
            RichTextBox richTextBox = classA.A();
            richTextBox.AppendText("void B");
            classA.A().AppendText("Hello, World!");
            classA.A().Refresh();
            Debug.WriteLine("test" + classA.A());
        }

といった感じに記述するだけでいいはずです。（デザイナーにて追加したコントロールのNameプロパティをclassAとした場合）

またコードにてコントロールを追加する場合は

        public Form1(ClassA classA)
        {
            InitializeComponent();
            _classA = classA;
            this.Controls.Add(_classA);
            B();
        }

という感じでthis.Controls.Add(_classA)とコンテナ（フォームやパネル等）に対して追加しないと表示されません。

Comments

[hatte]

ご回答ありがとうございます。
教えられた通りのコードで解決いたしました。

デザイナーFileなどを変えるべきではないというのは、Visual Stduioにも記されていたり、ほかの質問者の方々や、ChatGPTにも言われました。私の認識では、デザイナーファイルを変更した方がコードを変更するよりも「楽」という程度の認識だったのですが、どうも、デザイナーファイルを変更すると「苦」になるので、やめた方が良いという認識の方が近いのでしょうか。

もう一つお聞きしたいのですが、Program.csの中身ですが、この構成であれば、本来どうなっていたのでしょうか？初期状態を覚えていないのです。
ApplicationConfiguration.Initialize();
ClassA classA = new ClassA();
Application.Run(new Form1(classA));
この３行でしょうか？

[YAmaGNZ]

フォームを新規に作成した場合、コンストラクタに引数はありません。
なので
Application.Run(new Form1());
となっているはずです。
またClassAを作成するコードも存在しないはずです。

そのあたりは別に新規でプロジェクトを作成すれば確認できます。

[YAmaGNZ]

また自動生成されるものを変更すべきではないというのは、フォームデザイナーなど自動生成されたコードを元に表示すべきものを表示しているので下手に変更するとフォームデザイナー自体が使えなくなったりします。
知識があってもデザイナーファイルを変更することはまずありません。コードで変更するよりデザイナーで変更したほうが楽ですし、もし大量のコントロールを作成したいというのであればコードで生成することになりそのコードはDesigner.csを変更するのではなく通常のコードに記載することになります。

Answer 2

ClassAのインスタンスが２つできています（newがあるところ）

Program.csでClassA classA = new ClassA();がひとつ目、
Form1.Designer.csでthis.classA = new NewApp.ClassA();がふたつ目です。

見えているものはいじった（AppendTextした）ものではないということです。

普通はデザイナで置いたものをいじればいいんですが、わざわざProgram.csで渡しているということはそれを使うべきなのでしょう。

Controls.Add(_classA);を追加してください。
デザイナで置いたclassAは削除してください。

using System.Diagnostics;
using System.Windows.Forms;


namespace Qr9k8rk196xmcbr
{
    public partial class Form1 : Form
    {
        private ClassA _classA;

        public Form1(ClassA classA)
        {
            //              ↑のclassAはFomr1.Designer.csにある「private NewApp.ClassA classA;」
            // とは同名だが別の変数（ややこしいのでForm1(ClassA c)とかに変えましょう）
            _classA = classA;

            // Program.csから渡されたclassA（_classA）はAddされていない。
            // Addすれば予想通り出る（この場合C()は不要だしデザイナで置いたclassAも不要）
            Controls.Add(_classA);

            InitializeComponent();
            B();

            // デザイナで置いたclassAを使用するならB()やProgram.csの変更も不要だが、
            // 何かしらの理由があってそうなっているのだろうから、おそらくこれではまずいんでしょう。
            //C(); 
        }

        // Program.csから渡されたインスタンスを使用する
        public void B()
        {
            RichTextBox richTextBox = _classA.A();
            richTextBox.AppendText("void B");
            _classA.A().AppendText("Hello, World!");
            //_classA.A().Refresh();
            Debug.WriteLine("test" + _classA.A());
        }

        // デザイナで置いたインスタンスを使用する
        //public void C()
        //{
        //    RichTextBox richTextBox = classA.A();
        //    richTextBox.AppendText("void C");
        //    classA.A().AppendText("Hello, World!");
        //    //classA.A().Refresh();
        //    Debug.WriteLine("test" + classA.A());
        //}
    }
}

using System.Drawing;


namespace Qr9k8rk196xmcbr
{
    partial class Form1
    {
        private System.ComponentModel.IContainer components = null;

        protected override void Dispose(bool disposing)
        {
            if (disposing && (components != null))
            {
                components.Dispose();
            }
            base.Dispose(disposing);
        }
        #region Windows Form Designer generated code
        private void InitializeComponent()
        {
            AutoScaleDimensions = new SizeF(7F, 15F);
            AutoScaleMode = System.Windows.Forms.AutoScaleMode.Font;
            ClientSize = new Size(800, 450);
            Name = "Form1";
            Text = "Form1";
            ResumeLayout(false);
            //this.classA = new Qr9k8rk196xmcbr.ClassA();
            //this.Controls.Add(this.classA);
        }

        #endregion
        //private Qr9k8rk196xmcbr.ClassA classA;
    }
}

Comments

[TN8001]

> 実際の環境では諸事情で、デザイナーズファイルを使用して、コードを追加するということが出来なくなってしまいました。

いじってしまった結果デザイナが開けないならわかりますが、もう一度取得しなおせば開けるようになりませんかね？
この先もデザイナなしでやるとなると、かなりつらそうですが...

[setoppu]

ユーザーコントロールとかは一度ビルドされたあとでないとフォームデザイナーがエラーになったような。

今回の場合、ClassAのrichTextBoxはPrivateなんだからClssAに操作用のPublicメソッドを用意するべきなんじゃないかなぁ。とか思いますが。
コントロールをPublicメソッドの戻り値で返すならPrivateにする意味がなさそう。

[TN8001]

@setoppuさん
いやまあ思うところはあるんですが、最小限の変更でないと質問者さんが混乱するようなので^^;

閲覧者向け
関連質問
[.NET Formアプリで、文字列をAppendTextメソッドを使って、RichTextBoxに追加し、UI上に文字列に表示させたい。](https://teratail.com/questions/9r9sq3i79y5jib)

[hatte]

コメント、ご回答ありがとうございます。また、前回に引き続き助言をくださったことに深く感謝します。
こちらのやり方でも、期待通りに動作しました。
何故、自分のコードではダメだったのか、なぜ教えていただいたコードでは動作したのかを暫く、見つめなおしたいと思います。

publicやprivateの設定に関してですが、外部からアクセスすることを許可するかしないかという意味という認識をしています。
回顧すると、「ただ、なんとなく外部からのアクセスを許可すると、なんとなく無知な自分にはたたりそう」という理由で、privateにしたのだと思います。なので、深い理由はないです。きちんと使い分けないと長期的にはだめですよね。

私はプログラミングを必要に駆られたときにしか使う気がないので、深く勉強する気になれないというところが、なんとも...。消火器みたいなものですね。使う機会がなければ、それに越したことはない、みたいな。

[hatte]

>わざわざProgram.csで渡しているということはそれを使うべきなのでしょう

いえ、そうすべくして、そうしたというより、ChatGPTの言うことを黙ってコピーアンドペーストした結果そうなったのであって、元々のオープンソースがそうなっていた、というわけではないのです。

Program.csの記述がおかしい、普通ではやらないようなことをしているのでしょうか？

[TN8001]

> こちらのやり方でも、期待通りに動作しました。

Program.csをいじっているのは、元コード（OSS）がそういう作りになっていたからだと思ったんですがそうではないんでしょうか？（ChatGPTがそういう回答をしたってこと？？）

そういうことであれば前回の回答のように、ほぼデザイナだけの操作でいいのですが...
「OSSのほうのプロジェクトでデザイナが開けないので、そういわれても困る」ということなら、開けるようにするのが一番近道な気がします。

> publicやprivateの設定に関してですが、外部からアクセスすることを許可するかしないかという意味という認識をしています。

これはsetoppuさんのコメントを受けての反応でしょうか？

> private System.Windows.Forms.RichTextBox richTextBox;
> public RichTextBox A() { return richTextBox; }

これらがあんまり意味をなしてないという指摘ですね。
前回の回答でも書いた通りModifiersをpublicにすると、Designer.csは、
public System.Windows.Forms.RichTextBox richTextBox;
と自動で変わります。

呼び出し側はメソッドを使わなくても、
_classA.richTextBox.AppendText("void B");
と直接アクセスできるようになります。

> 私はプログラミングを必要に駆られたときにしか使う気がないので、深く勉強する気になれないというところが、なんとも...。消火器みたいなものですね。使う機会がなければ、それに越したことはない、みたいな。

どうなんでしょうね。

私がプログラミングを始めたきっかけは昔のことなのであまり覚えていない（なんとなく面白そうだったから？）ですが、実際にプログラムを書ききったのは必要に迫られてなのは間違いないです。

C#で初めて作ったプログラムは今でも毎日使っています^^
結局使う当てのないプログラムは途中で飽きてしまうんですよね^^;
まあそこまでに欲しいけど作れなかったプログラムも結構あったはずですがｗ

少なくとも必要ないのに勉強するよりはずっといいですし、Windowsアプリに関してはC#+Visual Studioは最強の環境です。
がんばってください^^

[TN8001]

書いている間に入違ってしまいました^^;

> いえ、そうすべくして、そうしたというより、ChatGPTの言うことを黙ってコピーアンドペーストした結果そうなったのであって、元々のオープンソースがそうなっていた、というわけではないのです。

へーそんな謎コードを提案するんですね。

> Program.csの記述がおかしい、普通ではやらないようなことをしているのでしょうか？

普通はまずそういうことはしないと思います。
そういうことが必要なことはあるかもしれませんが、提示コードではこれで事足りる話です（さらに言えばデザイナで設置すればいいだけの話）
public partial class Form1 : Form
{
private ClassA _classA;
public Form1()
{
InitializeComponent();

_classA = new ClassA();
Controls.Add(_classA);
B();
}