use constants and instead of string