Is this 2 minutes? Would be good to use constants instead of magic numbers