I am new to Arduino programming.
Can anybody please help me simplify my code,
retaining the original functions exactly as intended,
Thank you in advance.
Kindly provide an alternative code
int buttonPinLeft = 2; // Left pushbutton pin
int buttonPinRight = 3; // Right pushbutton pin
int buttonPinStop = 4; // Right pushbutton pin
int ledPin1 = 8; // the number of the LED pin
int ledPin2 = 9; // the number of the LED pin
int ledPin3 = 10; // the number of the LED pin
int ledPin4 = 11; // the number of the LED pin
int ledPin5 = 12; // the number of the LED pin
int ledPins[] = {8, 9, 10, 11, 12}; // variables will change:
int buttonState = 0; // variable for reading the pushbutton status
void setup(){
pinMode(ledPin1, OUTPUT);
pinMode(ledPin2, OUTPUT);
pinMode(ledPin3, OUTPUT);
pinMode(ledPin4, OUTPUT);
pinMode(ledPin5, OUTPUT);
// initialize the pushbutton pins as inputs:
pinMode(buttonPinRight, INPUT);
pinMode(buttonPinLeft, INPUT);
}
void loop(){
buttonState = digitalRead(buttonPinRight);
// check if the pushbutton is pressed. If it is, the buttonState is HIGH:
if (buttonState == HIGH) {
// turns on Right Direction LED thrice then turns off after button is released:
digitalWrite(ledPin1, HIGH);
delay(100);
digitalWrite(ledPin2, HIGH);
delay(100);
digitalWrite(ledPin3, HIGH);
delay(100);
digitalWrite(ledPin4, HIGH);
delay(100);
digitalWrite(ledPin5, HIGH);
delay(100);
digitalWrite(ledPin1, LOW);
digitalWrite(ledPin2, LOW);
digitalWrite(ledPin3, LOW);
digitalWrite(ledPin4, LOW);
digitalWrite(ledPin5, LOW);
delay(100);
digitalWrite(ledPin1, HIGH);
delay(100);
digitalWrite(ledPin2, HIGH);
delay(100);
digitalWrite(ledPin3, HIGH);
delay(100);
digitalWrite(ledPin4, HIGH);
delay(100);
digitalWrite(ledPin5, HIGH);
delay(100);
digitalWrite(ledPin1, LOW);
digitalWrite(ledPin2, LOW);
digitalWrite(ledPin3, LOW);
digitalWrite(ledPin4, LOW);
digitalWrite(ledPin5, LOW);
delay(100);
digitalWrite(ledPin1, HIGH);
delay(100);
digitalWrite(ledPin2, HIGH);
delay(100);
digitalWrite(ledPin3, HIGH);
delay(100);
digitalWrite(ledPin4, HIGH);
delay(100);
digitalWrite(ledPin5, HIGH);
delay(100);
digitalWrite(ledPin1, LOW);
digitalWrite(ledPin2, LOW);
digitalWrite(ledPin3, LOW);
digitalWrite(ledPin4, LOW);
digitalWrite(ledPin5, LOW);
delay(100);
}
{
buttonState = digitalRead(buttonPinLeft);
// check if the pushbutton is pressed. If it is, the buttonState is HIGH:
if (buttonState == HIGH) {
// turns on Left Direction LED thrice then turns off after button is released:
digitalWrite(ledPin5, HIGH);
delay(100);
digitalWrite(ledPin4, HIGH);
delay(100);
digitalWrite(ledPin3, HIGH);
delay(100);
digitalWrite(ledPin2, HIGH);
delay(100);
digitalWrite(ledPin1, HIGH);
delay(100);
digitalWrite(ledPin1, LOW);
digitalWrite(ledPin2, LOW);
digitalWrite(ledPin3, LOW);
digitalWrite(ledPin4, LOW);
digitalWrite(ledPin5, LOW);
delay(100);
digitalWrite(ledPin5, HIGH);
delay(100);
digitalWrite(ledPin4, HIGH);
delay(100);
digitalWrite(ledPin3, HIGH);
delay(100);
digitalWrite(ledPin2, HIGH);
delay(100);
digitalWrite(ledPin1, HIGH);
delay(100);
digitalWrite(ledPin1, LOW);
digitalWrite(ledPin2, LOW);
digitalWrite(ledPin3, LOW);
digitalWrite(ledPin4, LOW);
digitalWrite(ledPin5, LOW);
delay(100);
digitalWrite(ledPin5, HIGH);
delay(100);
digitalWrite(ledPin4, HIGH);
delay(100);
digitalWrite(ledPin3, HIGH);
delay(100);
digitalWrite(ledPin2, HIGH);
delay(100);
digitalWrite(ledPin1, HIGH);
delay(100);
digitalWrite(ledPin1, LOW);
digitalWrite(ledPin2, LOW);
digitalWrite(ledPin3, LOW);
digitalWrite(ledPin4, LOW);
digitalWrite(ledPin5, LOW);
delay(100);
}
{
buttonState = digitalRead(buttonPinStop);
// check if the pushbutton is pressed. If it is, the buttonState is HIGH:
if (buttonState == HIGH) {
// turn LEDs on Stop
digitalWrite(ledPin5, HIGH);
digitalWrite(ledPin4, HIGH);
digitalWrite(ledPin3, HIGH);
digitalWrite(ledPin2, HIGH);
digitalWrite(ledPin1, HIGH);
} else
{
buttonState = digitalRead(buttonPinStop);
// check if the pushbutton is pressed. If it is, the buttonState is HIGH:
if (buttonState == LOW) {
// turn LEDs on Stop
digitalWrite(ledPin5, LOW);
digitalWrite(ledPin4, LOW);
digitalWrite(ledPin3, LOW);
digitalWrite(ledPin2, LOW);
digitalWrite(ledPin1, LOW);
}
// I commented out this code line
// because it interferes with the
// rest of the code, which works
// fine with out this line of code.
// Basically it flashes the LEDs,
// but when applied to the program
// the flashing sequence affects
// the directional behavior of the LEDs.
// FLASH LEDs
//digitalWrite(ledPin1, HIGH);
//digitalWrite(ledPin2, HIGH);
//digitalWrite(ledPin3, HIGH);
//digitalWrite(ledPin4, HIGH);
//digitalWrite(ledPin5, HIGH);
//delay(100);
//digitalWrite(ledPin1, LOW);
//digitalWrite(ledPin2, LOW);
//digitalWrite(ledPin3, LOW);
//digitalWrite(ledPin4, LOW);
//digitalWrite(ledPin5, LOW);
//delay(100);
}
}
}
}
