get rid of magic number