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)这句是不是应该用两个等于号==