rails モデル名.where(created_at: Time.current.all_month)で今月投稿されたデータを取得したいが先月のデータも取得してしまう。

Question

前提・実現したいこと

SNS系のアプリを作成しております。
いいね数の多い順にランキング機能を実装しています。
その際に月間ランキングで、今月投稿された投稿の中でいいね数が多い順にランキングにしたいと考えております。（今であれば2月に投稿された内容のみ）

発生している問題・エラーメッセージ

先月投稿された投稿にいいねをすると、今月投稿されたものと認識されてしまいランキングページに表示されてしまいます。
例）
・現在が1月と仮定
1月に新規投稿、1月中いいね数0、1月のランキングに表示されない→正常（確認済み）
1月に新規投稿、1月中いいね数10、1月のランキングに表示される→正常（確認済み）

・現在が2月と仮定
1月に新規投稿、1月中いいね数10、2月中いいね数0、2月のランキングに表示されない→正常（確認済み）
1月に新規投稿、1月中いいね数10、2月中に1回以上いいね、2月のランキングに表示される→正常でない（今回の問題点）
※いいねを取り消すとランキングからも消えます。
2月の場合は、2月に新規投稿された投稿のいいね数でランキング表示を行いたいです。

posts_controller.rbのrankingアクション内の@posts_good_rankingで問題が起きております。

該当のソースコード

def ranking
  @posts_pv_ranking = Post.where(created_at: Time.current.all_month).order(impressions_count: 'DESC').limit(4)  
  @posts_good_ranking = Post.find(Like.group(:post_id).where(created_at: Time.current.all_month).order('count(post_id) DESC').limit(4).pluck(:post_id))
  @date1 = Date.current.strftime('%m')
end

def pv_ranking
  @posts_pv_ranking = Post.where(created_at: Time.current.all_month).order(impressions_count: 'DESC').page(params[:page]).per(8)
  @date1 = Date.current.strftime('%m')
end

def good_ranking
  @posts_good_ranking = Post.find(Like.group(:post_id).where(created_at: Time.current.all_month).order('count(post_id) DESC').pluck(:post_id))
  @posts_good_ranking = Kaminari.paginate_array(@posts_good_ranking).page(params[:page]).per(8)
  @date1 = Date.current.strftime('%m')
end

<div class="container">
  <h4>グッド!!が多い投稿（<%= @date1 %>月）</h4>
  <div class="row">
    <%= render "good_ranking", posts_good_ranking: @posts_good_ranking %>
  </div>
  <div class=" row d-flex justify-content-end ranking-list">
    <%= link_to "一覧で見る", posts_good_ranking_path %>
  </div>
  <h4>閲覧数が多い投稿（<%= @date1 %>月）</h4>
  <div class="row">
    <%= render "pv_ranking", posts_pv_ranking: @posts_pv_ranking %>
  </div>
  <div class=" row d-flex justify-content-end ranking-list">
    <%= link_to "一覧で見る", posts_pv_ranking_path %>
  </div>
</div>

<% posts_good_ranking.each do |post| %>
  <div class="col-lg-3 d-flex flex-column postranking">
    <div class="ranking-user-name">
      <%= link_to user_path(post.user.id) do %>
        <%= attachment_image_tag post.user, :profile_image, size: '50x50', fallback: "no_image.jpg" %>
        <%= post.user.name %>
      <% end %>
    </div>
    <div>
      <%= post.title %>
    </div>
    <div class="mb-auto ranking-post">
      <% post.post_images.first(1).each do |image| %>
        <%= link_to post_path(post.id) do %>
          <%= attachment_image_tag image, :image, class: "ranking_image" %>
        <% end %>
      <% end %>
  　</div>
  　<div>
  　   <i class="fas fa-thumbs-up"></i>
  　   <%= post.likes.count %>
  　   <i class="far fa-clock clock-icon"></i>
      <%= time_ago_in_words(post.created_at) %>前
  　</div>
  </div>
<% end %>

試したこと

def ranking
  @post = Post.where(created_at: Time.current.all_month)
  @posts_good_ranking = @post.find(Like.group(:post_id).order('count(post_id) DESC').limit(4).pluck(:post_id))

まず上記のように最初に今月作成されたデータだけを取得しその上でいいね数を数えて並べるようにしましたが下記エラーが表示され解決方法がわからない状態です。
ActiveRecord::RecordNotFound in PostsController#ranking
Couldn't find all Posts with 'id': (1, 2, 3, 4) [WHERE "posts"."created_at" BETWEEN ? AND ?] (found 1 results, but was looking for 4).

どのようにしたら、先月投稿された投稿にいいねをしても、今月投稿されたものと認識されず、今月新規投稿された投稿だけでランキング機能を実装できますでしょうか？

Answer 1

わかりにくい質問です、
view が ranking.html だから action ranking だろうとはおもいますが 問題が起きているのが
@posts_pv_ranking なのか @posts_good_ranking なのか が明記されていないのでわかりにくいのです。

code をみると 前者では起きそうにない、後者だけ起きている、と思われるのでその線で
Post.find(Like.group(:post_id).where(created_at: Time.cu ..... とあります。
これは、「今月 Like された」 Post を拾ってます。のでそういう動作になります。
Post.where(created_at: 。。。).find( ) にしてください。find ではなく where（id: ...) の方が例外立たないからよいかも。
今月Postされたもので、Like されたもの を拾うのでしたら、 Likeは今月しかないですから、Like に created_at の条件は不要です。
入れても良いけど。

Comments

[0.ohsdhekl.7]

回答いただきありがとうございます。

質問がわかりにくく申し訳ありません。
おっしゃる通り@posts_good_rankingで問題が起きております。

ご教示いただいたように、Post.where(created_at: 。。。).find( )としたところ自身で試したことで記載したのと同じエラーが発生しました。
LIKEのcreated_atは削除しました。

findではなくwhere(id: ...)に変更したら今月の投稿のみ表示することができました。
しかし、こうしたことで1番いいね数の多い投稿しか取得することができず1件のみの表示になりました。
LIKEにcreated_atを入れ試したところ表示は2つに増え、消すと1つになります。
規則性がわかりません。

やりたいこととしては、今月の投稿のいいね数が多い順に4つ取り出すことです。

[0.ohsdhekl.7]

すみません。
.limit(4)をwhere(id: ...)の外に記述したらうまくいきました。

大変助かりました。
ありがとうございます。

ちなみに、find()の時とwhere(id: ... )の時でどうして違いが生まれたのでしょうか？

[winterboum]

find は hit しないと 例外となります
where はhit しないと 空の配列(のようなもの）が帰ります

[0.ohsdhekl.7]

今回の場合は例外になっていたんですね。
ありがとうございます！

[winterboum]

ActiveRecord::RecordNotFound in PostsController#ranking
これが例外 ActiveRecord::RecordNotFound が発生した、というメッセージです