ok like this? or do this in the constructor?