Question

I know little about ruby, my following implementation works, but I really feel the approach curbsome

@surfaces = []
Surface.all.each do |surface|
    @surfaces << surface.name
end

the purpose is to extract name value of each surface objects and form them into an array, can anyone tell me the smart way to do this? thanks

Answer 1

Use Enumerable#map to turn one enumerable into another enumerable with a 1-to-1 mapping of elements. Combine with Symbol#to_proc when the mapping involves simply invoking a single method on each element of the input enumerable.

Both of these are equivalent:

@surfaces = Surface.all.map(&:name)

@surfaces = Surface.all.map do |surface|
  surface.name
end