this is basically a constant: why are we re-calculating every time in the loop? Should it be defined outside the loop?