testing inputs while outputing

system · November 14, 2014, 9:14pm

Hey! Thanks in advance for all help and suggestions. Every idea is valuable. So, basically what I want be able to do is switch the mode (counter) while blinky() is running. I want to stop the blinky() and run case:2 by pushing select. This is just example code from a huge project I'm working on where blinky() would be really big. Do I need to use interrupts? And how would I do that. Thanks again for all the support, you guys are great!

int mode = 8;  //Mode selection button
int select = 7; //Select button
int counter = 0; //Counter # for modes
int LED1 = 9;
int LED2 = 10;
int LED3 = 11;
int LED4 = 12;

void setup() 
{

  pinMode(mode, INPUT_PULLUP);
  pinMode(select, INPUT_PULLUP);
  pinMode(LED1, OUTPUT);
  pinMode(LED2, OUTPUT);
  pinMode(LED3, OUTPUT);
  pinMode(LED4, OUTPUT);

}

/////////////////////////////////////////////////////////////////////////////////////
void loop() 
{
  int selectVal = digitalRead(select);
  int switchVal = digitalRead(mode);
  if(switchVal == LOW)
  {
    
    delay(500);
    counter ++;
    
   if(counter == 3)
    {
      counter = 1;
    }
  }
///////////////////////////////////////////  
  switch(counter)
  {
   case 1:   
      digitalWrite(LED4, LOW);
      digitalWrite(LED3, HIGH);
      if(selectVal == LOW)
      {
        blinky();  
      }
    break;
    
    
  case 2:   
    digitalWrite(LED3, LOW);
    digitalWrite(LED4, HIGH);
    
    break;


  }
}

////////////////////////////////////////////////////////////////////////////////////////////
// METHODS // METHODS // METHODS // METHODS // METHODS // METHODS // METHODS // METHODS ////
////////////////////////////////////////////////////////////////////////////////////////////

void blinky()
{
 digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
   digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
     digitalWrite(LED1, HIGH);
 delay(200);
 digitalWrite(LED1, LOW);
  digitalWrite(LED2, HIGH);
  delay(200);
  digitalWrite(LED2, LOW);
}

holmes4 · November 14, 2014, 9:16pm

read forum.arduino.cc/index.php?topic=223286.0

Mark

system · November 14, 2014, 9:17pm

Do I need to use interrupts?

No, you probably don't.
Much better would be better to get rid of the calls to delay()

Look at the blink without delay example in the IDE.

system · November 14, 2014, 9:26pm

I looked into using blink without delay, but I still would need multiple If statements to switch the counter (I assume). The bigger project that I am building is a synchronized x-mas lights to music. Each of the 4 cases will be a song, and each song is about 1500 lines. Is there a way to keep the delay and still switch the counter or is there no choice? I already have all the songs using delay. Thanks again!

system · November 14, 2014, 9:29pm

What do you think an interrupt is?
How do you think it will help you?

system · November 14, 2014, 9:33pm

My original idea was to have the select button as an interrupt and trigger to run a ISR to exit the loop and switch the counter. I wasn't able to find any examples of people doing this, or even if I'm understanding it appropriately. Please correct me if I'm wrong. I'm here to learn.

system · November 14, 2014, 9:38pm

You're sitting reading a book.
The phone rings, so you go to answer it.
You put the phone down and go back to baking your cake.

Does that sound like an interrupt?

Grumpy_Mike · November 14, 2014, 9:41pm

I wasn't able to find any examples of people doing this,

Not surprising.

or even if I'm understanding it appropriately

You are not.

An interrupt service routine can only interact with code in the main program by changing the value of variables used in both. The ISR can not control or divert the flow of another piece of code.

system · November 14, 2014, 9:43pm

In programming terms, no. It would be a simple test for the ring. Could I then place the test outside of the blinky() in this example?

system · November 14, 2014, 9:46pm

Yes, but the latency because of all the calls to delay() would be horrific.

system · November 14, 2014, 9:49pm

So basically with delay() I would have to wait for the method blinky() to finish before I could switch the mode. However if I use blink without delay, I could very easily interrupt it?

holmes4 · November 14, 2014, 9:55pm

Yes and the thread you where pointed at shows how to doit! So READ IT!

Mark

system · November 14, 2014, 10:03pm

Mark,
Thanks for the link. It does cover how to do that properly and it is what I will have to follow. I was more just curious if there was a way around it still using delay. I now understand that it cant be simply done. Thanks for your help!
Jake

system · November 14, 2014, 10:08pm

It's not that it can't be done, it's just that it's a lot easier to do it ~~properly~~appropriately.

system · November 14, 2014, 10:20pm

The function blinky() below does exactly what you want without delay() other than adding other pins.. you could add a counter and loop through until the song is over..

unsigned long dtime = 200;
int LED1 = 13;
unsigned long starttime = millis();
boolean offon =true;

  void setup() 
{
  pinMode(LED1, OUTPUT);
}
void loop(){
  
  Blinky();
  
}
void Blinky(){
  
  if (millis() - starttime >= dtime){
    if(offon){
      digitalWrite(LED1, LOW);
      offon = false;
    }else{
      
       digitalWrite(LED1, HIGH);
       offon = true;
    }
    starttime = starttime + dtime;
  }
}

system · November 14, 2014, 10:25pm

I was just looking for a way to still use delay, because I have three songs done. Each song is about 33,000 bytes big and around 3000 lines and they all use delay. Next time I will use blink without delay. But for now I will just have to make do. Thanks
Jake

system · November 14, 2014, 10:28pm

Thank you for the example, that is what I will have to do next time. As of now I have hundreds of different delay times. I will know next time. Thanks kycountry!

system · November 14, 2014, 10:31pm

Thanks for the example kycountry! But as of now I have hundreds of different delay times. I will probably have to change them to match your example in the future though. Thank you!
Jake

system · November 14, 2014, 10:32pm

Thanks to all who have helped so far! I have learned a lot.

Topic		Replies	Views
Breaking out of attachInterrupt void during countdown General Guidance	38	213	January 3, 2026
Latest updated delay vs timer demo Programming	15	236	December 18, 2025
Blink LED 2 times with millis() and then restar cycle Programming	12	241	July 21, 2024
IO program control Programming	27	520	December 8, 2024
Interrupts / Delay Programming	24	5854	May 5, 2021

testing inputs while outputing

Related topics