int switchPin = 4; // but you use pin 2
int relayPin = 7; // but you use pin 3
pinMode(switchPin, INPUT_PULLUP); // then you should use a switch between pin and ground (no resistor)
but you use the switch between pin and VCC, and use a pull down resistor.
Leo..