Need help to configure LED running by preesing switch

[krushna137]

Please help to correct program..
I want to run my LED ON sequence while pressing Pushbutton once LED ON sequentially and OFF... not run in loop
IN below program, problem is LEDs are ON without pressing pushbutton...
please help

const int BUTTON = 2;
const int LED1 = 3;
const int LED2 = 4;
const int LED3 = 5;
const int LED4 = 6;
const int LED5 = 7;
const int LED6 = 8;
const int LED7 = 9;
const int LED8 = 10;
const int LED9 = 11;
const int LED10 = 12;
const int LED11 = 13;
//#define B_PIN 2
int BUTTONstate = 0;

void setup()
{

pinMode(LED1, OUTPUT);
pinMode(LED2, OUTPUT);
pinMode(LED3, OUTPUT);
pinMode(LED4, OUTPUT);
pinMode(LED5, OUTPUT);
pinMode(LED6, OUTPUT);
pinMode(LED7, OUTPUT);
pinMode(LED8, OUTPUT);
pinMode(LED9, OUTPUT);
pinMode(LED11, OUTPUT);
pinMode(LED10, OUTPUT);
pinMode(BUTTONstate, INPUT);
}

void loop()
{
BUTTONstate = digitalRead(BUTTON);

if (BUTTONstate == HIGH)
{
delay(3000);
digitalWrite(LED1, HIGH);
delay(5000);
digitalWrite(LED2, HIGH);
delay(5000);
digitalWrite(LED3, HIGH);
delay(2000);
digitalWrite(LED4, HIGH);
delay(5000);
digitalWrite(LED5, HIGH);
delay(5000);
digitalWrite(LED6, HIGH);
delay(5000);
digitalWrite(LED7, HIGH);
delay(5000);
digitalWrite(LED8, HIGH);
delay(5000);
digitalWrite(LED9, HIGH);
delay(5000);
digitalWrite(LED11, HIGH);
delay(5000);
digitalWrite(LED10, HIGH);
delay(5000);

digitalWrite(LED1, LOW);
digitalWrite(LED2, LOW); 
digitalWrite(LED3, LOW); 
digitalWrite(LED4, LOW); 
digitalWrite(LED5, LOW); 
digitalWrite(LED6, LOW); 
digitalWrite(LED7, LOW); 
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED10, LOW);
digitalWrite(LED11, LOW);


digitalWrite(LED1, HIGH);
delay(500);
digitalWrite(LED1, LOW);
delay(500);
digitalWrite(LED1, HIGH);
delay(500);
digitalWrite(LED1, LOW);
delay(500);
digitalWrite(LED1, HIGH);
delay(500);
digitalWrite(LED1, LOW);
delay(500);
digitalWrite(LED1, HIGH);
delay(500);
digitalWrite(LED1, LOW);
delay(500);


digitalWrite(LED1, HIGH);
digitalWrite(LED8, LOW);
delay(500);
digitalWrite(LED1, LOW);
digitalWrite(LED8, HIGH);;
delay(500);
digitalWrite(LED1, HIGH);
digitalWrite(LED8, LOW);
delay(500);
digitalWrite(LED1, LOW);
digitalWrite(LED8, HIGH);;
delay(500);
digitalWrite(LED1, HIGH);
digitalWrite(LED8, LOW);
delay(500);
digitalWrite(LED1, LOW);
digitalWrite(LED8, HIGH);
delay(500);
digitalWrite(LED1, HIGH);



digitalWrite(LED8, HIGH);
digitalWrite(LED9, LOW);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, HIGH);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED11, HIGH);
delay(400);
digitalWrite(LED8, HIGH);
digitalWrite(LED9, LOW);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, HIGH);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED11, HIGH);
delay(400);
digitalWrite(LED8, HIGH);
digitalWrite(LED9, LOW);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, HIGH);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED11, HIGH);
delay(400);
digitalWrite(LED8, HIGH);
digitalWrite(LED9, LOW);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, HIGH);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED11, HIGH);
delay(400);
digitalWrite(LED8, HIGH);
digitalWrite(LED9, LOW);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, HIGH);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED11, HIGH);
delay(400);
digitalWrite(LED8, HIGH);
digitalWrite(LED9, LOW);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, HIGH);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED11, HIGH);
delay(400);
digitalWrite(LED8, HIGH);
digitalWrite(LED9, LOW);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, HIGH);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED11, HIGH);
delay(400);
digitalWrite(LED8, HIGH);
digitalWrite(LED9, LOW);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, HIGH);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED11, HIGH);
delay(400);
digitalWrite(LED8, HIGH);
digitalWrite(LED9, LOW);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, HIGH);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED11, HIGH);
delay(400);
digitalWrite(LED8, HIGH);
digitalWrite(LED9, LOW);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, HIGH);
digitalWrite(LED11, LOW);
delay(400);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED11, HIGH);
delay(400);
//digitalWrite(LED1, LOW);
//digitalWrite(LED2, LOW); 

/* digitalWrite(LED3, LOW);
digitalWrite(LED4, LOW);
digitalWrite(LED5, LOW);
digitalWrite(LED6, LOW);
digitalWrite(LED7, LOW);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED10, LOW);
digitalWrite(LED11, LOW);*/
// delay(5000000) ;
}
else{
digitalWrite(LED1, LOW);
digitalWrite(LED2, LOW);
digitalWrite(LED3, LOW);
digitalWrite(LED4, LOW);
digitalWrite(LED5, LOW);
digitalWrite(LED6, LOW);
digitalWrite(LED7, LOW);
digitalWrite(LED8, LOW);
digitalWrite(LED9, LOW);
digitalWrite(LED10, LOW);
digitalWrite(LED11, LOW);
}

}

[paulpaulson]

Hello krushna137

The delay() function is blocking the execution of the sketch.
Replace all calls of the delay() functions by timer function based on the BLINKWITHOUTDELAY example of the IDE.
Either you brew your own Led-sequencer or you ask the WWW for a solution.
Do you familar in coding in C++?
If yes, build data structures per Led sequence, containing port pin address and timing data etc., and some services to handle this information.

Have a nice day and enjoy coding in C++.

[Grumpy_Mike]

You might want to look at this How to get the best from this from this Forum before you proceed any further.

Your code is not formatted correctly the above link will tell you how to do it.

How is your button wired up? Is it using any external resistors?

@paulpaulson

Do you familar in coding in C++?

You can tell this is the work of someone who struggles to reach the heights of a beginner.

Once the button is pressed it will take more that 60 seconds before the button is looked at again.

That code is totally turgid, please learn how to use for loops and arrays.

[JCA34F]

Do you have a pulldown resistor (10k) connected from pin 2 to GND?

[TomGeorge]

Hi,
From previous threads we seem to have a failure to communicate.
@krushna137 what is your native language?

And for the umpteenth time;
To add code please click this link;

How to get the best out of this forum

See also FAQ - Arduino Forum for general rules on forum behaviour and etiquette. Hello, Welcome to the Arduino Forum. This guide explains how to get the best out of this forum. Please read and follow the instructions below. Being new here you might think this is having rules for the sake of rules, but that is not the case. If you don’t follow the guidelines all that happens is there is a long exchange of posts while we try to get you to tell us what we need in order to help you, which is fru…

Thanks... Tom..