each文の中で、hashにキーと値を追加したい。 言語ruby

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

each文の中で、hashにキーと値を追加していきたいが、each文が実行される度に、追加ではなく、更新されてしまう。

該当のソースコード

def express_addresses(address)

domains = {}
addresses = address.group_by {|domain| domain.split("@")[1]}

addresses.each do |address|
domains = {address[0] => address[1].size}
end
puts domains
end

address = [
"[e-mail]", "[e-mail]" , "[e-mail]" , "[e-mail]" , "[e-mail]" ,
"[e-mail]" , "[e-mail]" ,
]

express_addresses(address)

ruby
ソースコード

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

{"gmail.com"=>3, "yahoo.jp"=>2, "ezweb.ne.jp"=>1, "softbank.ne.jp"=>1}
上記のように出力させたいのですが、
{"softbank.ne.jp"=>1}
この文しか出力されません。
each文の中で、　domainsというハッシュにキーと値を追加するように書いているつもりなのですが、実際には更新されているみたいです。
大変初歩的な質問ではありますが、お答えしていただけると幸いです。

Answer 1

each文の中で、　domainsというハッシュにキーと値を追加するように書いているつもりなのですが

実際のコードはそうなっておらず、毎回ゼロからハッシュを作成する形になっているので、最後のしか残りません。値1つで代入しましょう。

addresses.each do |address|
  domains[address[0]] = address[1].size
end

Comments

[kanjin555]

ご回答ありがとうございました。
修正した所、キーと値を追加する事ができました。

Answer 2

いろいろな修正方法を考えてみました。

def express_addresses(address_array)
  addresses = address_array.group_by { |domain| domain.split('@')[1] }

  # 質問文の方法
  domains = {}
  addresses.each do |address|
    domains = { address[0] => address[1].size }
  end
  puts domains
  puts

  # 修正 1
  domains = {}
  addresses.each do |address|
    domains[address[0]] = address[1].size
  end
  puts domains
  puts

  # 修正 2 (key.val で受ける)　
  # [0[, [1] より、key, val のほうが意味が明確になる
  domains = {}
  addresses.each { |(key, val)| domains[key] = val.size }
  puts domains
  puts

  # 修正 3 (inject を使う)
  # domains = {} の初期化が不要になる
  domains = addresses.inject({}) do |memo, (key, val)|
    memo[key] = val.size
    memo
  end
  puts domains
  puts

  # 修正 4 (each_with_object を使う)
  # ブロックに最後の memo が不要になる
  puts addresses.each_with_object({}) { |(key, val), memo| memo[key] = val.size }
  puts

  # 修正 5 (map を使う)
  puts addresses.map { |k, v| [k, v.size] }.to_h
end

address = [
  '[e-mail]', '[e-mail]', '[e-mail]',
  '[e-mail]', '[e-mail]', '[e-mail]',
  '[e-mail]'
]

express_addresses(address)

実行結果

inject, each_with_object, map については、API リファレンスなどを google で検索して読むことをお勧めします。

Comments

[kanjin555]

教えてくださりありがとうございました。
試した所、値を追加する事ができました！

Answer 3

domains = {address[0] => address[1].size}

の一行で、ペア一個のハッシュに上書きしています。

二行目で宣言した空のハッシュに追加したいなら、以下のような感じです。

domains[address[0]] = address[1].size

Comments

[kazto]

タッチの差で出遅れました・・・

[kanjin555]

ご回答ありがとうございました。
上書きではなく、追加できるようになりました。