RailsのDBクエリメソッドをより効率的にする方法

あなたは正しいです

私はDBレベルではなくRubyで多くのことをしているので、これは非常に効率的だとは思いません。

ActiveRecordは単純なクエリには適していますが、物事が複雑になっている場合は、プレーンSQLを使用するのが妥当です。それでは、テストケースに一致するクエリを作成してみましょう。

1）この呼び出しtest("politics people china", true)の場合、クエリは次のようになります。

SELECT DISTINCT ON (AR.id) AR.*
FROM articles AR
  JOIN articles_hashtags AHSH ON AHSH.article_id = AR.id
  JOIN hashtags HSH ON HSH.id = AHSH.hashtag_id
WHERE LOWER(HSH.value) IN ('politics', 'people', 'china')
ORDER BY AR.id;

（結合テーブルの名前がわからないので、名前が付けられていると仮定しますarticles_hashtags）。

無地でシンプルに：私たちは、からデータを取りarticles、内2と合流使用してテーブルarticles_hashtagsとhashtagsし、where私たちが見てみたいハッシュタグをフィルタリング条件を、。そして最終的には、そのハッシュタグが付いたすべての記事が表示されます。フィルタリングするINハッシュタグの数に関係なく、リストにハッシュタグが1つしかない場合でも、ステートメントは適切に機能します。

DISTINCT ONに注意してください：同じ記事が特定のハッシュタグリストから複数のハッシュタグを持っている場合、結果セットから重複する記事を削除するために必要です。

2）呼び出しの場合test("politics people china", false)、クエリはもう少し複雑です。ハッシュタグを付けた記事は除外する必要があります。したがって、ハッシュタグの異なる記事だけでなく、ハッシュタグのない記事も返す必要があります。物事を単純に保つために、前のクエリを使用できます。

SELECT A.*
FROM articles A
WHERE A.id NOT IN (
    SELECT DISTINCT ON (AR.id) AR.id
    FROM articles AR
      JOIN articles_hashtags AHSH ON AHSH.article_id = AR.id
      JOIN hashtags HSH ON HSH.id = AHSH.hashtag_id
    WHERE LOWER(HSH.value) IN ('politics', 'people', 'china')
    ORDER BY AR.id
);

ここではすべての記事を取得していますが、特定のハッシュタグのいずれかを持っている記事を取得しています。

3）これらのクエリをRubyメソッドに変換すると、次のようになります。

def test3(hashtags, include = true)
  # code guard to prevent SQL-error when there are no hashtags given
  if hashtags.nil? || hashtags.strip.blank?
    return include ? [] : Article.all.to_a
  end

  basic_query = "
    SELECT DISTINCT ON (AR.id) AR.*
    FROM #{Article.table_name} AR
      JOIN articles_hashtags AHSH ON AHSH.article_id = AR.id
      JOIN #{Hashtag.table_name} HSH ON HSH.id = AHSH.hashtag_id
    WHERE LOWER(HSH.value) IN (:hashtags)
    ORDER BY AR.id"

  query = if include
            basic_query
          else
            "SELECT A.*
            FROM #{Article.table_name} A
            WHERE A.id NOT IN (#{basic_query.sub('AR.*', 'AR.id')})"
          end

  hashtag_arr = hashtags.split(' ').map(&:downcase) # to convert hashtags string into a list

  Article.find_by_sql [query, { hashtags: hashtag_arr }]
end

上記のメソッドは、空かどうかに関係なく、条件に一致する記事の配列を返します。