※ 引述《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 : 懇請建議，謝謝。 寫code駑鈍.. 我參考了版友建議，改了寫法。 我用了類似生產者消費者方式。當 select uart 有資料時，讀取 sizeof(rbuf) = 256 的長度。根據uart處於何種狀態(MSG_WAIT_ID1, MSG_WAIT_ID2....)，來決定抓取的數量 完整header共5byte: ID1, ID2 , OP, SEQ, LEN 假設 select 到了 5 個 byte了，我就去一個byte一個byte去檢查，如果這五個byte是 xx,xx,ID1,ID2,OP ，跑了我的 find_header 後就會求出目前 state 狀態為 MSG_WAIT_SEQ，根據此狀態發現，我還需要再抓取2byte，才足夠去判定他是不是一個 完整的 5 個 byte header。 先姑且不論效率和coding style ，我想求問的是以下: rcv = FRONT_UART_read(fd, rbuf + idx, sizeof(rbuf)); 有沒有好的改寫方式? 我怕的是此行，如果我 idx 停留在 255，當 uart device 搞我送出 256 byte資料，我不就超出記憶體範圍了? 還有請問我 99~106 有沒有邏輯上的錯誤呢? 萬分感謝.. https://reurl.cc/NrZ4We --

※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 59.115.95.86 (臺灣) ※ 文章網址: https://webptt.com/m.aspx?n=bbs/C_and_CPP/M.1622974519.A.2FD.html