Question

我将CRC附加到数据流。我正在使用RS-232的串口libary for Linux和Windows。

我希望获得与comport工具图片上显示的值相同的值：

Serialport communcation from a comport tool

我使用const char[4086]="123456789";作为缓冲区。

返回的CRC是bb3d，然后我从unsigned short转换为char *。

#define SERIALPORT_BUFSIZE       4088

char *im_buf = (char *)malloc(SERIALPORT_BUFSIZE  + 8);

char *tx_buf = (char *)malloc(SERIALPORT_RCV_BUFSIZE  + 8);

const char buf[SERIALPORT_BUFSIZE]="123456789";

strcpy(im_buf, buf);
WriteToSerialPort(); // call to function

void SerialPortCom::WriteToSerialPort(void)
{
    if(!port_open) return;

    int i,n, tmp, tmp2;

    if(newline == 1)
    {
        strcat(im_buf, "0a");
    }
    else if(newline == 2)
    {
        strcat(im_buf, "0d");
    }
    else if(newline == 3)
    {
        strcat(im_buf, "0d0a");
    }

    n = strlen(im_buf) / 2;

    if(n < 1) return;

    for(i=0; i< n; i++)
    {
        tmp = im_buf[i*2];

        if((tmp >= '0') && (tmp <= '9'))
        {
            tmp -= '0';
        }
        else if((tmp >= 'a') && (tmp <= 'f'))
        {
            tmp -= ('a' - 10);
        }
        else if((tmp >= 'A') && (tmp <= 'F'))
        {
            tmp -= ('A' - 10);
        }

        tmp *= 16;

        tmp2 = im_buf[(i*2)+1];

        if((tmp2 >= '0') && (tmp2 <= '9'))
        {
            tmp2 -= '0';
        }
        else if((tmp2 >= 'a') && (tmp2 <= 'f'))
        {
            tmp2 -= ('a' - 10);
        }
        else if((tmp2 >= 'A') && (tmp2 <= 'F'))
        {
            tmp2 -= ('A' - 10);
        }

        tmp += tmp2;

        tx_buf[i] = tmp;

    }
    //generate crc
    unsigned short crc  = GetCrc16(im_buf, strlen(im_buf));
    //append crc
    memcpy( tx_buf + n, &crc, 2 );
    //send to serial port
    RS232_SendBuf(comport_nr, (unsigned char *)tx_buf, n+2);

}

我得到的结果：（12 34 56 78 70 11 4A 3D）
我想要的是：（12 34 56 78 7B 34 FD FD FD FD）

如何达到预期效果？

unsigned short SerialPortCom::GetCrc16(const char* InStr,unsigned int len)
{
    //Crc16
    unsigned short Crc16Table[256];
    unsigned int i,j;
    unsigned short Crc;      
    unsigned char CRCHigh = 0xFF, CRCLow = 0xFF;

    for (i = 0; i < 256; i++)
    {
        Crc = i;
        for (j = 0; j < 8; j++)
        {
            if(Crc & 0x1)
                Crc = (Crc >> 1) ^ 0xA001;
            else
                Crc >>= 1;
        }
        Crc16Table[i] = Crc;
    }
    //CRC16
    Crc=0x0000;
    for(i=0; i<len; i++)
    {
        Crc = (Crc >> 8) ^ Crc16Table[(Crc & 0xFF) ^(uint8_t) InStr[i]];

    }
    //Crc ^= 0x0000;

    return Crc;
}

Answer 1

首先，一些注意事项：

您的输入是ASCII十六进制。每个字符都是半字节，但您的输入字符串有奇数个字符：您丢弃最后一个字符。
您期望的是12 34 56 78 7B 34：0x347B是0x12 0x34 0x56 0x78的CRC。额外的FD看起来像缓冲区的一些不相关的结尾（或另一个错误？）。

您的计划存在的问题：

您需要使用RS232_SendBuf发送数据缓冲区。您将ASCII转换为数据并将其放入tx_buf然后取出其CRC。没问题，但是你使用strcat将CRC附加到缓冲区;你不能这样做，strcat用于文本。 strcat期望有两个空终止字符串，但是你有两个数据缓冲区。

您的CRC功能也存在问题：您有一个多余的掩码& 0x7F，它会删除每个CRC字节的最后一位。 0xFF掩码就足够了（实际上甚至不需要）。

您还可以使用new char[2]分配缓冲区，但永远不会delete。这会有效，但很难看。

请注意，您的CRC只有两个字节;只需将Crc直接作为unsigned short返回。

之后，只需使用memcpy将CRC附加到缓冲区在您的情况下，那将是memcpy( tx_buf + n, &my_crc_short, 2 );

将CRC（校验和）附加到数据流串口通信C ++

1 个答案: