any help?
Use nested loops:
if(Button != OldButton)
{
// A transition occurred. But, which one?
if(Button == HIGH)
{
}
else
{
}
}
OldButton = Button;
Use meaningful names. Button is a lousy name. Is that supposed to be the name of a pin or the state of the pin? A good name would make that clear.
Separate the "implement the change" code from the "decide what change to make" code. The switch state code should only decide what the LED state should be. It should NOT implement that change.
That happens regardless of whether the switch was pressed or not.
Finally, when you post code that compiles, it works. That is, it does something. You want it to do something. When you say "it doesn't work" all we can tell is that what it does is not what you want. We can't tell what it does or what you want or how what it does differs from what you want. Provide those details if you really expect help.