You do have some unnecessary curly braces in there (they aren't hurting anything) and your style differs a bit from mine 
but I don't see any obvious reason this should ever print "D1". Quick question - if you press that button again, does it go to D2 or D4 (or something else)?