proper use of constants