Re: [問題] 一個讀取 uart 的 process

看板C_and_CPP作者 (非常念舊)時間2年前 (2021/06/06 18:15), 編輯推噓0(004)
留言4則, 2人參與, 2年前最新討論串3/3 (看更多)
※ 引述《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://www.ptt.cc/bbs/C_and_CPP/M.1622974519.A.2FD.html

06/06 21:54, 2年前 , 1F
恩...它第三個參數是放 buffer size 吧?
06/06 21:54, 1F

06/06 23:53, 2年前 , 2F
你的意思是說,我不該寫死,應該動態調整讀取長度?
06/06 23:53, 2F

06/07 00:18, 2年前 , 3F
動態調整嗎...我覺得應該講成給的 buffer size 有多少
06/07 00:18, 3F

06/07 00:18, 2年前 , 4F
,就該填多少
06/07 00:18, 4F
文章代碼(AID): #1WlA0tBz (C_and_CPP)
文章代碼(AID): #1WlA0tBz (C_and_CPP)