Calculating CRC16 in C#

Question

我试图将旧代码从C移植到c#，它基本上接收字符串并返回它的CRC16…

C方法如下:

#define CRC_MASK 0x1021 /* x^16 + x^12 + x^5 + x^0 */
UINT16 CRC_Calc (unsigned char *pbData, int iLength)
{
    UINT16 wData, wCRC = 0;
    int i;
    for ( ;iLength > 0; iLength--, pbData++) {
        wData = (UINT16) (((UINT16) *pbData) << 8);
        for (i = 0; i < 8; i++, wData <<= 1) {
            if ((wCRC ^ wData) & 0x8000)
                wCRC = (UINT16) ((wCRC << 1) ^ CRC_MASK);
            else
                wCRC <<= 1;
        }
    }
    return wCRC;
}

我移植的c#代码如下:

private static ushort Calc(byte[] data)
    {
        ushort wData, wCRC = 0;
        for (int i = 0; i < data.Length; i++)
        {
            wData = Convert.ToUInt16(data[i] << 8);
            for (int j = 0; j < 8; j++, wData <<= 1)
            {
                var a = (wCRC ^ wData) & 0x8000;
                if ( a != 0)
                {
                    var c = (wCRC << 1) ^ 0x1021;
                    wCRC = Convert.ToUInt16(c);
                }
                else
                {
                    wCRC <<= 1;
                }
            }
        }
        return wCRC;
    }

测试字符串是"OPN"…它必须返回一个(ofc) 2字节的单位A8 A9， #CRC_MASK是该计算的多项式。我确实在这里和网络上找到了几个CRC16的例子，但它们都没有达到这个结果，因为这个CRC计算必须与我们连接的设备相匹配。

错误在哪里?我非常感谢你的帮助。

谢谢!

问好古腾堡所谓

根据@rcgldr的回答，我整理了以下示例:

_serial = new SerialPort("COM6", 19200, Parity.None, 8, StopBits.One);
        _serial.Open();
        _serial.Encoding = Encoding.GetEncoding(1252);
        _serial.DataReceived += Serial_DataReceived;
        var msg = "OPN";
        var data = Encoding.GetEncoding(1252).GetBytes(msg);
        var crc = BitConverter.GetBytes(Calc(data));
        var msb = crc[0].ToString("X");
        var lsb = crc[1].ToString("X");
        //The following line must be something like: x16OPNx17xA8xA9
        var cmd = string.Format(@"{0}{1}{2}x{3}x{4}", SYN, msg, ETB, msb, lsb);
        //var cmd = "x16OPNx17xA8xA9";
        _serial.Write(cmd);

cmd变量的值是我试图发送到设备的内容。如果你看一下注释的cmd值，这是一个工作字符串。CRC16的2个字节放在最后两个参数(msb和lsb)中。因此，在这里的示例中，msb必须为"xA8"，lsb必须为"xA9"，以便命令工作(设备上的CRC16匹配)。

什么线索吗?

再次感谢。

更新2 对于那些遇到同样情况的人，你需要用x格式化字符串，我就是这样做的:

protected string ToMessage(string data)
    {
        var msg = data + ETB;
        var crc = CRC16.Compute(msg);
        var fullMsg = string.Format(@"{0}{1}{2:X}{3:X}", SYN, msg, crc[0], crc[1]);
        return fullMsg;
    }

这返回给我完整的消息，我需要包括它上面的x。SYN变量为'x16'， ETB为'x17'

谢谢大家的帮助!

古腾堡所谓

Answer 1

这里的问题是，包括ETB (x17)的消息是4字节长(前边同步字节不用于CRC): "OPNx17"== {'O'， 'P'， 'N'， 0x17}，这导致一个{0xA8, 0xA9}的CRC被附加到消息中。所以CRC函数是正确的，但是原始测试数据不包括第4个字节，即0x17。

这是一个工作示例(至少在VS2015 express中)。

private static ushort Calc(byte[] data)
{
    ushort wCRC = 0;
    for (int i = 0; i < data.Length; i++)
    {
        wCRC ^= (ushort)(data[i] << 8);
        for (int j = 0; j < 8; j++)
        {
            if ((ushort)(wCRC & (ushort)0x8000u) != 0)
                wCRC = (ushort)((ushort)(wCRC << 1) ^ (ushort)0x1021u);
            else
                wCRC <<= 1;
        }
    }
    return wCRC;
}