can't you move this to the constructor?