Anything wrong with my first loop?

Printable View