next topic »

[on:]

hay.. there is function to restart looping

example..

Code:

int val = analogRead(1);

void loop()
{
digitalWrite(led, HIGH);
delay(1000)
digitalWrite(led, LOW);
delay(1000)
digitalWrite(led2, HIGH);
delay(1000)
digitalWrite(led2, LOW);
if (val == 5)               <============= if val = 5 restart looping from begining
{   restart looping from begining      }
delay(1000)
digitalWrite(led3, HIGH);
delay(1000)
digitalWrite(led3, LOW);
delay(1000)
}

there is function to do it?

[Reply #1 on:]

It is ugly and not recommended but you could use a goto statement. Always better to restructure your code so that the condition that causes the looping is in the loop statement, if that makes sense.

[Reply #2 on:]

Code:

return;

[Reply #3 on:]

Couldn't you just say

if(val !=5){

// all the other stuff

}

[Reply #4 on:]

int val = analogRead(1);

Is a bad idea.  The hardware has not been initialized at the point analogRead is called.  Move the initialization to setup where it belongs.

[Reply #5 on:]

Code:

void loop() {
int loopVal;

while (1){
loopVal=1;

while (analogRead(1)!= 5) {
switch(loopVal) {
case 1:
digitalWrite(led, HIGH);
loopVal++;
case 2:
digitalWrite(led, LOW);
loopVal++;
case 3:
digitalWrite(led2, HIGH);
loopVal++;
case 4:
digitalWrite(led2, LOW);
loopVal++;
case 5:
digitalWrite(led3, HIGH);
loopVal++;
case 6:
digitalWrite(led3, LOW);
loopVal=1;
}
delay(1000);
}
}
}

[Reply #6 on:]

@Arrch

yeah the function return; is work..

thanks for all

[Reply #7 on:]

 inner loop can be shortened by using bitmasks.

Code:

    while (analogRead(1) != 5) {

switch(loopVal) {
case 1:
digitalWrite(led, HIGH);
loopVal++;
case 2:
digitalWrite(led, LOW);
loopVal++;
case 3:
digitalWrite(led2, HIGH);
loopVal++;
case 4:
digitalWrite(led2, LOW);
loopVal++;
case 5:
digitalWrite(led3, HIGH);
loopVal++;
case 6:
digitalWrite(led3, LOW);
loopVal=1;
}
delay(1000);
}

Code:

while (analogRead(1) != 5)
{
  if (loopVal & 0x01) digitalWrite(led, HIGH);
  else digitalWrite(led, LOW);
  loopVal++;
  if (loopVal == 6 ) loopVal=1;
  delay(1000);
}

[Reply #8 on:]

Code:

(analogRead(1) != 5)

Am I the only one who thinks this is a really bad idea?

[Reply #9 on:]

yeah 5 is such a magic number, one should use a #define for that

Code:

#define MAGICNUMBER 5
...

(analogRead(1) != MAGICNUMBER )

but then, it might be just what the OP intended ...

[Reply #10 on:]

No, not the magicness, just the precision, from an analogue input.