配列内にハッシュを作る方法を教えてください

Question

array = [
 {name: hoge, age: 15},
 {name: fuga, age: 20},
]

上記のような複数のハッシュをもつ配列をeach文で作成したいです。

@user = User.all
@user.each do |user|
 array = []
 array << {
  name: user.name, age: user.age
 }
end

#=> [{{name: fuga, age: 20}] 

このようなコードを書いたのですが、#=> [{{name: fuga, age: 20}] というように@userの最後尾のデータしか格納されていません。

初歩的な質問で申し訳ありませんがよろしくお願いいたします。

Answer 1

複数のレコードを持つような変数の名前は複数形を使うようにしましょう。
変数名は大事ですよ。

@users = User.all

array変数はループの外で初期化しましょう。

array = []
@users.each do |user|
 array << {
  name: user.name, age: user.age
 }
end

ちなみに

array = @users.map do |user|
  user.slice :name, :age
end

でも同じことができます。

Comments

[lilywhite]

ありがとうございます！そうですよね、初期化も繰り返してたら当然ああいう結果になりますよね。初歩的な質問だったにも関わらず丁寧に回答してくださってありがとうございました！変数名も気を付けます！

Answer 2

ループ中でarray = []しているので、
・空にする
・1件追加する
を繰り返しています。

Comments

[lilywhite]

単純すぎるミスでした！ご指摘ありがとうございました！

Answer 3

each をつかった方法だけでなく、それ以外の方法も示してみます。

attrs = []
User.order(:name).each {|u| attrs << {name: u.name, age: u.age}}

User.order(:name).pluck(:name, :age).map{|x| {user: x[0], age: x[1]}}

User.order(:name).map{|x| {user: x.name, age: x.age}}

User.order(:name).select(:user, :age).map(&:attributes)

実行例
次のような User の内容の状態を seed.rb で作っています。

各方法をためしてみています。

Answer 4

これでいかがでしょう！！

# 変更前
@user = User.all

@user.each do |user|
 array = []
 array << {
  name: user.name, age: user.age
 }
end

# 変更後
@user = User.all
array = []

@user.each do |user|
  array << {
    name: user.name, age: user.age
  }
end

eachループの中でarray = []を宣言してしまっていたので、イテレーション毎にarray変数が初期化されてしまっていたので、ループの外に出しました

補足

array という名前は他と被ったりすることもありますし、、というかそもそも、rubyに存在するメソッド名を上書きしてしまっている気がするので、他の名前に変えた方がよさそうです。。。

Comments

[siruku6]

（みんな同じこと書いてるやん...）(´・ω・`)

[lilywhite]

初歩的な質問にも関わらず丁寧に回答いただきありがとうございます！解決いたしました！
実際書いていたコードが変数だらけだったので例として分かりやすい？ものに置き換えてみたんですが、逆にわかりづらくなってしまっていたかもしれません、すいません！命名に気を付けるようにします。