SQL文をメソッドとして定義したい。

Question

SQL文をメソッドとして定義したい

SQL文をメソッドとして定義したいです。
そもそもそれが可能なのかわかりませんが、よろしくお願いいたします。
以下は、いいねのついた記事を多い順にデータベースから取り出す処理になります。
コントローラーに定義しています。

@like_trend_posts = Post.joins("left join likes on posts.id = likes.post_id")
.group("posts.id").order(Arel.sql("count(likes.id) desc"))

上記の、次のSQL文をメソッドとして定義し、繰り返し使えるようにしたいです。

joins("left join likes on posts.id = likes.post_id")
.group("posts.id").order(Arel.sql("count(likes.id) desc"))

試したこと

Postモデルに以下のように定義してみました。

def like_trend_sql
 joins("left join likes on posts.id = likes.post_id")
.group("posts.id").order(Arel.sql("count(likes.id) desc"))
end

コントローラーにも以下のように定義してみました。

private

def like_trend_sql
 joins("left join likes on posts.id = likes.post_id")
.group("posts.id").order(Arel.sql("count(likes.id) desc"))
end

そして、コントローラーにて、下記のように書き換えて使用してみました。

@like_trend_posts = Post.like_trend_sql

結果、「like_trend_sql」メソッドは定義されていないとのエラーが出ました。
定義の仕方がそもそも合っているのかわかりません。
お分かりになられる方がいらっしゃるのであれば、ぜひご教授願います。よろしくお願いいたします。

Answer 1

まずこの2点を理解する必要があります。

• オブエジェクト志向におけるクラスメソッド、インスタンスメソッドの概念について理解する
• Rubyにおいて、具体的なクラスメソッド、インスタンスメソッドの定義の仕方を理解する

細かくは説明しませんが、ざっくり言えば
Post.joinsは当然クラスメソッドを呼び出しているので、これと同じ事をしたいのであれば、
クラスメソッドとして定義しなければなりません。

インスタンスメソッド、クラスメソッドのざっくりした定義方法

とりあえずこういう風に単にモデルにメソッドを定義したら、インスタンスメソッドですよね。
なので、これはPost.new.method1という使い方になります。
もしくは、Post.find(1).method1など。

class Post 

  def method1
  end

end

クラスメソッドは、こう書きます。

class Post

  def self.method1
  end

end

このようにすると、Post.method1と使うことができます。
つまり、Rubyにおいてクラスメソッドを定義するには、def self.メソッド名 ～ endと書きます。

本題

さて、これを踏まえて「試したこと」で書いてもらったコードを見れば、インスタンスメソッドだから
Post.like_trend_sqlという使い方が出来ない事が分かります。

class Post

  def like_trend_sql
    joins("left join likes on posts.id = likes.post_id")
      .group("posts.id").order(Arel.sql("count(likes.id) desc"))
  end

end

これをクラスメソッドにするには、self.をメソッド名の前に付けてやれば良いですね。

class Post

  def self.like_trend_sql
    joins("left join likes on posts.id = likes.post_id")
      .group("posts.id").order(Arel.sql("count(likes.id) desc"))
  end

end

これで、Post.like_trend_sqlという使い方ができるはずです。

Comments

[Naon]

ご回答ありがとうございます！
とても丁寧な説明もありがとうございます。言われてみれば、selfがないとダメだなと感じました。
今一度、その部分について調べ直してみたいと思います。
問題も解決することができました！
ベストアンサーにさせていただきたいと思います！

Answer 2

メソッドを作る場合は、module として切り出すか、maisumakunさんが仰る通りselfを使ってclassメソッドにすれば良いと思います。

Answer 3

railsのバージョンとエラー内容がわからないので、間違っているかもしれませんが、
Postモデルに記載するだけでほかのところでも使用できるようになると思います。
記載方法ですが、

def like_trend_sql
 self.joins("left join likes on posts.id = likes.post_id")
.group("posts.id").order(Arel.sql("count(likes.id) desc"))
end

もしくは、

scope :like_trend_sql, ->{ joins("left join likes on posts.id = likes.post_id")
.group("posts.id").order(Arel.sql("count(likes.id) desc")) }

こんな感じで
Post.like_trend_sql
で使えるかと。

Comments

[Naon]

コントローラーには記載しなくて良いのですね、、、。
勉強になります。ご回答ありがとうございます！
問題も解決することができました！

Answer 4

Postモデルに以下のように定義してみました。

Post.like_trend_sqlの形で使いたいのであれば、クラスメソッドとして定義してください。

Comments

[Naon]

ご回答ありがとうございます。
Postモデルに以下のように定義すれば良いということでしょうか？

def self.like_trend_sql
　 joins("left join likes on posts.id = likes.post_id")
　.group("posts.id").order(Arel.sql("count(likes.id) desc"))
end

[maisumakun]

そうですね、そんな感じです。

[Naon]

ご教授ありがとうございます！
問題も解決することができました！