replace magical number with constant