我需要将数据写入一个结构,其中数据的长度取决于我要发送到设备的命令。为此,我定义了以下结构:
typedef struct {
uint8 len; // Command length (cmd ... crc)
uint8 cmd; // Command code
uint8 data_length; // Data length
uint8 data[12]; // Data: max 12 Byte
uint8 crc_h; // CRC value MSB
uint8 crc_l; // CRC value LSB
}CMD_TYPE;
注意:始终存在的成员cmd、*数据长度*和crc,而成员数据可以为空或最多包含12个字节。
我创建了一个函数,该函数根据传递给函数的参数返回初始化的命令:
CMD_TYPE Device::get_cmd(uint8 cmd, uint8 data_len, uint8 *data)
{
CMD_TYPE cmd;
cmd.len = (4 + data_len) * sizeof(uint8);
cmd.cmd = cmd;
cmd.data_length = data_len;
cmd.data = (uint8 *)realloc(cmd.data, data_len*sizeof(uint8));
if(data_len > 0) memcpy(cmd.data, data, data_len);
add_crc16((uint8*)&cmd);
return cmd;
}
函数get_cmd()的用法如下:
uint8 cmd_code = 0x01;
uint8 data[2] = {0xAB, 0xCD};
CMD_TYPE cmd = local_device->get_cmd(cmd_code, 2, data);
retVal = local_device->send(cmd);
当我试图编译这段代码时,编译器会出错:
cmd.data = (uint8 *)realloc(cmd.data, data_len*sizeof(uint8));
编译器错误是:
error: lvalue required as left operand of assignment
使用realloc()的目的是重新调整数组数据的大小,或者从我的新命令结构中完全删除它我的密码怎么了?这是用动态内存分配初始化结构的正确方法吗?
最佳答案
你想要的是臭名昭著的
typedef struct
{
uint8 len; // Command length (cmd ... crc)
uint8 cmd; // Command code
uint8 data_length; // Data length
uint8 crc_h; // CRC value MSB
uint8 crc_l; // CRC value LSB
uint8 data[1]; // Data: max 12 Byte
} CMD_TYPE;
诀窍是为结构的所有成员分配足够的空间,直到
data[]
,然后为data[]
成员添加足够的字节:CMD_TYPE * allocCmd(int dataSize)
{
int len;
CMD_TYPE * p;
len = sizeof(CMD_TYPE) + (dataSize-1)*sizeof(uint8);
p = (CMD_TYPE *) malloc(len);
memset(p, 0, len);
p->data_length = dataSize;
return p;
}
这里,
len
被计算为结构的大小,减去空成员的大小,再加上data
为dataSize
数组指定的许多元素。问题是,您必须小心,不要访问
data
中实际分配给它的元素之外的任何元素(在结构内部)。