This working fine! An improvement could be to put the mathematical operation out of the loop so that it is not called in each iteration, and avoiding redundant calculation.