positions = Hash.new
import_profile.headings.each do |h|
positions[h.table_name + '.' + h.column_name] = h.position
end
これが完了すると、興味があるのはpositionsです。これはPHPでこのようなことをどのように記述するのかとかなり正確ですが、私はRubyのmapとcollectの機能に魅了されました。これを書くために多分1行の方法がありますか？このRubyコードをより簡潔に書くことはできますか？
Hash[import_profile.headings.map { |h| ["#{h.table_name}.#{h.column_name}", h.position] }]
簡潔は、このようなものだと思います。
positions = Hash[*import_profile.headings.map do|h|
[ "#{h.table_name}.#{h.column_name}", h.position ]
end.flatten
]
しかし、それはひどく読めるわけではありません。私はあなたのコードが好きです。
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 ]
前者のみHash.[]が2値配列の配列を受信するように許可されているルビー1.8.7+に働きます。後者は、Hash.[]が偶数のパラメータだけを受け取ることが許されていた旧バージョンでも動作します。
興味があれば、これはinjectベースのソリューションの外観です。短くないが、もう少しFP-っぽい：
positions = import_profile.headings.inject({}) do |acc,h|
acc["#{h.table_name}.#{h.column_name}"] = h.position
acc
end
一緒に言葉を結合する別の方法はjoinを使用することです：
positions = Hash.new
import_profile.headings.each do |h|
positions[[h.table_name, h.column_name].join(".")] = h.position
end
あなたはあなたのマップで表現周りの角括弧が欠落しています。結果をスプラットする場合は、 'flatten'への呼び出しもありません。このコードは構文チェックに合格しません。 – Phrogz
あなたは正しいです。私はsplatオペレータで平らにする必要はないと思っていました。私はまた、他のマシンからコピーしたときに角括弧を追加するのを忘れてしまった。 – AboutRuby
'.map.flatten（1）' == '.flat_map' – Nakilon