Which are the most common pitfalls with conditional (if) statements in C++? [closed]

Answer 1

Misconceptions of conditional expressions

• if(x = 1) // ...
  

  Equality comparisons are expressed using the ==. = is an assignment, and the result is evaluated as a cast to bool. I.e. any value evaluated to != 0 results in true. As a prevention mechanism consider the below expressions:

•    if(1 = x)  // invalid assignment compilation error!
     if(1 == x) // valid equality comparison
  

  A wrong use of an assignment operator can be avoided by always placing the constant on the left hand side of the expression. The compiler will flag any mistake triggering an invalid assignment error.

• if(answer == 'y' || 'Y')
  

  _{Variations: if(answer == 'y','Y')}
  Conditions must be tested with separate comparisons. The || operator binding doesn't do what's expected here. Use if(answer == 'y' || answer == 'Y')instead.

• if (0 < x < 42)  
  

  Valid syntax in Python, with expected behaviour, that syntax is valid in C++, but parsed as if ((0 < x) < 42) so false/true converted to 0/1 and then tested against < 42 -> always true. Condition must be tested with separate comparisons: if (0 < x && x < 42)