[C#/Windows.Form]シリアル通信でのDataReceive中のWrite処理について

前提・実現したいこと

USBでデバイスと通信するプログラムを実装しています。
プログラムではSerialPortクラスを使用していて、
DataReceivedメソッド内でWriteメソッドを呼び出して最初に受け取ったデータと
Writeメソッドにて受け取ったデータの両方をDBに格納したいです。
※デバイスのほうがまだないため、手探りで実装中ですので、確認がすぐにはできない状況です。

お力添えをお願い致します。

#####修正、加筆
通信フォーマットは、テキスト形式です。
区切りは、CRLFの改行コードになります。

発生しそうな問題・エラーメッセージ

DataReceivedメソッド内での処理のため、うまくいかないのではと思っています。

該当のソースコード

C#
1/// <summary>
2/// データ受信時イベント
3/// </summary>
4/// <param name="sender"></param>
5/// <param name="e"></param>
6private void SerialPort_DataReceived(object sender, SerialDataReceivedEventArgs e)
7{
8　if (serialPort != null && serialPort.IsOpen)
9　{
10　　try
11　　{
12　　　// 受信データを読み込む
13　　　SerialPort sp = (SerialPort)sender;
14　　　string sChildData = sp.ReadExisting();
15　　　if (データが二つそろっている) {
16　　　　// DBへ格納処理
17　　　} else {
18　　　　// write処理
19　　　　DataWrite(string buffer);
20　　　}
21　　}
22　}
23}
24
25/// <summary>
26/// データ書き込み処理
27/// </summary>
28/// <param name="command"></param>
29private bool DataWrite(string buff)
30{
31	int iRetry = 1;
32
33	try
34	{
35		serialPort.Write(buff);
36		return true;
37	}
38	catch (IOException ex)
39	{
40		// リトライは3回まで
41		iRetry += 1;
42
43		if (3 < iRetry)
44		{
45			this.DataWrite(buff);
46		}
47		return false;
48	}
49	catch (Exception ex)
50	{
51		iRetry += 1;
52
53		if (3 < iRetry)
54		{
55			this.DataWrite(buff);
56		}
57		return false;
58	}
59}

試したこと

デバイスがないため試せておりません。

補足情報（FW/ツールのバージョンなど）

Windows 10
Visual Studio 2017

Wind

2020/01/17 01:00

何がわからないのか全く不明ですが、デバイスが無いのでしたらUSBシリアル変換を２つ用意して、 COMポート2つを使って確認出来ないでしょうか？一度確認出来れば、不具合発生時にデバイス側とどちらが悪いのかの切り分けにも使えます。

coco_bauer

2020/01/17 01:16

まず、シリアル通信をするための道具（シリアル通信ケーブル、USB-シリアル通信変換器など）を用意しましょう。どんな機器を使うかによってプログラムの修正(例えば、通信速度の設定をする部分で利用可能な通信速度は機器によって異なる）が必要になるかもしれませんから。どんなデバイスを使おうとしているのですか、Windows 10と書かれているので一方はWindowsのPCだと思うのですが、もう一方は？　シリアルポートを備えたPCどうしだと、ケーブルでつなぐだけで簡単です。

entaro12345

2020/01/17 01:16

DataReceivedメソッド処理中にDataWriteメソッドにて処理をした場合に、 DataReceivedイベントでちゃんとひろえるのかがわかっておりませんでしたので、質問いたしました。

shinami

2020/01/17 02:55

実機が無いなら相手を別のパソコンでやってもよいし自分自身パソコンのシリアルポートへ繋いでもよいので実機相当のアプリを別途作り両方実行して通信してみればよいと思います。

testset

2020/01/17 16:47

※以下は質問者の内容に対する回答ではありません DataWrite内でDataWriteを再起呼び出しています。場合によってはStackoverflowとなる可能性がありますが問題ないのでしょうか。 (retryが意味を成していないように見えます) ------------ 試験環境がない方のための解決策を提示いたします。シリアルポートがない場合の試験方法についての提案 https://qiita.com/indication@github/items/4f35813b326b34743cf1

Q71

2020/01/18 07:51 編集

private bool DataWrite() { 　　int iRetry = 1;　　…再呼び出ししたときに1に初期化されるので、　　... 　　catch (IOException e) 　　{ 　　　　iRetry += 1;　・・・必ず2となり、　　　　if (3 < iRetry)　・・・かならずfalseになる…あれ？大小が逆じゃない？

entaro12345

2020/01/19 23:30

testsetさま、Q71さまありがとうございます。おっしゃる通りですね。修正します。