请教坛里很多MODBUS计算CRC16的方法两个uint8_t return能得到uint16_t吗？

mangolu

请教坛里很多MODBUS计算CRC16的方法两个uint8_t return (CRCcode_L << 8 | CRCcode_H)能得到uint16_t？

代码如下：

1. /**************************************************************************************************
   
2.     NAME : comp_crc16
   
3.    INPUT : unsigned char *pack 数据缓冲首地址, unsigned char num 数据字节数
   
4.   OUTPUT : unsigned int 计算结果，低位在前 高位在后（MODBUS 规定）。
   
5. FUNCTION :
   
6.            CRC16 计算 MODBUS使用公式：X16+X15+X2+1 。此不同于CITT公式
   
7. **************************************************************************************************/
   
8. unsigned int comp_crc16(unsigned char *pack, unsigned char num)
   
9. {
   
10.         unsigned char CRCcode_H = 0XFF;                // 高CRC字节初始化
    
11.         unsigned char CRCcode_L = 0XFF;                // 低CRC 字节初始化
    
12.         unsigned char index;                        // 数据索引
    
13. 
    
14.         while (num--)
    
15.         {
    
16.                 index = CRCcode_L ^ (*pack++);       
    
17. 
    
18.                 CRCcode_L = CRCcode_H ^ CRC_H[index];
    
19. 
    
20.                 CRCcode_H = CRC_L[index];
    
21.         }
    
22.         return (CRCcode_L << 8 | CRCcode_H);        // MODBUS 规定低位在前
    
23. }

复制代码

按照我的理解，CRCcode_L为uint8_t，左移8位后应应该为0x00吧？两个uint8_t相或后，能返回uint16_t值？我看了坛里很多代码都是这样的。

而freemodbus里的代码为：

1. USHORT
   
2. usMBCRC16( UCHAR * pucFrame, USHORT usLen )
   
3. {
   
4.     UCHAR           ucCRCHi = 0xFF;
   
5.     UCHAR           ucCRCLo = 0xFF;
   
6.     int             iIndex;
   
7. 
   
8.     while( usLen-- )
   
9.     {
   
10.         iIndex = ucCRCLo ^ *( pucFrame++ );
    
11.         ucCRCLo = ( UCHAR )( ucCRCHi ^ aucCRCHi[iIndex] );
    
12.         ucCRCHi = aucCRCLo[iIndex];
    
13.     }
    
14.     return ( USHORT )( ucCRCHi << 8 | ucCRCLo );
    
15. }

复制代码

明显这里的代码采用了强制类型转换，这样的结果才是正确的。

我就很疑惑，坛里很多人的代码真能正常计算吗？

takashiki

符合C语言标准的当然是必须可以，不符合标准的编译器要特立独行就不一定了。

zhugean

return (  ( USHORT )ucCRCHi << 8 | ucCRCLo ); 才是正确的写法。前面都需要依赖C语言的自动整型提升

Error.Dan

看你的函数声明的返回类型,这里有个隐含的类型转换,最后一步运算两个byte都升到unsigned int类型计算了.

JasonGao

楼主C语言基础不牢啊。建议时常复习一下K&R的那本C语言圣经，基础不牢，地动山摇嘛

tangmin

直接用freemodbus里面的一个专门计算crc的c文件就可以了，直接调用函数

kation122

班门弄斧，抛砖引玉，分享使用联合体来做 2byte to uint16,  4byte to uint32 的方法，
不浪费MCU 移位运算，不逻辑运算,简洁明了，稳定使用多年，抛砖引玉，请各位大侠点评。

# define LSB 1
# define MSB 0

typedef unsigned char U8;
typedef unsigned int U16;

typedef union UU16
{
   U16 U16;
   U8 U8[2];
} UU16;
//-----------------------------//
UU16 CRC;
CRC.U8[MSB]=0XAB;
CRC.U8[LSB]=0X34;
return CRC.U16;
这里CRC.U16 的值就是0XAB34，数据类型是unsigned int ，

//-----------------------------//
CRC.U16=0XBD56;
CRC.U8[MSB]值就是0XBD
CRC.U8[LSB]值就是0X56

编译器大小端不同，只需要调整# define LSB ，# define MSB  两个参数。

mangolu

Error.Dan 发表于 2021-1-18 08:48
看你的函数声明的返回类型,这里有个隐含的类型转换,最后一步运算两个byte都升到unsigned int类型计算了. ...

哦，原来这里还有个隐形转换，这个一直没有注意过。

mangolu

kation122 发表于 2021-1-18 10:50
班门弄斧，抛砖引玉，分享使用联合体来做 2byte to uint16,  4byte to uint32 的方法，
不浪费MCU 移位运算 ...

这个方法不错，谢谢