Railsのwhereとupdate_allを組み合わせた場合に後にwhereの値が空になる問題

Question

下記のようなコードを書いています。

users = User.where(hoge: nil) # User(hoge: nil), User(hoge: nil), User(hoge: true)
users.update_all(hoge: true) # User(hoge: true), User(hoge: true), User(hoge: true)

p users # [User(hoge: nil), User(hoge: nil)]を期待するが[]が帰ってくる

あくまで推測なのですが、p usersしたタイミングでUser.whereが実行されるため、その時はもうupdate_allが走っていて、結果が取れていないと考えています。

例えば以下のようにすれば、意図通りに結果が得られます。

users = User.where(hoge: nil) # User(hoge: nil), User(hoge: nil), User(hoge: true)
p users # [User(hoge: nil), User(hoge: nil)]
users.update_all(hoge: true) # User(hoge: true), User(hoge: true), User(hoge: true)

p users # [User(hoge: nil), User(hoge: nil)]

p usersしたら、実際にUser.whereが実行されるから上手く行っているのかもしれませんが、綺麗に書くとしたら

users = User.where(hoge: nil).今すぐ実行
users.update_all(hoge: true)

p users # [User(hoge: nil), User(hoge: nil)]

のような、今すぐ実行メソッド（のようなもの）を叩きたいと思っています。
何かアドバイスいただけませんでしょうか。よろしくお願い致します。

Answer 1

おっしゃるように実際のクエリが走るタイミングの問題です。

user = User.where(...) #このタイミングではクエリは走らない
user.update_all(...) #上記 where と組み合わせて UPDATE users ... FROM ... WHERE ... が走る

p user # レコードがまだ取得されてないので SELECT * FROM users WHERE ... が走る

user = User.where(...) #このタイミングではクエリは走らない
p user # レコードがまだ取得されてないので SELECT * FROM users WHERE ... が走る
user.update_all(...) #上記 where と組み合わせて UPDATE users ... FROM ... WHERE ... が走る
p user # 2行目で取得されたレコードがそのまま

1番目のコードがDBを正しく反映してるのにたいし、2番目のコードの方はダーティーデータが読めてしまっているので、どちらかというと後者の方が適切でない動作となります。#reloadすると

(2番目のコードの続き)
users.reload
p users # []

こうなります。

update_allはこのような挙動になるため意図せず変更前の状態に依存してしまったり、アプリケーションとDBで状態が一致しなくなるので思わぬところで状態が更新されるとおかしな事になります。他にもバリデーションやコールバックが走らないなど癖がありますので、使用は避けた方が良いです。

取り急ぎであれば

user = User.where(...)
users_before_update = user.to_a 

などとしてRelationオブジェクトではなくしてしまって保存しておくのが考えられますが、やはり前後のコード含めて見直した方が良いと思います。

Comments

[yoneapp]

設計を見なおしたほうが良いことを理解しました。
ありがとうございました！

Answer 2

p users = User.where(hoge: nil).update_all(hoge: true)