Is there anything wrong with this code?

Printable View