prefer 'map' over 'to_a.each'