Argthtjtr

Imagine a system where you have two transient non-exclusive buttons and two LEDs.

When you press button 1, the LED 1 is HIGH and LED 2 is LOW (and stays that way).

When you press button 2, the LED 1 is LOW and LED 2 is HIGH (also stays that way).

Which C++ code allow me, to keep the LED 1 turned on indefinitely (until button 2 is pressed)?

Basically, my problem is, that the buttons are transient and the behaviour of LEDs shouldn't be.

Here is code, that would work for pernament exclusive buttons, but doenst work for transient non-exclusive buttons.

if (digitalRead(button1) == HIGH) 

{   

    led1 = HIGH; 

}

else 

{ 

    led2 = LOW; 

}

if (digitalRead(button2) == HIGH) 

{ 

    led2 = HIGH; 

}

else 

{ 

    led2 = LOW; 

}

Something like this:

if (digitalRead(button1) == HIGH) 

{   

    led1 = HIGH; 

    led2 = LOW; 

}

if (digitalRead(button2) == HIGH) 

{ 

    led2 = HIGH; 

    led1 = LOW; 

}

Don't react to the button by changing the LED; change a state-variable in memory, instead. That saves the information of which button was most recently pressed. Then, each time through your loop, set your LED's according to the contents of that persistent variable. Remember that it will have to be a static variable (global or static local), as an automatic one will disappear and reappear after each exit and re-entry to loop(), and you cannot count on its contents when that happens.

Update:

How do I prevent overwriting the state variable with different input ?

You don't prevent it - you must over-write it. The purpose of the state variable is to remember which button was last pressed. I had in mind something like the following, where lastButton saves the number of the most recent button:

void setup(){

   pinmode(LED1PIN, OUTPUT);

   pinmode(LED2PIN, OUTPUT);

   pinmode(BUTN1PIN, INPUT_PULLUP);  // you might not want the pullup

   pinmode(BUTN2PIN, INPUT_PULLUP);  //  depending on your wiring.

}





void loop(){

   static uint8_t lastButton = 0;



   // Save the number of the last button pressed.

   if (digitalRead(button1) == HIGH) 

      lastButton = 1;

   else if (digitalRead(button2) == HIGH) 

      lastButton = 2;



   // Set the LEDs according to most recent button

   // Note that at program-start, when no button has

   // been pressed yet, this code leaves both LED's off.

   if (lastButton == 1) 

   {   

      digitalWrite(LED1PIN, HIGH);

      digitalWrite(LED2PIN, LOW);

   }

   else if(lastButton == 2)

      digitalWrite(LED1PIN, LOW);

      digitalWrite(LED2PIN, HIGH);

   }

}