whats wrong with this if clause

[cooper1200]

i have the following if statement

if (((temp_x - 2 >= 0 || temp_x - 2 <= 7) && (temp_y + 2 * multiplyer_y >= 0 || temp_y + 2 * multiplyer_y <=7)) &&
            ((temp_x + 2 >= 0 || temp_x + 2 <= 7) && (temp_y + 2 * multiplyer_y >= 0 || temp_y + 2 * multiplyer_y <=7)))
        {
            //record branch
            route[route_count][move][0] = 2;
            move += 1;
            //jump left
            route[route_count][move][0] = 0;
            temp_x -= 2;
            temp_y += 2 * multiplyer_y;
            move += 1;
        }

for some reason it is always true on debug temp_x got upto 32
coop

[jimblumberg]

So what are the values of the variables, temp_x, temp_y, and multiplyer_y?

What order do you think the following will be interpreted: temp_y + 2 * multiplyer_y?

[flp1969]

So what are the values of the variables, temp_x, temp_y, and multiplyer_y?

What order do you think the following will be interpreted: temp_y + 2 * multiplyer_y?

Doesn't matter, the entire expression is always true.

[cooper1200]

multiplyer_y is either 1 or -1 in this particular scenario it is 1
temp_x and temp_y should be between 0 and 7
temp_y + 2 * multiplyer_y should evaluate to either :
temp_y + 2 * -1 = temp_y -2 or
temp_y + 2 * 1= temp_y +2

[cooper1200]

i see my mistake i have || instead of && between the limits so im saying if temp is >= 0 equates true.... doesnt matter if temp_x is 0 or 500 billion where as i want to limit the values from 0 to 7

I have been staring at it for an hour as well duh!!!!