use constants instead of string