Get rid of the 10K resistor & the VCC connection. Connect one side of switch to arduino input pin and turn on its internal pullup resistor, connect the other side of the switch to ground and when the switch is pressed the pin goes low.
Your current design has flaws as it will always draw current when switch is not pressed. When switch is pressed your connecting VCC direct to the pin and unless you turn on internal pullup on the pin it will float and give false readings when switch is not pressed.