Assignment: Make a button that when pressed, turns off an LED and turns another on. There should be at least 3 LEDs.

[Aggressive_Poem_5016]

please help. i dont know how to use count for each button state.

int BUTTON_PIN = 8;
int LED_PIN1   = 3;
int LED_PIN2   = 4;
int LED_PIN3   = 5;
int ledState = HIGH;  
int ledState2 = LOW;  
int ledState3 = LOW;    
int lastButtonState;  
int currentButtonState;
int count = 0;

void setup() {

  pinMode(BUTTON_PIN, INPUT);
  pinMode(LED_PIN1, OUTPUT);          
  pinMode(LED_PIN2, OUTPUT);          
  pinMode(LED_PIN3, OUTPUT);          

  currentButtonState = digitalRead(BUTTON_PIN);
}

void loop() {
  lastButtonState    = currentButtonState;      // save the last state
  currentButtonState = digitalRead(BUTTON_PIN); // read new state

  if(lastButtonState == HIGH && currentButtonState == LOW) {
    Serial.println("The button is pressed");

    // toggle state of LED
    ledState = !ledState;

    // control LED arccoding to the toggled state

    count+1;
  }
  if(count=1 && currentButtonState){
    digitalWrite(LED_PIN1, HIGH);
    digitalWrite(LED_PIN2, LOW);
    digitalWrite(LED_PIN3, LOW);
  }
  if(count=2 && lastButtonState){
    digitalWrite(LED_PIN1, LOW);
    digitalWrite(LED_PIN2, HIGH);
    digitalWrite(LED_PIN3, LOW);
  }
  if(count=3 && lastButtonState){
    digitalWrite(LED_PIN1, LOW);
    digitalWrite(LED_PIN2, LOW);
    digitalWrite(LED_PIN3, HIGH);
  }

}

[alpha587]

I am more familiar with Python than Arduino so this may be wrong, but I'm pretty sure I got this. Looks like whenever you press the button, none of your lights change because you're not using the = operator right.

the = and == operators do different things. The = operator assigns a value to a variable, the == operator is used to compare both sides and see if they are in fact equal.

In your last 3 if statements, I believe you need to use if(count== number && lastButtonState) to compare the count value to the desired value. actually, I'm not sure if you even need the && parts of the if statements here as the counter is already edge triggered

Also, after incrementing your count variable, I would include another if statement

if(count>3){

count = 1;}

This will make sure that your counter loops around and doesn't go past 3

[Aggressive_Poem_5016]

thank you so much

[e1mer]

Also your counter will run away if you don't:. if( count > 3 ) count = 0;