toggle two leds not happening

Projects Programming
1 
const int ledPin1    = 12;
const int ledPin2    = 11;
const int ledPin3    = 10;

const int buttonPin1 =  2;
const int buttonPin2 =  3;
const int buttonPin3 =  4;
const int buttonPin4 =  5;


int ledState1    =  -1;
int ledState2    =  -1;
int ledState3    =  -1;

int buttonState1 = LOW;
int buttonState2 = LOW;
int buttonState3 = LOW;
int buttonState4 = LOW;


long lastDebounceTime = 0;
long debounceDelay    = 200;


void setup(){
 pinMode  (ledPin1, OUTPUT);
 pinMode  (ledPin2, OUTPUT);
 pinMode  (ledPin3, OUTPUT);
 
 pinMode (buttonPin1, INPUT);
 pinMode (buttonPin2, INPUT);
 pinMode (buttonPin3, INPUT);
 pinMode (buttonPin4, INPUT);
}
 
  void loop(){
    
  buttonState1 = digitalRead(buttonPin1);
  buttonState2 = digitalRead(buttonPin2);
  buttonState3 = digitalRead(buttonPin3);
  buttonState4 = digitalRead(buttonPin4);
  
  
  
  
  if( (buttonState1 == HIGH) && (ledState1 < 0)  ){
   
  digitalWrite(ledPin1,HIGH);
  digitalWrite(ledPin2,LOW);
  digitalWrite(ledPin3,LOW);

  
  ledState1 = -ledState1;
  
  lastDebounceTime = millis();
}
  else 
  
  if( (buttonState2 == HIGH) && (ledState2 < 0)  ){
   
  digitalWrite(ledPin2,HIGH);
  digitalWrite(ledPin1,LOW);
  digitalWrite(ledPin3,LOW);
  
  ledState2 = -ledState2;
  
  lastDebounceTime = millis();
}
  else
   
  if( (buttonState3 == HIGH) && (ledState3 < 0)  ){
   
  digitalWrite(ledPin3,HIGH);
  digitalWrite(ledPin2,LOW);
  digitalWrite(ledPin1,LOW);
  
  ledState3 = -ledState3;
  
  lastDebounceTime = millis();
  }
   else
   
  if( (buttonState3 == HIGH) && (ledState3 > 0)  ){
   
  digitalWrite(ledPin3,HIGH);
  digitalWrite(ledPin2,LOW);
  digitalWrite(ledPin1,LOW);
  
  ledState3 = ledState3;
  
  lastDebounceTime = millis();
  }
   else
   
  if( (buttonState2 == HIGH) && (ledState2 > 0)  ){
   
  digitalWrite(ledPin2,HIGH);
  digitalWrite(ledPin3,LOW);
  digitalWrite(ledPin1,LOW);
  
  ledState2 = ledState2;
  
  lastDebounceTime = millis();
  }
   else
   
  if( (buttonState1 == HIGH) && (ledState1 > 0)  ){
   
  digitalWrite(ledPin1,HIGH);
  digitalWrite(ledPin2,LOW);
  digitalWrite(ledPin3,LOW);
  
  ledState1 = ledState1;
  
  lastDebounceTime = millis();
  
  }
  else
  
  if( (buttonState3 == LOW) && (ledState3 < 0)  ){
   
  digitalWrite(ledPin3,HIGH);
  digitalWrite(ledPin2,LOW);
  digitalWrite(ledPin1,LOW);
  
  ledState3 = -ledState3;
  
  lastDebounceTime = millis();

  }
  
  else
  
  if( (buttonState4 == HIGH)&& (ledState2 <0)  ){
   
  digitalWrite(ledPin3,LOW);
  digitalWrite(ledPin2,HIGH);
  digitalWrite(ledPin1,LOW);
  
  ledState2 = -ledState1;
  
  lastDebounceTime = millis();
  }
  else
  
  if( (buttonState4 == LOW)&& (ledState2 >0)  ){
   
  digitalWrite(ledPin3,LOW);
  digitalWrite(ledPin2,HIGH);
  digitalWrite(ledPin1,LOW);
  
  ledState2 = -ledState2;
  
  lastDebounceTime = millis();
  
  }
  

  
  
  }

Hi all i am trying to make this code since a while now.I have 3 leds and 4 switches.I managed to get the 3 leds working with switch 1/2/3 upwards and backwards and the 4th switch which is suppose to toggle led 1 and led 2 does not work.Kindly help.Thanks.

[/code]

2

You need to describe more precisely how the LEDs are supposed to behave.

You have partial support for debouncing in the code, but you aren't using the debounce
time variable - get this working first with a single button to prove its working, then
extend to four buttons. You are probably only wanting to respond to changes in
button state, not the button state itself.

3

When the unit comes on LED 3 LIGHTS UP

When switch 2 is pressed LED2 COMES ON.
When switch 3 is pressed LED3 COMES ON.
When switch 1 is pressed again LED1 COMES ON and back and forth.

When switch 4 is pressed LED1 comes on but when its pressed again LED2 is suppose to come on but nothing happens.Led 1 stays on continuosly until Switch 1/2/3 is pressed and goes back to the same sequence.

4 
ledState3 = ledState3;
ledState2 = ledState2;
ledState1 = ledState1;

What do those lines of code achieve, apart from "let water be wet."

ledState2 = -ledState1;

I think this may be a typo.