better to use a setter