should this be set in the constructor?