#Rails ActiveRecordのアンチパターン
##はじめに
急にスケジュールが詰め詰めになったので、残り４日とわずかですが、今回は少なめです。すみません...
頑張ります。
今回はActiveRecordにおけるアンチパターンについて。パフォーマンスを落とすようなコードを書かないようにアンチパターンを日々、覚えるようにしましょう。
##All Each Pattern
User.allなどでレコードを全取得してそこからeachで回すパターン。
User.all.each do|user|
if user.created_at >= Date.new(2018)
user.point += 100
user.save
end
end
2018年以降に登録してくれたユーザにポイントをプレゼントしよう的なコード（ほぼ参考元のコードです。すみません）
ダメなとこ
1,Userの全件取得でメモリを圧迫。
2,eachメソッドにより取得したユーザー全件を一気にメモリに展開してから処理するので、こちらもよろしくない。
改善策
1,そもそも全件取得する必要がない。whereを使って取得件数を下げる。
2,find_eachメソッドを使って、分割して処理する。（find_eachメソッドはeachメソッドと違って、一気にではなく分割して処理する。）
##改善後
User.where("created_at >= ?", Date.new(2018)).find_each do |user|
user.point += 100
user.save
end
よくなりました...と思いきや、まだ問題があります。
##N+1 update queries pattern
前にN+1問題は書きましたね。↓
RailsのN+1問題の解決方法 初心者→中級者へのSTEP9/25
似たような感じの問題です。１回のselectに対し、N回のupdateが起きています。１回にしましょう。
→update_allを使う。
##再改善後
User.where("created_at >= ?", Date.new(2018)).update_all("point = point + 100")
以前書きましたが、update_allはバリデーションをしませんので今回は問題ないですが、パフォーマンスが上がる！と言っってむやみに使ってはダメです。
これでパフォーマンスがぐぐっと上がるはずです。
##まとめ
なんかほぼ自分のブログみたいになってきたこのカレンダー。お家がゴタゴタして書く暇あるか微妙ですが、残り数日休まず記事をあげたいです。今回のアンチパターンは参考元のスライドが非常にわかりやすいです。興味持ったらぜひそちらを読んでください。他の例も載せてくださってます。
##参考にしたの
ActiveRecordデータ処理アンチパターン
https://speakerdeck.com/toshimaru/active-record-anti-patterns?slide=26
簡単に始められるRailsパフォーマンス改善
https://qiita.com/yu-croco/items/d7a39b34036d638c5026