Sorry to resurrect a very old post. But I too have run into a situation where an entire else condition block was not executed. It's as if the compiler has optimized it away? But that doesn't make much sense to me.
In my case, I have braces in my else block, because I'm executing compound statements. But I did not have braces in my if / else if preceding blocks. Here's what I did initially:
if (fVoltage < vSettingLo || fVoltage > vSettingHi)
if (++unitRetry[slot] >= MAX_UNIT_TRIES)
{
GoStopUnit(bank,unit,VFAIL); // now fail the unit
unitRetry[slot] = 0; // reset the retry counter
}
else if (fCurrent < iSettingLo || fCurrent > iSettingHi)
if (++unitRetry[slot] >= MAX_UNIT_TRIES)
{
GoStopUnit(bank,unit,IFAIL); // now fail the unit
unitRetry[slot] = 0; // reset the retry counter
}
else
{
// update the meters
SetCtrlAttribute(panel,GetCtrlArrayItem(serialArray,slot),ATTR_TEXT_COLOR,VAL_DK_GREEN);
SetCtrlAttribute(panel,GetCtrlArrayItem(snArray,slot),ATTR_TEXT_COLOR,VAL_DK_GREEN);
SetCtrlAttribute(panel,GetCtrlArrayItem(fvoltageArray,slot),ATTR_TEXT_BGCOLOR,VAL_DK_GREEN);
SetCtrlAttribute(panel,GetCtrlArrayItem(fcurrentArray,slot),ATTR_TEXT_BGCOLOR,VAL_DK_GREEN);
}
In my view, since the if and the else if conditions on executed one line (a nested if block), that shouldn't have affected my last else block. But it did! This is repeatable. The last else block gets ignored. The solution is to put braces around the nested if blocks in the first if and in the next else if conditions. Then my last else gets left in by the compiler.
Odd!
