HereHere you can find an excellent answer for how to draw a good schematic.
What you are trying to do is called "high-side switching" and good answer / explanation can be found in thisthis answer.
Your Zener diode has no current limiting element and it might blow. As you can see here the differential resistance of the zener diode is very low after reaching the zener voltage. With no current limiting element (resistor) the current through the zener get very high and it might blow. A zener voltage regulator is usually designed like this. Be careful, a zener voltage regulator has usually a quit low output current as the load current is flowing through the zener resistor as well and the resulting voltage drop is cutting of your zener voltage for higher currents.