Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

While with multiple conditions

Tags:

c

Can somebody please explain why a while statement like

while (ch != '\n' || ch != '\t' || ch != ' ') { ... } 

does not work as I expected?

like image 403
lego69 Avatar asked May 01 '10 20:05

lego69


2 Answers

UPDATE: As per your other comment, your expression is wrong - it has nothing to do with "while" having multiple conditions.

ch != '\n' || ch != ' ' is ALWAYS true, no matter what the characters is.

If the character is NOT a space, the second condition is true so the OR is true.

If the character is a space, the first condition is true (since space is not newline) and the OR is true.

The correct way is ch != '\n' && ch != ' ' ...

OLD answer:

Under normal circumstances there's no problem whatsoever with the expression above (assuming you wanted to do just that).

The only issue with yours is that it can sometimes be less than optimal (e.g. if b and c never change throughout the loop, in which case you need to cache the value of b!=1 in a variable).

while with multiple conditions may have a problem in one case - if those multiple conditions actually have intended side effects.

That is due to lazy evaluation of || and && in C, so that if the first expression is true, the rest will NOT be evaluated and thus their side effects will not happen.

like image 175
DVK Avatar answered Oct 03 '22 00:10

DVK


You should be careful when using not in boolean expressions that you don't get the AND and OR mixed up. Read about De Morgan's Laws. Often it is easier to read with only one negative. Applying De Morgan's Law to your expression gives this:

while (!(ch=='\r' && ch=='\n' && ch==' '))

If you had written it in this form you will hopefully immediate notice that (ch == '\r' && ch == '\n') can never be true.

The solution is to change this:

while (ch != '\n' || ch != '\t' || ch != ' ')

into this:

while (!(ch == '\n' || ch == '\t' || ch == ' '))

You can read it as "While we don't have \n or \t or space, do this...". Note that "while not" is similar to "until" in English (and some programming languages), so you can also read it as "Until we have \n or \t or space, do this".

like image 40
Mark Byers Avatar answered Oct 03 '22 00:10

Mark Byers