Product.unchecked_no_replied_productsスコープのリファクタリング

[komagata]

概要

Product.unchecked_no_replied_productsスコープのN+1問題とメモリ効率の問題を解決し、rails-patterns gemのQueryオブジェクトにリファクタリングしてください。

QueryObjectパターンについて

rails-patterns gemのQueryオブジェクトの詳細については、以下のwikiを参照してください：
https://github.com/fjordllc/bootcamp/wiki/QueryObject

既存の実装例としてapp/queries/completed_learnings_query.rbも参考にしてください。

現在の問題

• N+1問題: filter内でproduct.comments.lastを呼び出してN+1問題が発生
• メモリ効率: 全レコードをRubyメモリに読み込んでからフィルタリング
• スケーラビリティ: データ量の増加でパフォーマンスが大幅に悪化
• データベース最適化の機会損失: データベースレベルで処理可能なロジックをRuby側で実行

対象となるメソッド

• Product.unchecked_no_replied_products (app/models/product.rb:58-65)

期待する成果

• app/queries/product_unchecked_no_replied_query.rbの作成
• N+1問題の完全な解消

実装上の注意点

• Patterns::QueryはActiveRecord::Relationを返すことが前提です
• データベースレベルでのクエリー最適化を重視してください
• 既存の動作は変更しないでください
• パフォーマンスが劣化しないよう注意してください

パフォーマンス要件

• N+1問題を完全に解消すること

確認項目

• 既存のテストがすべて通ること
• N+1問題が解消されていること
• Rubocop違反がないこと

[thmz337]

@komagata

お疲れ様です。QueryObjectのテストの件で1点ご相談させてください。
QueryObjectパターンガイド（新人向け）のテストの項目について、assert_no_queriesやassert_queriesを使用していますがこちらrails6にはない機能らしく、代替を探していますが良い候補が見つかっていません。

こちら何か良い代替案があれば、ご教示頂けますと幸いです。

[komagata]

@thmz337 想定しているデータが返ってきているかどうかを確かめる感じで良いと思います。assert_equalとかになると思います。

[thmz337]

@komagata

お疲れ様です。五月雨に申し訳ございません。
今回の注意点に、既存の動作を変更しないことが含まれていますが、
Product.unchecked_no_replied_productsについて、既存の実装においてはcheckedな提出物も拾ってきているようでして、そもそもの動作が誤っているように思います。
こちらも修正はしない方が良いでしょうか？

[komagata]

@thmz337 不具合がある場合はバグのIssueを作ってそちらを先にやっていただければありがたいです。
git blameなどを使えばどのコミットでその不具合が混入したのかも調べられるので調査お願いいたします。

[thmz337]

@komagata

お疲れ様です。#8993として、issueを作成しました。こちらポイントの割り当て等お願いできますでしょうか。

よろしくお願いいたします。

[komagata]

@thmz337 issueの作成ありがとうございます。ポイントを割り振りました。