Magic values should be made into constants