switch to constructor init vs inline