get rid of quotes
refactor for private methods and putting values in attributes