This seems like a bad idea - should be done by the constructor instead