3

コード自体はかけたのですが、もう少し上手くかけないかと思い質問致します。

a = [[2014, 3], [2014, 5], [2015, 4], [2016, 2], [2016, 8]]

という配列から、配列の初めの要素に入っている年号をキーにした

{2014 => [3, 5],
 2015 => [4],
 2016 => [2, 8]}

というHashを生成したく

a.group_by(&:first).map{|k, v| [k, v.map(&:last)]}.to_h

と書いたのですがかなりわかりにくい気がするので

h = {}
a.each do |k, v|
  h[k] ||= [] 
  h[k] << v
end
h

としました。できればもう少しすっきり書きたいのですがもっとシンプルな書き方はありますか?

4 件の回答 4

1

inject を使うのはどうでしょうか。

a.inject(Hash.new([])){|h, (k, v)| h[k] += [v]; h}
1
  • hash の default 値に mutable な要素を使ってはいけない、と避けていましたが、非破壊メソッドを使うことで初期値としてだけ使うことができるんですね。目から鱗でした! Commented 2016年7月19日 1:03
0

私もいつも悩みます。
その中でもやっているのは、デフォルト値を使うことで、key の判定を省略する方法です。

h = Hash.new { |h, k| h[k] = [] }
a.each do |k, v|
  h[k] << v
end
h

またこちらの記事 http://qiita.com/cuzic/items/a265f140fdff289d5c07#objecttap を読んでから、tap を使うことも多いです。

years_month_lists = {}.tap do |h|
  a.each do |k, v|
    h[k] ||= []
    h[k] << v
  end  
end
1
  • tapはこんな風にも使えるんですね。ただインデントを増やすのが好きではないのでデフォルト値を使う方法を使うことにします。
    – ironsand
    Commented 2016年7月18日 1:53
0
h = a.each_with_object({}) do |(k, v), res|
  res[k] ||= []
  res[k] << v
end

p h
0

あくまで、aを破壊してもいいという前提ですが・・・

a.group_by(&:shift).each_value(&:flatten!)
1
  • なるほど、面白い方法ですね。ただ自分のレベルですと、後から読み返した時に何してるかパッとは理解できなさそうです…。
    – ironsand
    Commented 2016年7月18日 1:56

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。