why isn't my loop working properly?

Printable View