getting rid of hard coded values