this is in-efficient (doesnt matter here but we want to do good style). Get once and then access in loop