Electronic – Using a microcontroller to make a series of LEDs blink one after the other

From your description, it seems you need to debounce the switch.

You can do this

• in software or
• in hardware

Since you seem to be using some kind of resistive ladder to read multiple buttons on one ADC port, this will be a little more complicated.

The equality testing operator is ==, the assignment operator is =.

You are using = instead of == so the if condition returns always true.

This line if(keypressed == '1') assigns character 1 to the keypressed variable, since keypressed becomes a non zero value the if condition is always true.

The code should be like:

if(keypressed == '1')
{
    digitalWrite(led1, HIGH);
    delay(500);
    digitalWrite(led1, LOW);
    delay(500);
}

if(keypressed == '2')
{

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

}