2 led's and 2 switches

this is the code i used for 2 switches and led's

int led1pin = D0; 
int led2pin = D1; 
int button1 = D2; 
int button2 = D3; 

int button1State=0;
int button2State=0;

void setup() {
 pinMode(led1pin, OUTPUT);
 pinMode(led2pin, OUTPUT);
 pinMode(button1, INPUT);
 pinMode(button2, INPUT);
}

void loop() {
 button1State=digitalRead(button1); // put your main code here, to run repeatedly:
 if (button1State == 1)
 {
 delay(200);
 digitalWrite(led1pin, HIGH);
 delay(200);
 digitalWrite(led1pin, LOW);
 }
 if (button1State==0)
 {
 digitalWrite(led1pin, LOW); 
 delay(200);
 }
 button2State=digitalRead(button2); // put your main code here, to run repeatedly:
 if (button2State == 1)
 {
 delay(200);
 digitalWrite(led2pin, HIGH);
 delay(200);
 digitalWrite(led2pin, LOW);
 }
 if (button2State==0)
 {
 digitalWrite(led2pin, LOW); 
 delay(200);
 }
}

when i use this the led's malfunction are there any mistakes that i ahve made ?