ジャンル検索においての重複のない抽出の仕方

Question

前提・実現したいこと

Windows 10 Home
ruby 2.5.1p57
Rails 5.2.4.2
SQLite 3.30.1

Progateで学習を終え、
Ruby on Rails5でショップをまとめ、検索出来るサイトの作成中なのですが、
ジャンル指定でのショップ絞り込み機能の実装中に躓いてしまいました。

Genreモデルは

id	genre_id	floor	name
1	1	1	本・コミック・雑誌
12	1	2	本

というカラムを持っています。

発生している問題

一つのショップに、同じgenre_idのジャンルを複数設定してある場合
ShopsGenre.where(genre_id: @search_genre.genre_id)では、
同じshop_idを持つデータも抜き取ってしまい、出力すると画像のように
ショップが重複して表示されてしまいます。

現在のソースコードは以下の通りです

  before_action :set_search_genre

  def set_search_genre
    @search_genre = Genre.find_by(id: session[:search_genre_id])
  end

  def index
    # session[:search_genre_id]セット
    if @search_genre
      @genres = Genre.where(genre_id: @search_genre.genre_id, floor: 2)
    else
      @genres = Genre.where(floor: 1)
    end
    if params[:genre]
      @genre = Genre.find_by(id: params[:genre])
      session[:search_genre_id] = @genre.id
      redirect_to("/shops/index")
    end
    -----------------------------------------------------------------------------------
    if @search_genre
      if params[:keyword]
        @shops = Shop.where("name Like ?", "%#{params[:keyword]}%").order(id: :desc)
      else
        if @search_genre.floor == 1
          @hit_shops = ShopsGenre.where(genre_id: @search_genre.genre_id)
        else
          @hit_shops = ShopsGenre.where(genre_column_id: @search_genre.id)
        end
        if @hit_shops != nil
          @shops = []
          @hit_shops.each do |hit_shop|
            @shops.insert(0, Shop.find_by(id: hit_shop.shop_id ) )
          end
        end
      end
    -----------------------------------------------------------------------------------
    else
      if params[:keyword]
        @shops = Shop.where("name Like ?", "%#{params[:keyword]}%").order(id: :desc)
      else
        @shops = Shop.all.order(id: :desc)
      end
    end
    if @shops
      @result_amount = @shops.count
    end
  end

ShopsGenreモデルの内容

試したこと

ShopsGenre.select(:shop_id).distinctだと、
重複はなくなりますが、where(genre_id: @search_genre.genre_id)の
指定ができないので、genre_idが異なるジャンルを設定してあるショップも
表示されてしまいます。
ShopsGenre.select(:shop_id).distinctと
ShopsGenre.where(genre_id: @search_genre.genre_id)を
両方適用するような処理はどのようにすればいいのでしょうか。

Answer 1

genre_idとfloorでジャンル内の階層を表現していると仮定しますが
genre_idじゃなくidで絞り込むのが良いと思います。

Comments

[choco_duck]

回答ありがとうございます!
仰る通り、genre_idとfloorでジャンル内の階層を表現しています。
絞り込み条件に、floorが2のジャンルを選択したときは、
idで絞り込みをしているのですが、
floorが1のジャンルを選択したときは、
そのジャンル(同じgenre_id)の子(floorが2)のジャンルが
設定されているショップを一覧表示したいのです。
ショップに設定するジャンルの階層はすべてfloorが2です。

Answer 2

pluckとuniqを用いることで解決することが出来ました。
if @search_genre.floor == 1の中の処理を以下のように変更することで、
目的の動作を実現することが出来ました。

if @search_genre.floor == 1
    @hit_duplication_shops = ShopsGenre.where(genre_id: @search_genre.genre_id)
    @hit_duplication_shops_ids = @hit_duplication_shops.pluck(:shop_id)
    @hit_shops_ids = @hit_duplication_shops_ids.uniq
    @hit_shops = []

    @hit_shops_ids.each do |hit_shops_id|
      @hit_shops.insert(0, ShopsGenre.find_by(shop_id: hit_shops_id))
    end
else