Может ли этот код на Ruby быть написан более кратко?

Question

positions = Hash.new
import_profile.headings.each do |h|
  positions[h.table_name + '.' + h.column_name] = h.position
end

Как только это будет сделано, меня заинтересует positions. Это почти то же самое, что я написал бы в PHP, но меня привлекло Ruby для его функций map и collect. Есть ли способ написать это в одну строку?

Answer 1

Я думаю, что наиболее кратким будет что-то вроде этого.

positions = Hash[*import_profile.headings.map do|h|
    [ "#{h.table_name}.#{h.column_name}", h.position ]
  end.flatten
]

Но это не очень читабельно. Мне больше нравится ваш код.

Answer 2

Если вам интересно, вот как будет выглядеть решение на основе inject.Не короче, но немного больше FP-иша:

positions = import_profile.headings.inject({}) do |acc,h|
  acc["#{h.table_name}.#{h.column_name}"] = h.position
  acc
end

Answer 3

positions = Hash[ import_profile.headings.map do |h|
  [ "#{h.table_name}.#{h.column_name}", h.position ]
end ]

или

positions = Hash[ *import_profile.headings.map do |h|
  [ "#{h.table_name}.#{h.column_name}", h.position ]
end.flatten ]

Первый работает только в Ruby 1.8.7+, где Hash.[] разрешено получать массив массивов из двух значений. Последний работает в более ранних версиях, где Hash.[] было разрешено получать только четное количество параметров.

Answer 4

Hash[import_profile.headings.map { |h| ["#{h.table_name}.#{h.column_name}", h.position] }]

Answer 5

Другой способ объединения слов - использовать join:

positions = Hash.new
import_profile.headings.each do |h|
  positions[[h.table_name, h.column_name].join(".")] = h.position
end