If-else chain not working properly.

RE: If-else chain not working properly.

adragocelestica — Wed, 18 Apr 2018 06:20:58 GMT

You are right. I have edited the version of the code that still worked like that, replaced the & with &&, and it works. Thank you very much for your help.

RE: If-else chain not working properly.

hmolesworth — Tue, 17 Apr 2018 14:55:11 GMT

It looks like the c syntax is incorrect. '&' is a bit-wise and, not the '&&' logical and that this test requires. Instead of this:

if((command[0]==108)&(command[1]==101)&(command[2]==100)&(command_length==3))//led

Try this:

if((command[0]==108)&&(command[1]==101)&&(command[2]==100)&&(command_length==3))//led

Then the original code may work better

RE: If-else chain not working properly.

adragocelestica — Tue, 17 Apr 2018 05:15:17 GMT

Hi,

Thanks for the quick response.

The array is declared in the main loop, and passed as a pointer to this function. What is quite bothering is that, if above all that, i put this:

if(condition5){
    printf("found condition5");
}

It just works and executes the printf. And the first 4 conditions on the if-else chain work properly. That's why I discarded that there may be any issue in the declaration of condition5 or the access to the array command[].

Also, condition1 is the one using more elements of the array command[] (7 elements), and that one does work properly, so I discarded that there may be any issue of access to the array.

Finally, I have found a workaround by doing first the following:

uint8_t index=0;
for (uint8_t i=0; i<command_length; i++){
	index += sprintf(&command_string[index], "%c", command[i]);
}

And then I have the info in a string, so I re-write the if conditions like this:

if(strcmp(command_string, "led")==0){
    //do stuff
}

Now it works properly with all conditions.

I don't know why it "likes" the string compare better, but for now I will go with it. If you want, I can try to declare the array as volatile and see what happens in the code that doesn't work properly, perhaps just to figure out why it was not working and to prevent similar things from happening in the future. One of my colleagues has the theory that it is an optimizer thing, and during compilation it "decides" that the fifth if will never happen, so it just blows it away. It makes sense, because when debugging that if statement is not implemented in the assembly code.

Best regards,

Alin O. Dragomir

RE: If-else chain not working properly.

Håkon Alseth — Mon, 16 Apr 2018 07:42:33 GMT

Hi,

Is your array written from an interrupt context?

I suspect that you need to declare the command[] array as volatile.

Could you try this and report back?

Best regards,

Håkon