scopeの中でexists?メソッドは使えますか？

Question

ViewHelperに書かれているコードが長いため、scopeに移して、問題なく動くようにしたいです。

既存のviewHelperにあったコードです

CompanyInvestor.where(company_id: company_id, investor_id: investor_id, connect_requested: true, connect_responded: false, connect_approved: false).exists?

長いのでモデルに移します。

scope :company_requesting?, -> (company_id, investor_id) { where(company_id: company_id, investor_id: investor_id, connect_requested: true, connect_responded: false, connect_approved: false).exists? }

そしてhelper側では、

CompanyInvestor.company_requesting?(company_id, investor_id)

としました。

しかし、これだとcompany_requesting?で返ってくる値がactive_relationになってしまいます。

#<ActiveRecord::Relation [#<CompanyInvestor id: 1, investor_id: 1, company_id: 1, connect_requested: true, connect_responded: false, connect_approved: false, created_at: "2019-11-07 02:35:59", updated_at: "2019-11-07 02:35:59">]>

exists?をスコープから外し、

scope :company_requesting?, -> (company_id, investor_id) { where(company_id: company_id, investor_id: investor_id, connect_requested: true, connect_responded: false, connect_approved: false)}

helperの中に記述すると、booleanが返ってきます。

CompanyInvestor.company_requesting?(company_id, investor_id).exists? => true か false

active_relationが返ってきているため、exist?がscope内で実行されず、whereの返り値が出ているような気がします。

scopeではexists?は実行されないのでしょうか？

何卒よろしくお願いします。

Answer 1

そもそもの話になりますが、ActiveRelation以外のものが返ってきて欲しいものをscopeで書くべきではないです。
「scope」という意味を考えれば分かるようにレコードの絞り込みに使うものですので。

なのでメソッドにしましょう。

Comments

[tomoharu]

承知しました。そのルールは知りませんでした。ありがとうございます。

Answer 2

scopeではexists?は実行されないのでしょうか？

いえ、実行はされますが、ブロックがnilやfalseを返した場合、scopeは代わりに.allを返します。

If it returns nil or false, an all scope is returned instead.（Rails APIより）

一般に、リレーションを返さないメソッドはscopeではなく、単なるクラスメソッドにしたほうがいいかと思います。

def self.company_requesting?
  where(company_id: company_id, investor_id: investor_id, connect_requested: true, connect_responded: false, connect_approved: false).exists?
end

Comments

[maisumakun]

なお、クラスメソッドでも、「CompanyInvestor.where(別な条件).company_requesting?」のようなことも可能です。

[tomoharu]

ご回答ありがとうございます。
allと言いますと、exists?がある場合でのscopeの返り値がfalseの場合、exists?の後にallがついて実行される感じですか？　少しそこのイメージがつかず。。。

[maisumakun]

CompanyInvestor.company_requesting?(...)の結果がCompanyInvestor.allになる、ということです。

（もっとも、falseを返したいことを考えれば、こちらを突き詰めても良い結果は得られないかと思います）

[tomoharu]

承知しました。ありがとうございます。
一応深く理解しておこうと思いました。