C语言拼接uint8_t为有符号16位值累加的符号位处理问题

核心结论 #

你额外增加整体int16_t强转的第二种写法是更严谨、全平台通用的正确实现，第一种写法存在跨平台兼容性风险。 #

原理说明 #

1. 第一种写法的潜在问题 #

第一种写法中的表达式((int16_t)dataReceived[3] << 8) | dataReceived[4]的最终类型由C语言的整数提升规则决定，和编译器默认的int宽度直接相关：

• 针对你当前使用的PIC24F平台，XC16编译器默认int为16位：该表达式的运算结果会是int16_t类型，符号位会被正确识别，赋值到int32_t变量时会自动做符号扩展，你担心的-1（对应16位值0xFFFF）会被正确扩展为0xFFFFFFFF，运算结果符合预期。
• 如果后续你将代码移植到int为32位的平台：(int16_t)dataReceived[3] << 8会被提前提升为32位有符号整数，运算结果为正数0x0000XX00，和uint8_t类型的dataReceived[4]位或后得到的是32位正数0x0000XXXX，哪怕16位位置的最高位为1也不会被识别为负数，加到int32_t变量时会被当做65535而非-1，直接出现计算错误。

2. 第二种写法的正确性 #

你对外层拼接结果增加(int16_t)强转的写法，相当于明确告知编译器：无论中间运算的临时类型是什么，最终这个16位数值是有符号类型，符号位以16位的最高位为准。
这种写法下，无论平台默认int是16位还是32位，拼接后的结果都会先被转换为正确的int16_t有符号值，再累加到int32_t变量时会自动完成符号扩展，完全符合你的设计预期。 #

可选优化方案 #

你还可以用联合体拼接的写法规避移位运算的潜在符号问题，可读性也更高：

union {
    uint8_t bytes[2];
    int16_t value;
} sensor_val;
// 注意字节顺序要和传感器输出的大小端匹配
sensor_val.bytes[1] = dataReceived[3];
sensor_val.bytes[0] = dataReceived[4];
val_x += sensor_val.value;

内容的提问来源于stack exchange，提问作者jjstcool