please use constants rather than magic numbers