Looks like it will work but your using the long datatype for a number with a max of 2000. Thats overkill, just use int you'll save a little space. This wont matter for your purpose because its a small program but you don't wont to get into the habit. If you wanted to get fancy you could even use byte for aswitch because its just 12