Ruby on Railsにおけるモデルからのデータの取り出し方

Question

account = Account.find_by(id: params[:id])
account.tasks.update_all('notice = notice + 1 ')

こちらの２行のコードを一行にまとめることはできますか？

scopeやincludesなど調べてみたのですがイマイチ理解できなかったので・・・

Answer 1

accountを再利用する必要がないなら、

Account.find_by(id: params[:id]).tasks.update_all('notice = notice + 1 ')

とつなげればよいです。ここで、find_by(id: params[:id])はfind(params[:id])とした方がよいでしょう。

ここまでは単純な回答です。

AccountsControllerの中での話であればAccount.find(params[:id])は複数のアクションで行われるでしょうから、メソッドにまとめた方がよいです(scaffoldのコードを参考にしてください)。代替主キーを使うような時にも変更が1カ所ですみます。

Taskを直接検索する回答がついていますが、同様に適不適がありますので慎重に判断してください。そもそもAccountモデルやAccountsControllerの中にTask.where...のようなコードが現れるのはよろしくないです。

また、update_allはバリデーションが呼ばれないとかタイムスタンプの絡むが更新されないとか癖があります。普通は避けた方がよいとされています。

Comments

[maisumakun]

update_allを使っていたので、データ移行とかコンソールを叩いてとかの、いわゆる「書捨て」に近いコードという前提で回答していました（恒常的に使うコードでしたら、別な書き方の方がいいというのは間違いないです）。

[dongw]

AccountsControllerの中で、どうしてもTaskモデルからデータを直接取ってくるwhereなど使ってしまうこともありそうですが、出来るだけ避けたほうがいいのですね。

update_allはそういう癖もあるんですね。
今回の場合はバリデーションは当該カラムには掛けていないのでいいかな、とは思っているのですが。
タイムスタンプカラムが更新されない、というのはバグなんでしょうか？

[suzukis]

仕様です。ActiveRecordの様々な仕組みをバイパスすることで一括更新を実現するためのメソッドなので。

[dongw]

仕様なんですね。
そういったメソッドの細かい仕様みたいな事が記述されている資料というのはネット上にありますか？
あれば参考に見てみたいのですが・・・

[suzukis]

リファレンスをお読みください。http://api.rubyonrails.org/ です。

[dongw]

大変参考になりました。
ありがとうございました。

Answer 2

バックエンドがRDBMSであれば、直接リレーションのキーを引くことで、子要素だけも抽出できます。なお、「レールに乗った」命名をしてあるのが前提なので、違う場合には名前の変更などが必要です。

Task.where(account_id: params[:id]).update_all('notice = notice + 1 ')