I dont see anything wrong with this(simple beginner code)

Printable View