use constants instead of enum