LED, push button and analog values ain't working they way it should

Projects General Guidance
4 
int outputValue = map(analogValue, 0, 1023, 0, 255);   
int analogValue = analogRead(analogPin);

Probably be a good idea to do this AFTER the { for loop, not before.

It would also be a good idea to read the value first, then map it, so you don't look (so much) like an idiot.

Also, what is it you expect to do with this value? How does the analog value correspond to a switch state?