Have I laid out my code wrong?

Printable View