I have another strange error (again!)!!
I have an array of longs and I have to find two numbers: 1170000000 or 1180000000.
When I found one of these two numbers (the first or the second, it's the same) I have to break the loop.
I wrote
while (instr[EIP] != 1170000000 || instr[EIP] != 118000000)
 EIP++;
But it continues after it has found the numbers. I put a serial.print instruction just after the while declaration to print all the numbers it find and I found 1170000000! It is a strange comportament!
I'm very confused...
Essentially you have an infinite loop because the variable cannot simultaneously be both of those numbers. Your current logic says, "if either of those conditions is fulfilled, the loop will continue", whereas you think it says, "if either of those conditions is NOT fulfilled, the loop will NOT continue." Both conditions would have to be fulfilled to break the loop. Change || to && and your problem is solved. New logic says, "Both of these conditions must be fulfilled or else the loop will break."
While (true)
{
if (instr[EIP] == 1170000000 || instr[EIP] == 1180000000)
Break;
EIP++;
}
cyperrage: you're wrong: if I put && instead of || both of the condition have to be fulfilled, only one have to be fulfilled. Anyway: the same variable can't have two different values at the same time.
You want both to be fulfilled in order to keep incrementing. If either one is fulfilled youwant to stop incrementing.
So if you find 11700000 then the while loop says while (false or true) since 1170000 doesn't equal 1180000 so the while loop keeps incrementing. You say you want it to stop incrementing if you get either one of those numbers. So you should use && in the original code and it will work. That way it says only keep incrementing if the number is not 1170000 or 1180000. If it is either one of those numbers then the while condition comes back false and it stops incrementing.
You're thing if it equals this OR that, but what you've written is if it is NOT equal to this or NOT equal to that. That's always true, a number can't equal two things at once.
cyperrage: you're wrong: if I put && instead of || both of the condition have to be fulfilled, only one have to be fulfilled. Anyway: the same variable can't have two different values at the same time.
Thanks a lot!
Jymmy097
Simply put: No.
Loops work based on the following properties:
Whenever the logic inside the parentheses is TRUE, the loop continues.
Whenever the logic inside the parentheses is FALSE, the loop breaks.
Let me lay out some Logical operator tables for you. (&& = logical AND, || = logical OR)
A | B | AND | OR
FALSE | FALSE | FALSE | FALSE
FALSE | TRUE | FALSE | TRUE
TRUE | FALSE | FALSE | TRUE
TRUE | TRUE | TRUE | TRUE
Right now, you are using OR, therefore, the loop will ONLY break when the FALSE condition comes up, which only happens when both conditions come up false, which is impossible granted your logic. If you use the AND operator, it is obvious to see that when either condition breaks, the loop will break as well.
Also, as stated above, if you change the != to ==, then your logic will work fine, you just have to change either one, != to ==, or || to &&, the logic will be the same regardless, but you must do either one if you want it to be correct.
Attached is a simple program that demos what I think jymmy097 is trying to do when he says:
When I found one of these two numbers (the first or the second, it's the same) I have to break the loop.
unsigned long instr[] = { 0L, 1L, 2L, 3L, 1170000000L, 0L};
void setup() {
 Serial.begin(9600);
}
void loop() {
 int EIP = 0;
Â
 while (true) {
  if (instr[EIP] == 1170000000L || instr[EIP] == 118000000L) {
   Serial.print("Match: EIP = ");
   Serial.println(EIP);
    break;
  }
  Serial.print("No match: EIP = ");
  Serial.println(EIP);
  EIP++;
 }
}
That's sort of the long way round but yes if you want to use == then you can use OR. If you want to use != like in the OP then you'd use AND. And you get simpler code that way.
Jymmy097:
cyperrage: you're wrong: if I put && instead of || both of the condition have to be fulfilled, only one have to be fulfilled. Anyway: the same variable can't have two different values at the same time.
No. Cyperrage is correct.
Lets say your numbers are 117 and 118 for simplicity. With ||...
Input 116. The first condition is true (not 117) and the second condition is true (not 118).
true || true = true
Input 117. The first condition is false (= 117) and the second condition is true (not 118).
false || true = true
Input 118. The first condition is true (not 117) and the second condition is false (=118).
true || false = true
With ANY input the while loop will continue and not exit. For the loop to exit, both inputs would have to be false (equal to both 117 and 118), and as you say, the same variable can't have two different values at the same time.
Now with &&...
Input 116. The first condition is true (not 117) and the second condition is true (not 118).
true && true = true
Input 117. The first condition is false (=117) and the second condition is true (not 118).
false && true = false
Input 118. The first condition is true (not 117) and the second condition is false (=118).
true && false = false
If either input gives a false (equals either 117 or 118) then the while loop will exit.
cyperrage: you're wrong: if I put && instead of || both of the condition have to be fulfilled, only one have to be fulfilled. Anyway: the same variable can't have two different values at the same time.
You changed != to == which reverses the logic (see first post).
Something can't be green AND blue, that's true, but it can be not green AND not blue (eg. it could be red).
However it is alwaysnot green OR not blue (if it is green it is not blue, and if it is blue it is not green).