const int led = 7;
const int led1 = 8;
Do you count things like "1, 2, 3...", or do you count things like "Yes, I have a thing, 2, 3..."?
Maybe you need three or four calls to pinMode that do exactly the same thing.
buttonState = digitalRead(buttonPin);
int buttonState1 = digitalRead(buttonPin);
Unless you are faster than lightening, these two variables will contain the same value.
buttonState == LOW;
This is performing a useless test.
im try to make an led to turn on when the switch is pressed for 1 time and then turn off and when thbe switch is pressed again another led is switched offf can someone tell me what is wrong in my above code to perform this.
Aside from those things, you have several basic problems. For one, you only want to do something when the switch state changes, to pressed. You are not testing for a state change.
Second, you are not counting presses, so you have no idea how many times, on any given pass through loop(), the switch has been pressed.
Finally, hasn't anyone ever told you to "Turn the damned lights off when you are done!"?