samxon 發表于 2024-9-18 17:16
謝謝你熱情給力的解析，還把代碼增加了注釋。非常感謝。希望占用您寶貴時間幫我看看下面的代碼。給點思路 ...

感覺協議制定得不太合理，putCmd()應該能解決你的第3個難點。

1. char putchar(char c) // 發1個
   
2. {
   
3.         while (!TI)
   
4.                 ; // 注意uart初始化時要TI = 1;
   
5.         TI = 0;
   
6.         SBUF = c;
   
7.         return c;
   
8. }
   
9. void putBuff(unsigned char *buf, unsigned char len) // 發多個
   
10. {
    
11.         while (len--)
    
12.                 putchar(*buf++);
    
13. }
    
14. /*
    
15. */
    
16. void putCmd(unsigned char cmd, unsigned short var_add, const unsigned char *dat, unsigned char len)
    
17. {
    
18.         putchar(Pack_Head >> 8);
    
19.         putchar(Pack_Head);
    
20.         putchar(len + 3);
    
21.         putchar(cmd);
    
22.         putchar(var_add >> 8);
    
23.         putchar(var_add);
    
24.         putBuff(dat, len);
    
25. }

復制代碼

putCmd()使用示范

1. #define CMD_1_ADD 0x1200
   
2. #define CMD_2_ADD 0x1350
   
3. #define CMD_3_ADD 0x1400
   
4. code unsigned char CMD_1[] = {0x06, 0x1b};                                         // 寫入電壓
   
5. code unsigned char CMD_2[] = {0x34, 0x35, 0x57, 0x00};                 // 寫入測試標識
   
6. code unsigned char CMD_3[] = {0x50, 0x41, 0x53, 0x53, 0x00}; // 寫入測試結果(pass或fail)
   
7. void putCmd_demo()
   
8. {
   
9.         putCmd(WriteCmd, CMD_1_ADD, CMD_1, sizeof(CMD_1));
   
10.         putCmd(WriteCmd, CMD_2_ADD, CMD_2, sizeof(CMD_2));
    
11.         putCmd(WriteCmd, CMD_3_ADD, CMD_3, sizeof(CMD_3));
    
12.         putCmd(WriteCmd, 1234, "1234567890", sizeof("1234567890")-1);
    
13. }

復制代碼

簡化你之前的Uart_send_data()

1. void Uart_send_data()
   
2. {
   
3.         // unsigned char m;
   
4.         if (REC_COMPLETED)
   
5.         {
   
6.                 REC_COMPLETED = 0;
   
7.                 putBuff(USART_RX_BUF, DATA_LENGTH);
   
8.                 // for (m = 0; m < DATA_LENGTH; m++)
   
9.                 // {
   
10.                 //         SBUF = USART_RX_BUF[m];
    
11.                 //         while (!TI)
    
12.                 //                 ;
    
13.                 //         TI = 0;
    
14.                 // }
    
15.         }
    
16. }

復制代碼

samxon 發表于 2024-9-18 17:16
謝謝你熱情給力的解析，還把代碼增加了注釋。非常感謝。希望占用您寶貴時間幫我看看下面的代碼。給點思路 ...

一個一個來搞，請各位幫忙把關代碼。如有更好優化，請賜教。
添加數據包長度代碼
void Join_Test_Parameter(unsigned char cmd,unsigned short var_add,unsigned short jdata){
        unsigned char temp_buf[15],par_buf[16],i,par_buf_size,cnt=0;
        temp_buf[cnt++]= (Pack_Head&0xFF00)>>8;         //0
        temp_buf[cnt++]= Pack_Head&0xFF;                                        //1
        temp_buf[cnt++]=        cmd;                                                                                //2
        temp_buf[cnt++]=(var_add&0xFF00)>>8;                        //3
        temp_buf[cnt++]=var_add&0xFF;                                                        //4
        temp_buf[cnt++]=(jdata&0xFF00)>>8;                  //5
        temp_buf[cnt++]=jdata&0xFF;                                                        //6       
        for(i=0;i<10;i++){
                if(i<2){
                        par_buf=temp_buf;
                }else if(i==2){
                        par_buf=cnt-3; //將長度直接添加到數組下標2的位置。
                }else{
                        par_buf=temp_buf[i-1];
                }       
        }
        Send_Test_Parameter(par_buf,cnt+1);
}

ydatou 發表于 2024-9-18 12:00

謝謝你熱情給力的解析，還把代碼增加了注釋。非常感謝。希望占用您寶貴時間幫我看看下面的代碼。給點思路和辦法我。謝謝。

遇到的難點：
1) 如何把len這個長度字段插入到數據包的第3個字節。
2)由于有效數據部分是不確定的。可能有時是3個字節，也有可能是四個字節。如何把這些直接傳遞給Join_Test_Parameter函數來正確的產生一個數據包。
3)也許可用一個數組把想要的參數一次全部打包丟給Join_Test_Parameter這個函數處理吧。可是我不知道怎么實現。
謝謝大家，特別是謝謝這個ydatou友情幫忙

void Join_Test_Parameter(unsigned char len,unsigned char cmd,unsigned short var_add,unsigned short jdata){
        unsigned char par_buf[15],cnt=0;
        par_buf[cnt++]= (Pack_Head&0xFF00)>>8;       
        par_buf[cnt++]= Pack_Head&0xFF;       
        par_buf[cnt++]=len;
        par_buf[cnt++]=        cmd;
        par_buf[cnt++]=(var_add&0xFF00)>>8;
        par_buf[cnt++]=var_add&0xFF;
        par_buf[cnt++]=(jdata&0xFF00)>>8;
        par_buf[cnt++]=jdata&0xFF;
        Send_Test_Parameter(par_buf,cnt);
}

void Get_Test_Gear(){          
        if(REC_COMPLETED){
        REC_COMPLETED=0;       
        if(USART_RX_BUF[1]==0x31){
        switch(USART_RX_BUF[2]){
             case 0x00: TEST_GEAR=0;
                Join_Test_Parameter(0x05,WriteCmd,0x1200,0x61B); //寫入電壓
                Join_Test_Parameter(0x05,WriteCmd,0x1250,0x145); //寫入電流
                Join_Test_Parameter(0x05,WriteCmd,0x1300,0x1fb); //寫入功率
                Join_Test_Parameter(0x07,WriteCmd,0x1350,0x34 0x35 0x57 0x00); //寫入測試標識
                Join_Test_Parameter(0x08,WriteCmd,0x1400,0x50 0x41 0x53 0x53 0x00);        //寫入測試結果(pass或fail)
                Join_Test_Parameter(0x05,WriteCmd,0x1653,0x0400);        //改變字體顏色
                Join_Test_Parameter(0x05,WriteCmd,0x1500,0x1455); //寫入最大值
                Join_Test_Parameter(0x05,WriteCmd,0x1200,0x13F1);        //寫入平均值
                Join_Test_Parameter(0x05,WriteCmd,0x1200,0x1389);        //寫入最小值
                break;
             case 0x10: TEST_GEAR=1;
                ........
                }
        }
}

1. /*
   
2. 和樓主代碼比優點如下：
   
3. 占用ram資源少。
   
4. 可連續多幀處理，不會掉幀。
   
5. 多幀之間摻雜其他數據，不會掉幀。
   
6. */
   
7. #define MAX_LENGTH 32//最大數據長度
   
8. idata unsigned char USART_RX_BUF[MAX_LENGTH];//xdata
   
9. unsigned char DATA_LENGTH;//當前數據長度，只有REC_COMPLETED為1時才有效
   
10. bit REC_COMPLETED;
    
11. 
    
12. void Uart() interrupt 4
    
13. {
    
14.         unsigned char d;
    
15.         static unsigned char ccnt = 0;
    
16.         if (RI)
    
17.         {
    
18.                 RI = 0;
    
19.                 d = SBUF;
    
20. 
    
21.                 if (ccnt == 0)
    
22.                 {
    
23.                         if (d == 0x5A)
    
24.                                 ccnt = 1;
    
25.                 }
    
26.                 else if (ccnt == 1)
    
27.                 {
    
28.                         if (d == 0xA5)
    
29.                                 ccnt = 2;
    
30.                         else if (d != 0x5A)//連續多個5A也能正常接收，不掉幀。
    
31.                                 ccnt = 0;
    
32.                 }
    
33.                 else if (ccnt == 2)
    
34.                 {
    
35.                         if (d > 0 && d <= MAX_LENGTH)//數據長度檢查，防止溢出。
    
36.                         {
    
37.                                 DATA_LENGTH = d;
    
38.                                 ccnt = 3;
    
39.                         }
    
40.                         else if (d == 0x5A)//連續多個5A也能正常接收，不掉幀。
    
41.                                 ccnt = 1;
    
42.                         else
    
43.                                 ccnt = 0;
    
44.                 }
    
45.                 else
    
46.                 {
    
47.                         unsigned char i = ccnt - 3;
    
48.                         if (i < MAX_LENGTH)
    
49.                         {
    
50.                                 USART_RX_BUF[i] = d;
    
51.                                 ccnt++;
    
52.                                 if (i == DATA_LENGTH - 1)
    
53.                                 {
    
54.                                         REC_COMPLETED = 1;
    
55.                                         //要盡快(<=2個字符時間)處理使REC_COMPLETED = 0
    
56.                                         //9600波特率下,2個字符時間約2ms。
    
57.                                         ccnt = 0;
    
58.                                 }
    
59.                         }
    
60.                         else
    
61.                                 ccnt = 0;
    
62.                 }
    
63.         }
    
64.         if (TI)
    
65.         {
    
66.                 // TI=0;
    
67.         }
    
68. }
    
69. void Uart_send_data()
    
70. {
    
71.         unsigned char m;
    
72.         if (REC_COMPLETED)
    
73.         {
    
74.                 REC_COMPLETED = 0;
    
75.                 for (m = 0; m < DATA_LENGTH; m++)
    
76.                 {
    
77.                         SBUF = USART_RX_BUF[m];
    
78.                         while (!TI)
    
79.                                 ;
    
80.                         TI = 0;
    
81.                 }
    
82.         }
    
83. }
    

復制代碼

samxon 發表于 2024-9-17 08:18
能不等講一下，我看到很多都是用指針傳送接收數組uartx_Rx_buf的值。直接傳遞和用指針傳區別在哪里。直接 ...

這段代碼不能在51下工作。
51的ram最多只有256字節，單Buf[300]都不夠。
51的臨時變量不是分配在堆棧上，函數一般不支持再入。一個函數假如在某個中斷中有調用，那么它就不適合在其它中斷中和非中斷中調用。
用指針傳遞更靈活方便，但占用ram資源多些。用全局變量傳遞，占用ram資源少，使用限制多。

感謝壇子高工，問題已經解決，增加else條件。
void Uart() interrupt 4
{
        static unsigned char ccnt,bufccnt,recd_temp[5];
        if(RI && REC_COMPLETED==0){
                RI=0;
                if(ccnt<3){
                recd_temp[ccnt++]=SBUF;                       
                }else{
                        if(recd_temp[0]==0x5A && recd_temp[1]==0xA5){
                                DATA_LENGTH=recd_temp[2];
                                USART_RX_BUF[bufccnt++]=SBUF;                              
                                        if(bufccnt==DATA_LENGTH){  //5A A5 02 03 04   0>03,1;1>04,2
                                                REC_COMPLETED=1;
                                                bufccnt=0;
                                                ccnt=0;
                                                ES=0;
                                        }
                  }else{
                                ccnt=0;
                        }      
                }
        }      
if(TI){}      
}

void Uart_send_data(){
        static unsigned char m;               
        if(REC_COMPLETED){
                for(m=0;m<DATA_LENGTH;m++){
                        SBUF=USART_RX_BUF[m];
                        while(!TI);
                        TI=0;
                }
                DATA_LENGTH=0;
                REC_COMPLETED=0;
                m=0;
                ES=1;
        }
}

ydatou 發表于 2024-9-16 15:35
這段代碼還有問題。
1.串口是可以同時收發的。 if(RI && REC_COMPLETED==0)會使發送期間數據接收出錯。
...

能不等講一下，我看到很多都是用指針傳送接收數組uartx_Rx_buf的值。直接傳遞和用指針傳區別在哪里。直接傳程序看起來不是更好清晰嗎。謝謝解答。

void Encode_Receive(uint8_t bytedata)
{
        static uint8_t step=0;//狀態變量初始化為0 在函數中必須為靜態變量
        static uint8_t cnt=0,Buf[300],len,cmd,*data_ptr;
        static uint16_t crc16;
        //進行數據解析 狀態機
        switch(step)
        {
            case 0://接收幀頭1狀態
                if(bytedata== 0xAF)
                {
                    step++;
                    cnt = 0;
                    Buf[cnt++] = bytedata;
                }break;
            case 1://接收幀頭2狀態
                if(bytedata== 0xFA)
                {
                    step++;
                    Buf[cnt++] = bytedata;
                }
                else if(bytedata== 0XAF)
                {
                    step = 1;
                }
                else
                {
                    step = 0;
                }
                break;
            case 2://接收數據長度字節狀態
                step++;
                Buf[cnt++] = bytedata;
                len = bytedata;
                break;
            case 3://接收命令字節狀態
                step++;
                Buf[cnt++] = bytedata;
                cmd = bytedata;
                data_ptr = &Buf[cnt];//記錄數據指針首地址
                if(len == 0)step++;//數據字節長度為0則跳過數據接收狀態
                break;
            case 4://接收len字節數據狀態
                Buf[cnt++] = bytedata;
                if(data_ptr + len == &Buf[cnt])//利用指針地址偏移判斷是否接收完len位數據
                {
                    step++;
                }
                break;
            case 5://接收crc16校驗高8位字節
                step++;
                crc16 = bytedata;
                break;
            case 6://接收crc16校驗低8位字節
                crc16 <<= 8;
                crc16 += bytedata;
                if(crc16 == CRC16_Check(Buf,cnt))//校驗正確進入下一狀態
                {
                    step ++;
                }
                else if(bytedata == 0xAF)
                {
                    step = 1;
                }
                else
                {
                    step = 0;
                }
                break;
            case 7://接收幀尾
                if(bytedata== 0xFF)//幀尾接收正確
                {
                        Encode_Handle(cmd,data_ptr,len);//數據解析
                    step = 0;
                }
                else if(bytedata == 0xAF)
                {
                    step = 1;
                }
                else
                {
                    step = 0;
                }
                break;
            default:step=0;break;//多余狀態，正常情況下不可能出現
        }
}

ydatou 發表于 2024-9-16 15:35
這段代碼還有問題。
1.串口是可以同時收發的。 if(RI && REC_COMPLETED==0)會使發送期間數據接收出錯。
...

非常感謝，這個是狀態機的思想寫的代碼。當然最好。值得擁有。

samxon 發表于 2024-9-16 09:31
感謝壇子高工，問題已經解決，增加else條件。
void Uart() interrupt 4
{

這段代碼還有問題。
1.串口是可以同時收發的。 if(RI && REC_COMPLETED==0)會使發送期間數據接收出錯。
2.錯誤檢查有漏洞。數據長度要做范圍檢查。
3.浪費了太多全局變量，51的ram資源又比較少。

請參考我的代碼。

1. #define MAX_LENGTH 32
   
2. unsigned char USART_RX_BUF[MAX_LENGTH];
   
3. unsigned char DATA_LENGTH;
   
4. bit REC_COMPLETED;
   
5. 
   
6. void Uart() interrupt 4
   
7. {
   
8.         unsigned char d;
   
9.         static unsigned char ccnt=0;
   
10.         if (RI)
    
11.         {
    
12.                 RI = 0;
    
13.                 d = SBUF;
    
14. 
    
15.                 if (ccnt==0){
    
16.                         if(d==0x5A)
    
17.                                 ccnt=1;
    
18.                 }
    
19.                 else if (ccnt==1){
    
20.                         if(d==0xA5)
    
21.                                 ccnt=2;
    
22.                         else
    
23.                                 ccnt=0;
    
24.                 }
    
25.                 else if (ccnt==2){
    
26.                         if(d>0 && d<=MAX_LENGTH){
    
27.                                 DATA_LENGTH=d;
    
28.                                 ccnt=3;
    
29.                         }
    
30.                         else
    
31.                                 ccnt=0;
    
32.                 }
    
33.                 else {
    
34.                         unsigned char i=ccnt-3;
    
35.                         if(i<MAX_LENGTH){
    
36.                                 USART_RX_BUF[i] = d;
    
37.                                 ccnt++;
    
38.                                 if(i==DATA_LENGTH-1) {
    
39.                                         REC_COMPLETED = 1;
    
40.                                         ccnt=0;
    
41.                                 }
    
42.                         }
    
43.                         else
    
44.                                 ccnt=0;
    
45.                 }
    
46.         }
    
47.         if (TI)
    
48.         {
    
49.                 // TI=0;
    
50.         }
    
51. }
    
52. void Uart_send_data()
    
53. {
    
54.         unsigned char m;
    
55.         if (REC_COMPLETED)
    
56.         {
    
57.                 REC_COMPLETED = 0;
    
58.                 for (m = 0; m < DATA_LENGTH; m++)
    
59.                 {
    
60.                         SBUF = USART_RX_BUF[m];
    
61.                         while (!TI)
    
62.                                 ;
    
63.                         TI = 0;
    
64.                 }
    
65.         }
    
66. }

復制代碼

感謝壇子高工，問題已經解決，增加else條件。
void Uart() interrupt 4
{
        static unsigned char ccnt,bufccnt,recd_temp[5];
        if(RI && REC_COMPLETED==0){
                RI=0;
                if(ccnt<3){
                recd_temp[ccnt++]=SBUF;                       
                }else{
                        if(recd_temp[0]==0x5A && recd_temp[1]==0xA5){
                                DATA_LENGTH=recd_temp[2];
                                USART_RX_BUF[bufccnt++]=SBUF;                               
                                        if(bufccnt==DATA_LENGTH){  //5A A5 02 03 04   0>03,1;1>04,2
                                                REC_COMPLETED=1;
                                                bufccnt=0;
                                                ccnt=0;
                                                ES=0;
                                        }
                  }else{
                                ccnt=0;
                        }       
                }
        }       
if(TI){}       
}

void Uart_send_data(){
        static unsigned char m;               
        if(REC_COMPLETED){
                for(m=0;m<DATA_LENGTH;m++){
                        SBUF=USART_RX_BUF[m];
                        while(!TI);
                        TI=0;
                }
                DATA_LENGTH=0;
                REC_COMPLETED=0;
                m=0;
                ES=1;
        }
}

發表于 2024-9-15 09:51
if(bufccnt=DATA_LENGTH)這句是不是應該用兩個等于號==

修正，情況依舊，可能還是數組上面的問題。但沒有排查的思路

if(bufccnt=DATA_LENGTH)這句是不是應該用兩個等于號==