also PLEASE in the future (you can still fix your post above and re-edit it) add the code tags around your code so that code looks like this (edit, select your code and press the </> icon and save.)
if (buttonState1 == HIGH) {
// turn LED on:
digitalWrite(ledPin1, HIGH);
}
else if(buttonstate2 == HIGH)
{
// turn LED on:
digitalWrite(ledPin2, HIGH);
}
else if(buttonstate3 ==HIGH)
{
// turn LED on:
digitalWrite(ledPin3, HIGH);
}
else if(buttonstate4 ==HIGH)
{
// turn LED on:
digitalWrite(ledPin4, HIGH);
}
hopefully you understand that if button1 is pressed then because of the else statements you won't check the other buttons, right?
and you know that there is no way to turn off a LED once you turned it on?