回答編集履歴

テストプログラム追記

2020/09/19 12:14

投稿

thkana

スコア7703

test CHANGED Viewed

@@ -223,3 +223,47 @@
 ```
 32.45.703のあたりなど、微妙なタイミングによってハンドラが呼ばれる順番が変わったりしているのが見えますが、つまりほぼ同時に入力された信号をきちんと処理できているようです。
+マイコン側テストプログラムも追記
+```C++
+//mbed(STM32)
+#include "mbed.h"
+int main(void){
+    Serial pc(USBTX, USBRX);
+    pc.baud(115200);
+    char c='A';
+    int offset=0;
+    while(1){
+        printf("%c%c",c+offset,c+offset+1);
+        fflush(0);
+        printf("%c%c",c+offset,c+offset+1);
+        fflush(0);
+        offset=(offset+1)%('H'-'A');// 'A'～'H'を回す
+        wait(40/115.2);//送り返し時間分待ち
+    }
+    return 0;
+}
+```

サンプル追加

2020/09/19 12:14

投稿

thkana

スコア7703

test CHANGED Viewed

@@ -13,3 +13,213 @@
 それと、ハンドラは送信データに対して「いつ呼ばれるかわからない」ものです。極論、1回目の送信の途中で呼ばれるかも(1バイトでも受信すればハンドラは呼ばれるかも)、逆に2回送信が完了してしまってから呼ばれるかも(いつまでに呼ばれる、という期限もありません)、そのどんな場合でも正しくデータを得て、2回送信されたことの確認ができるようにプログラムを作らなきゃいけません。
 そうなっているでしょうか。
+---
+サンプル書いてみました。シリアル周りだけ、エラーとかは無視。
+全然マルチスレッドでないのは、解決したい内容にマルチスレッドが全く寄与しないためです。
+```C#
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using System.IO.Ports;
+namespace SerialMulti
+{
+    class Program
+    {
+        static void Main(string[] args)
+        {
+            object lockObj = new object();
+            //シリアルポート毎にインスタンスを生成する
+            Connection con1 = new Connection(args[0], lockObj);
+            Connection con2 = new Connection(args[1], lockObj);
+            string timeStamp = DateTime.Now.ToString("mm.ss.fff");
+            Console.WriteLine("Start: {0}\n", timeStamp);
+            //なにかキーを押したら終了
+            ConsoleKeyInfo tmp = Console.ReadKey();
+        }
+    }
+    // シリアルポートとハンドラをクラスにまとめておく
+    class Connection
+    {
+        SerialPort port = new SerialPort();
+        char[] buf = new char[4];
+        object lockObj;
+        public Connection(string portName, object lockObj)
+        {
+            this.lockObj = lockObj;
+            port.PortName = portName;
+            port.BaudRate = 115200;
+            port.DataReceived += Port_DataReceived;
+            port.Open();
+        }
+        ~Connection()
+        {
+            port.Close();
+        }
+        //インスタンス毎に各ポート「専用」のハンドラとなる
+        private void Port_DataReceived(object sender, SerialDataReceivedEventArgs e)
+        {
+            //受信文字がある限り処理する。
+            //受信文字が空になって一旦ハンドラを抜けても次の文字を受信したら動作が継続できなければいけない
+            while (port.BytesToRead != 0)
+            {
+                //直近4文字のバッファに受信する
+                for (int i = 0; i < buf.Length - 1; i++)
+                {
+                    buf[i] = buf[i + 1];
+                }
+                buf[buf.Length - 1] = (char)port.ReadByte();
+                //直近4文字において前半2文字と後半2文字が同じか検査
+                if (buf[0] == buf[2] && buf[1] == buf[3])
+                {
+                    string timeStamp = DateTime.Now.ToString("mm.ss.fff");
+                    //念の為排他処理にしておく
+                    lock (lockObj)
+                    {
+                        Console.WriteLine("{0} {1}:{2}{3}", timeStamp, port.PortName, buf[0], buf[1]);
+                    }
+                    //送り返し
+                    port.Write(String.Format("{0}{1}", buf[0], buf[1]));
+                }
+            }
+        }
+    }
+}
+```
+これに、マイコンと、そのUART出力をパラに別のUSB-UART変換を噛ませてPCのポートに入力、つまり「複数のシリアル信号を全く同時に入力」した時の出力がこんな感じです。(USBを通ったりしているうちにプログラムからみて完全に同時とは言えなくなるでしょうが、まぁそこそこ)
+```text
+32.45.494 COM3:BC
+32.45.495 COM16:BC
+32.45.529 COM3:CD
+32.45.543 COM16:CD
+32.45.564 COM3:DE
+32.45.575 COM16:DE
+32.45.599 COM3:EF
+32.45.607 COM16:EF
+32.45.634 COM3:FG
+32.45.639 COM16:FG
+32.45.669 COM3:GH
+32.45.671 COM16:GH
+32.45.703 COM16:AB
+32.45.704 COM3:AB
+32.45.739 COM3:BC
+32.45.751 COM16:BC
+32.45.774 COM3:CD
+32.45.783 COM16:CD
+32.45.809 COM3:DE
+32.45.815 COM16:DE
+```
+32.45.703のあたりなど、微妙なタイミングによってハンドラが呼ばれる順番が変わったりしているのが見えますが、つまりほぼ同時に入力された信号をきちんと処理できているようです。