refactor to setters an getters