Logic looks good!

(see previous comment on using a range in a for loop too)