You should have constants for magic numbers