use constants instead of strings