can anyone see anything wrong with this code

Printable View