c# シリアルポートからの受信データを複数のtextBoxに追加する

Question

ご助言をいただきたくよろしくお願いします
serial通信で受信したデータを表示するGuiを作っています。
下記のようにして受信するところまではできました。　
受信データは64個の温度データです。
これをtextBox1-64にそれぞれ格納するようにしたいのですが、
どのようにしたらいいのかわからず困っています。
dataに格納された文字列型のデータをどのように処理したらいいのかご教示いただいたいです。
for文を使ってデータをそれぞれtextBox1~64へ格納したいです。
そこから何をしていったらいいのかわからず困っております。

dataに入っている中身は”3
23
22
22
22
23
…というように数字（温度）
が64個繋がります。最初の3は固定値です。

```ここに言語を入力
private void SerialPort1_DataReceived_1(object sender, SerialDataReceivedEventArgs e)
//! シリアルポートをオープンしていない場合、処理を行わない.
            if (serialPort1.IsOpen == false)
            {
                return;
            }
            else
            {
                string data = string.Empty;

try
                {
                    //受信データの読み取り
                    data = serialPort1.ReadExisting()
      　       }
                catch (Exception ex)
                {
                    MessageBox.Show(ex.Message);
                }

AddRecievedDataDelegate add = new AddRecievedDataDelegate(AddRecievedData);
                this.Invoke(add, data);
       }        
}
 private void AddRecievedData(string data){
        textBox1.Text += data;
}

```
**ご回答をいただいたもの参考に以下のように変更してみました。**
this.Invoke(add, data);のところで
System.NullReferenceException: 'オブジェクト参照がオブジェクト インスタンスに設定されていません。とエラーが出てしまい、今までこのようなことがなかったのですが、どこを修正するとこのエラーが解消できるか悩んでおります。
ご教示いただきたくお願いします。

```ここに言語を入力
 public partial class Form1 : Form
    {
        private TextBox[] txtBoxes;

public Form1()
        {
            InitializeComponent();

int width = 100;
            int counter = 0;

for (var i = 0; i < 8; i++)
            {

for(var j =0; j < 8; j++)
                {
                    
                    new TextBox
                    {
                        Name = $"textBox{counter + 1}",
                        Location = new Point(j * width + 70, i * 28 + 30),
                        Size = new Size(20,5),
                        Parent = this,
                    };
                    new Label
                    {
                        Name = $"testPIXCEL{i + 1}",
                        Text = $"PIXECL{counter+ 1}",
                        Location = new Point(j * width + 6, i * 28 + 30),
                        Size = new Size(60, 15),
                        Parent = this,
                    };
                    counter++;
                    txtBoxes = new TextBox[counter];
                                
                }

}

private void Form1_Load(object sender, EventArgs e)
        {
            string[] ports = SerialPort.GetPortNames();

foreach (string port in ports)
            {
                comboBox1.Items.Add(port);
                //確認できるCOMポートを取得して表示する。
            }

if (comboBox1.Items.Count > 0)
                comboBox1.SelectedIndex = 0;

}
        private delegate void AddRecievedDataDelegate(string data);
        private void SerialPort1_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e)
        {
            //! シリアルポートをオープンしていない場合、処理を行わない.
            if (serialPort1.IsOpen == false)
            {
                return;
            }
            else
            {
                string data = string.Empty;

try
                {
                    //受信データの読み取り

data = serialPort1.ReadExisting();
                    
                }
                catch (Exception ex)
                {
                    MessageBox.Show(ex.Message);
                }
                AddRecievedDataDelegate add = new AddRecievedDataDelegate(AddRecievedData);
                this.Invoke(add, data);
            }
        }

private void AddRecievedData(string data)
        {
            string[] stringdata = new string[] { "
" };
            string[] disdata = data.Split(stringdata, StringSplitOptions.None);
            for(var i =0; i< txtBoxes.Length; i++) 
            {
                if(i < disdata.Length) 
                {
                    txtBoxes[i].Text = disdata[i];
                }
                else 
                {
                    txtBoxes[i].Text = "---";                
                }
            }
        }

private void button1_Click(object sender, EventArgs e)
        {
            if (serialPort1.IsOpen == true)
            {
                //! シリアルポートをクローズする.
                serialPort1.Close();

//! ボタンの表示を[切断]から[接続]に変える.
                button1.Text = "接続";
            }
            else
            {
                //! オープンするシリアルポートをコンボボックスから取得
                serialPort1.PortName = comboBox1.SelectedItem.ToString();

serialPort1.BaudRate = 115200;
                serialPort1.Parity = Parity.None;
                serialPort1.DataBits = 8;
                serialPort1.StopBits = StopBits.One;
                serialPort1.Handshake = Handshake.None;
                serialPort1.Open();

//! ボタンの表示を[切断]から[接続]に変える.
                button1.Text = "切断";
            }

}

}
}

コード
```

Accepted Answer

「多数の`TextBox`に値をどう入れたらいいのか」って話ですよね？よく↓みたいなことがしたい（Unityで多い）って質問があります（もちろんできませんよ^^; ```cs for(int i = 1; i <= 64; i++) { ("txtBox" + i).Text = "hogehoge"; } ``` 「配列かなんかを使いましょう」って回答になるわけですが、例がないとイメージがわかないでしょうからこんな雰囲気という例です。６４個はだいぶ多いので作成もforでやりたいところですが、もう置いてあるのでしょうから探してくる実装です。 .NET 6.0です^^ ```cs using System; using System.Drawing; using System.Linq; using System.Windows.Forms; namespace Qiiyta0b542pggh { public partial class Form1 : Form { private readonly TextBox[] txtBoxes; private readonly System.Timers.Timer timer; public Form1() { InitializeComponent(); // ６４個は面倒なので６個（デザイナで置いている場合はこのforは不要） for (var i = 0; i < 6; i++) { // 作成もforでやっていれば、わざわざFindで探す必要もないのですが... // txtBoxes[i] = new TextBox new TextBox { Name = $"textBox{i + 1}", Location = new Point(12, i * 37 + 12), Parent = this, }; } // 扱いやすいようにTextBoxの配列を用意 txtBoxes = new TextBox[6]; for (var i = 0; i < 6; i++) { // textBox1～6を探して配列に格納（見つからなかったら当然エラー） txtBoxes[i] = (TextBox)Controls.Find($"textBox{i + 1}", true).First(); } // SerialPortは大変なのでタイマーで代用（Timer_Elapsedも別スレッド） timer = new System.Timers.Timer(1000); timer.Elapsed += Timer_Elapsed; timer.Start(); } private void Timer_Elapsed(object? sender, System.Timers.ElapsedEventArgs e) { //string data = "3 23 22 22 22 23 "; // 特に意味はない（てきとうなデータをでっちあげてるだけ） string data = string.Join(" ", DateTime.Now.Ticks.ToString().ToCharArray().Reverse().Select(x => $"{x}")); // 改行で分割 string[] split = data.Split(" "); // ＵＩスレッドで実行 Invoke(() => { for (var i = 0; i < txtBoxes.Length; i++) { // データが足りない時があるのかは知りませんが、念のため if (i < split.Length) { txtBoxes[i].Text = split[i]; } else { txtBoxes[i].Text = ""; } } }); } } } ``` --- `AddRecievedDataDelegate`って元ネタどこなんですかね？もちろん当時はそう書くしかなかったんでしょうけど、いまだに「いつの時代の記事！？」みたいなのばっかりで驚きました（`Invoke`のところの話）コピペコピペで増殖したにしても、やけに多い気がします（本当に古い記事も多いですが^^; [C# SerialPort DataReceived - Google 検索](https://www.google.co.jp/search?q=C%23+SerialPort+DataReceived) --- 編集の通知が来なくて気が付くのが遅れました。 ```cs counter++; txtBoxes = new TextBox[counter]; ``` ここは明らかにおかしいです。 ```cs txtBoxes = new TextBox[1]; txtBoxes = new TextBox[2]; // ・・・ ``` と箱（配列）だけ用意して中身(`TextBox`)を入れていませんし、何度も無駄に箱だけ作っています。 ```cs public Form1() { InitializeComponent(); int width = 100; int counter = 0; txtBoxes = new TextBox[8 * 8]; for (var i = 0; i < 8; i++) { for (var j = 0; j < 8; j++) { txtBoxes[counter] = new TextBox { Name = $"textBox{counter + 1}", Location = new Point(j * width + 70, i * 28 + 30), Size = new Size(20, 5), Parent = this, }; new Label { Name = $"testPIXCEL{i + 1}", Text = $"PIXECL{counter + 1}", Location = new Point(j * width + 6, i * 28 + 30), Size = new Size(60, 15), Parent = this, }; counter++; } } } ``` こんな感じで箱は１回だけ64個で作り、中身を`counter`番目に入れます。 ```cs AddRecievedDataDelegate add = new AddRecievedDataDelegate(AddRecievedData); this.Invoke(add, data); ``` ここも、 ```cs this.Invoke(AddRecievedData, data); ``` こうでいいはずです。

Answer

> dataに入っている中身は”3
23
22
22
22
23
…というように数字（温度）
が64個繋がります。最初の3は固定値です。

String.Split メソッドを使ってはいかがですか？　以下のような感じ。取得できた String の配列の各要素を TextBox.Text に代入していけば良さそうです。

![イメージ説明](https://ddjkaamml8q8x.cloudfront.net/questions/2022-04-22/43b344b2-5853-4a8c-8159-17399e5055f0.jpeg)

---

**【追記】**

> this.Invoke(add, data);のところで System.NullReferenceException: 'オブジェクト参照がオブジェクト インスタンスに設定されていません。とエラーが出てしまい、今までこのようなことがなかったのですが、どこを修正するとこのエラーが解消できるか悩んでおります。

質問のコメントでも書きましたけど、デバッグはしたのですかね？　また無言ですが、デバッガの使い方が分かりませんとか、やったけど分かりませんでしたとかのフィードバックぐらいは返したらいかが？

注文を付けるばかりでは何なので、ヒントを書いておきます。Form1 のコンストラクタのコードをよく見てください。入れ物 txtBoxes を作って new TextBox で入れるものも作ったのですが、入れ物の中に作ったものを入れてないのが問題でしょう。（それ以外にも見直すべき所は多々ありそうですが）

上のヒントで解決しなければ、表題とは話が違ってきているので、このスレッドはクローズして（表題の件は解決したはず）、別に新たにスレッドを作って質問してください。一つの問題の解決が次の疑問をよんで次から次へと質問をするのは避けてください。

Answer

受信イベント中では別スレッドでの実行となるので、TextBoxで表示しようとすると一工夫必要になります。
そこらへんはぐぐるとやりかたがでてくるので調べてみてください。