C#　シリアル通信のバイナリデータ受信

現在、VisualStudio2015 のWindowsフォームアプリケーションにてソケットを使用したシリアル通信のコードを書いています。

基本的な通信（送受信）はできるようになったのですが、受信メソッドにて

C#
1 //受信データのバイト数分蓄積
2  do
3  {
4     int length = ComPort.Read(buffer, readlen, rbyte - readlen);
5     readlen += length;
6  } while (readlen < rbyte);
7   recData += System.Text.Encoding.ASCII.GetString(buffer);
8   logText(recData, ComPort);

これで送られてきたByteをそのバイト分貯めてから
文字列に変換してテキストボックスにinvokeして書き込んでいますが、受信時にデータが分割されて送られてしまいます。

なので改行まで取得するように改良して以下のようにしました。

C#
1 // 受信データを改行まで蓄積
2do
3{
4   int length = ComPort.Read(buffer, readlen, rbyte - readlen);
5   readlen += length;
6} while (readlen < rbyte);
7 recData += System.Text.Encoding.ASCII.GetString(buffer);
8 newline = recData.IndexOf(Environment.NewLine);
9if (newline >= 0)
10{
11   logText(recData, ComPort);
12}
13else
14{
15   Sendbuffa += recData;       //データをいったん保存
16}

ですが、これだと一度の2回送信分のデータ（改行コードがふたつある）を受信してしまったときに、
1回送信分の中に改行コードが２つあると判断されてしまいます。
なので、改行コードを見つけたらその時点でテキストボックスに書きだしたいのですが、どう修正すればよいでしょうか。

尚、上記両コードとも表記したコードをCOMポート接続されている間ループさせております。

よろしくお願いいたします。

行動規範の内容に同意します

回答2件

文字列を流すのを前提とするなら、素直にReadline メソッドを使うなりしたほうがいいと思いますねー

投稿2018/04/13 02:43

y_waiwai

総合スコア88024

退会済みユーザー

2018/04/13 03:46

y_waiwai 様いつもコメントありがとうございます。　今は文字列のテストですがバイナリデータを扱えるものにしたいのでこのようにしております。

y_waiwai

2018/04/13 04:00

シリアルに限らず、TCPとかUDPとかの通信も含め、XXXバイト送信したのに、受信ではXXバイトとXXバイトに分けて受信される、XXバイトとXXバイトの２回送ったのにXXXXバイトにまとめて受信されてしまったというのはしごく当然の話となります。それを前提としてコードを組む必要があります一回の受信で改行が２つ受信されてしまったらどうしよう、というのではなく、受信イベントではとにかく今まで受信したデータを貯めていく、そして、その中から改行があればそこまでを１行として取り出す、という考え方で行かないとあとで苦労することになろうかと思いますで、ReadLineメソッドというのは、上記の処理を実装しているんですね

退会済みユーザー

2018/04/13 07:49

＞＞受信イベントではとにかく今まで受信したデータを貯めていくなるほどです。ReadLineのことを認識違いしていました。これからなにかを受信するときは心がけるようにします！！ありがとうございます！

行動規範の内容に同意します

ベストアンサー

こんにちは。
Split()関数を使って改行(\r\n)で分割してはどうでしょうか？

csharp
1            // 改行で分割
2            string[] lines = recData.Split(
3                new[] { Environment.NewLine },
4                StringSplitOptions.None
5            );
6
7            if (lines.Length > 1)
8            {
9                // lines[0] だけを送信して残りはバッファに詰めて次回
10            }
11            else
12            {
13                // lines[0]を送信して終わり
14            }

EDIT

記述が不親切だったのでコード例を記載します。

csharp
1            recData += System.Text.Encoding.ASCII.GetString(buffer);
2            var temporary = Sendbuffa + recData;
3            while (true)
4            {
5                newline = temporary.IndexOf(Environment.NewLine);
6                if (newline >= 0)
7                {
8                    string[] lines = temporary.Split(
9                        new[] { Environment.NewLine },
10                        StringSplitOptions.None
11                    );
12                    logText(lines[0], ComPort);
13
14                    if (lines.Length > 1)
15                    {
16                        // temporaryに残りを詰めなおす
17                        temporary = temporary.Substring(newline + 2);
18                    }
19                }
20                else
21                {
22                    Sendbuffa = temporary;       //データをいったん保存
23                    break;
24                }
25            }