※ 引述《gn00618777 (非常念舊)》之銘言:
: 版友好
: 我是在 Linux 寫 C。之前寫了一個 process ,是 1個 byte 1 個 byte 讀取的
: blocking 程式,每讀1個byte就檢查是否是 header,但被說寫得不好。
: 於是乎再寫了一個 non-blocking的,主要是先蒐集我想要的長度後,再檢查是否是
: header。
: protocol header 如下:
: byte0 sig1 (0xab)
: byte1 sig2 (0xba)
: byte2 id
: byte3 packet length
: byte4 seq num
: 概念上,我會讀取 uart fd,讀到的 rcv_len 加總起來,若有我達到的長度
: 就去檢查這5個byte是不是有 header。 只要看到 0xab 0xba 我就認定是一個
: header 的起始。也有可能這2個byte出現在這5個byte的任何地方,甚至 byte4
: 會是 0xab,此時我們就有可能需要再讀取1個byte來做判定是否有header。當我發
: 現一個header時,我就會從此header到結尾整個位移到packet的起始。並回傳 left
: 值,此left值代表意思是說,我還需要讀取幾個byte來做判定。 我感覺我這寫法滿
: 囉嗦的,WAIT_HEADER status這樣寫,WAIT_PAYLOAD status也會這樣寫,後面勢必
: 一大坨,但又想不出啥更好的方法,所以想來求助一下版友看有沒有更好的寫法?
: 這是我的範例 code: https://reurl.cc/bzrez3
: 懇請建議,謝謝。
我會建議採用producer/consumer多執行緒的設計,一個producer thread 採用token rin
g方式把uart收進資料結構,很單純的把每一個byte都收進來。
那麼另一個consumer 作為解析token ring的thread,在把解析完的packet送進queue或者
是做task dispatch
以上做法是增加你系統容錯的程度,我還蠻建議的。