any012 发表于 2019-2-13 16:06:48

想将两个变量放到一个字节里去存储，结果出错了。

有两个变量需要存储，因为这两个变量取值范围都很小，是0-4，所以想将两个变量存到一个字节里，第一个变量存放到高字节，第二个变量存放到低字节。
于是写了个表达式，结果运算结果是错的，都是0...

adcWriteState = (AdcButton.curLevel<<4)&0xF0 + AdcButton.curLevel;
// adcWriteState = (AdcButton.curLevel<<4)&0xF0 + AdcButton.curLevel&0x0F;
// adcWriteState = (AdcButton.curLevel<<4)&0xF0;
//adcWriteState = AdcButton.curLevel&0x0F;
//adcWriteState = (uint8_t)((AdcButton.curLevel<<4)&0xF0) + (uint8_t)(AdcButton.curLevel&0x0F);


AdcButton是自己定义的结构体变量，而curLevel是结构体中的一个变量，类型是uint8_t；
第一个表达式的计算结果，高字节是正确的，低字节永远是0。
第二个表达式是一开始时按自己的思路写的，结果运行后高低字节都是0；
第三个表达式是为了验证问题，只保留高字节部分，结果也没问题；
第四个表达式，只保留低字节部分，结果也没问题。
第五个表达式，两个变量位移位与后，进行类型强制转换，再相加，结果正确。

又试了试高位部分不强制转换，低位部分强制转换，结果出错...

洛千叶 发表于 2019-2-13 16:06:49

struct Byte{
        unsigned char a:4;
        unsigned char b:4;
};
使用位域也可以

any012 发表于 2019-2-13 16:40:57

问题解决了，真惭愧...
加号的运算优先级高于位与。

stm1024 发表于 2019-2-13 16:53:02

本帖最后由 stm1024 于 2019-2-13 22:29 编辑

哈哈，很多人容易犯这样的错误，总以为逻辑运算，移位运算优先级高于加减运算。

any012 发表于 2019-2-13 16:59:39

唉，在这里卡了小半天。

Angel_YY 发表于 2019-2-13 18:44:53

表达式中有多个运算符的情况下，最好是使用括号明确优先级，不仅仅能够保证优先级的正确性，而且也便于别人阅读代码时能清晰理解。

dsjsjf 发表于 2019-2-13 18:48:35

我一般只要是2个及以上的运算符，都加括号，因为我搞不清哪个优先级高哪个优先级低

wudianjun2001 发表于 2019-2-14 10:18:11

括号能多就多

any012 发表于 2019-2-14 13:44:36

洛千叶 发表于 2019-2-14 12:03
struct Byte{
        unsigned char a:4;
        unsigned char b:4;


谢谢，第一次见到这种用法。学到了。
页: [1]
查看完整版本: 想将两个变量放到一个字节里去存储，结果出错了。