複数レコードのカラムに特定の値を置換したい

Question

Userモデルで、noteというカラムがあるとします。
@users = User.allで全てのユーザーを取り出し、その@usersのカラムnoteを全てparams[:note]に置き換えたい場合、eachメゾットをしようすれば、できると思ったのですが、NoMethodErrorが出てしまいました。
繰り返し処理で行うことはできないでしょうか？

def update
  @users = User.all
  @user.each do |user|
   user.note = params[:note]
  end
end

Answer 1

NoMethodErrorが出てしまいました。

@userには何も代入していないのでnilです。@usersの間違いではないかと思います。

そして、単にuser.note = params[:note]とするだけではデータベースに保存されません。user.saveを追加するか、user.update(note: params[:note])のようにしましょう。

さらに、全部同じ値に変更するならもっと簡便な方法もあります。User.update_all(note: params[:note])とすれば、モデルの処理を行わずにデータベースへ直接書き込むので最速です（ただし、コールバックやバリデーションなどが適用されないので、それらが必要な場合は要注意です）。

Comments

[oiykram]

mainsumakunさん
ありがとうございます。誤字でした。すみません。
updateを使うと、更新するので、一度@usersのカラムnoteに代入するのみと考えていました。その場合はどうすれば良いでしょうか？

[maisumakun]

> 一度@usersのカラムnoteに代入するのみと考えていました。

理屈の上では元のコードでそういうことはできますが、果たしてそのような操作が適切なのか（どのような背景でそのような操作が必要になったのか）はちょっと疑問が残ります。