please refactor to use Enumerations instead