Why don't do it in constructor?