use variables instead of magic values