ESP8266 Community Forum

Posted: **Tue Nov 17, 2015 7:37 pm**

Hi, I can't get this bit of code to pass the first "if" check (it always returns nZone = 17), what am I doing wrong?

rLen = (random(6));  // random number 0->5
if  (rLen == 0 || 1 || 2 ){  // if equal to 0, 1 or 2
nZone = 17;  //  nZone = 17, if == 0, 1 or 2
}
else if  (rLen == 3 || 4){  // if equal to 3 or 4
nZone = 16;  //  nZone = 16, if == 3 or 4
}
else {
nZone = 15;  //  nZone = 15, if == 5 (or none of the above)
{

I've got round by using

Code: Select all

rLen = (random(6));
if  (rLen < 3 ){
nZone = 17;
}
else if  (rLen <5){
nZone = 16;
}
else {
nZone - 15;
{

Which is fine for this appilcation. But thought it should have worked.

Thanks for looking.

Posted: **Tue Nov 17, 2015 7:50 pm**

Should the if statement have multiple evaluations in it.

Code: Select allif (rLen == 0 || rLen ==1 ||rLen == 2 ){

Posted: **Wed Nov 18, 2015 3:39 am**

I would also consider using a switch / case statement: https://www.arduino.cc/en/Reference/SwitchCase

Posted: **Wed Nov 18, 2015 5:30 am**

Mmiscool wrote:Should the if statement have multiple evaluations in it.

Code: Select allif (rLen == 0 || rLen ==1 ||rLen == 2 ){

Yep thats it. To long sat in a darkened room brain gone.

eduperez wrote:I would also consider using a switch / case statement: https://www.arduino.cc/en/Reference/SwitchCase

Hi, thanks. As it sets a variable for the number of times a for loop circles (its a lighting effet that 50% of the time runs the full effect, 34% and 14% runs a shortened effect). Is there an advantage to switch case over "if and else if"? Or generaly a better way to do this?

Code: Select all

rLen = (random(6));
if  (rLen == 0 || rLen ==  1 || rLen ==  2 ){
nZone = 17;
}
else if  (rLen == 3 || rLen ==  4){
nZone = 16;
}
else {
nZone = 15;
{

for (int Zone = 12; Zone <= nZone; Zone++) {

ESP8266 Community Forum

if, else if, else with or (||) should this work?

if, else if, else with or (||) should this work?

Re: if, else if, else with or (||) should this work?

Re: if, else if, else with or (||) should this work?

Re: if, else if, else with or (||) should this work?