C 表达式必须是可修改的左值（我为什么要使用此表达式）

Question

我有这个带有 ?: 运算符的表达式：

(adc.Voltage1[Counter.ADC_ConversionCount] = ADC_readResult(Handler.myAdc, ADC_ResultNumber_1))  > 10 ? Counter.WriteOut = 1 : Counter.WriteOut = 0;

与 if-else 相同的表达式：

if((adc.Voltage1[Counter.ADC_ConversionCount] = ADC_readResult(Handler.myAdc, ADC_ResultNumber_1)) > 10 ){
    Counter.WriteOut = 1;
}else{
    Counter.WriteOut = 0;
}

为什么在第一种情况下会出现“表达式必须是可修改的左值”错误？

ADC_readResult 函数返回类型是 uint_least16_t。这是 Counter 结构体定义和 ADC 结构体定义：

typedef struct __COUNTERS__ {
    uint16_t WriteOut;
    uint16_t ADC_ConversionCount;
    uint16_t ADC_CycleCount;
    uint8_t LimitADCReached1;
    uint8_t LimitADCReached2;
    uint8_t LimitADCReached3;
    uint8_t LimitADCReached4;
    uint8_t LimitADCReached5;
} COUNTERS;

typedef struct __ADC_VOLTAGES__ {
    uint16_t Voltage1[ADC_VAL];
    uint16_t Voltage2[ADC_VAL];
    uint16_t Voltage3[ADC_VAL];
    uint16_t Voltage4[ADC_VAL];
    uint16_t Voltage5[ADC_VAL];

} ADC;

Answer 1

您遇到的错误与解析表达式的方式有关。

您的表达式（简化）如下所示：

(a = b) < 10 ? c = 1 : c = 0

三元运算符的?:优先级高于赋值运算符=。虽然内部=被视为三元的一部分，但最右边的不是。所以表达式解析如下：

((a = b) < 10 ? c = 1 : c) = 0;

结果是您试图将值 0 分配给不是左值的表达式，即变量名或取消引用的指针。你需要括号来解析你想要的方式：

((a = b) < 10) ? (c = 1) : (c = 0);

由于您正在做的是c根据表达式为其分配值，因此可以简化如下：

c = ((a = b) < 10) ?  1 : 0;

甚至：

c = ((a = b) < 10);

翻译回您的代码：

Counter.WriteOut = (adc.Voltage1[Counter.ADC_ConversionCount] = ADC_readResult(Handler.myAdc, ADC_ResultNumber_1) > 10);

并通过拆分操作使其更具可读性：

adc.Voltage1[Counter.ADC_ConversionCount] = ADC_readResult(Handler.myAdc, ADC_ResultNumber_1);
Counter.WriteOut = (adc.Voltage1[Counter.ADC_ConversionCount] > 10);