again why not have it in constructor?