CRC-16/Modbus在C#故障中的实现

Question

我目前正在建立步进电机控制器和计算机之间的通信，用C#编写一个应用程序（这是我第一次使用这种编程语言，虽然我不是计算机科学家，而是工业工程师，但我肯定有一些优化功能的方法，我不知道，关于这一点的任何建议也将非常感谢）。因此，我一直使用控制器必须与其通信的RS-485，并实现了一个生成所需CRC（循环冗余校验）字节的算法。

class Program
    {
        static void Main(string[] args)
        {
            // 0x05, 0x06, 0x17, 0x70, 0x00, 0x01 
            byte[] prueba = new byte[] { 0x02, 0x07 };
            byte[] result = Aux.CRC(prueba);
            Console.WriteLine(result[0] + " " + result[1]);
        }
    }

class Aux{
public static byte[] CRC(byte[] to_evaluate)
        {
            byte[] CRC_Byte = new byte[2] { 0, 0 };
            UInt16 CRC_Register = 0xFFFF;            //16 bits 1111.1111.1111.1111
            UInt16 CRC_pol = 0xa001;                 //16 bits 1010.0000.0000.0001

            foreach (UInt16 byte_val in to_evaluate)
            {
                CRC_Register ^= byte_val;
                Console.WriteLine("XOR inicial : {0:X}", CRC_Register);

                for (byte i = 0; i < 8; i++)
                {
                    CRC_Register >>= 1;
                    Console.WriteLine("Desplazamiento " + (i + 1) + ": {0:X}", CRC_Register);

                    if ((CRC_Register & 1) != 0)
                    {
                        CRC_Register ^= CRC_pol;
                        Console.WriteLine("XOR: {0:X}", CRC_Register);
                    }
                }
            }
            Console.WriteLine("{0:X}",CRC_Register);

            byte low_byte_CRC = (byte)((CRC_Register << 8) >> 8);
            byte high_byte_CRC = (byte)(CRC_Register >> 8);

            CRC_Byte[0] = low_byte_CRC;
            CRC_Byte[1] = high_byte_CRC;

            return CRC_Byte;
        }
}

使用附加的测试数组和多项式0xA001的预期结果对于CRC_Register是0x1241，对于CRC_Byte是{0x41，0x12}。

Answer 1

我不得不在C#中为PPP实现一次CRC检查，这绝对没有乐趣！

我在这个链接中找到了应该正确生成CRC的代码。它遵循您共享链接到的文档第39页6.2.2节中的CRC生成过程。

// Compute the MODBUS RTU CRC
UInt16 ModRTU_CRC(byte[] buf, int len)
{
    UInt16 crc = 0xFFFF;
  
    for (int pos = 0; pos < len; pos++) 
    {
        crc ^= (UInt16)buf[pos];          // XOR byte into least sig. byte of crc
  
        for (int i = 8; i != 0; i--)      // Loop over each bit
        {    
            if ((crc & 0x0001) != 0)        // If the LSB is set
            {      
                crc >>= 1;                    // Shift right and XOR 0xA001
                crc ^= 0xA001;
            }
            else                            // Else LSB is not set
            {                            
                crc >>= 1;                    // Just shift right
            }
        }
    }
    // Note, this number has low and high bytes swapped, so use it accordingly (or swap bytes)
    return crc;  
}