railsでの絞り込み検索機能について

Question

プログラミングを独学で勉強しており現在railsにてポートフォリオ制作をしております。
お店の口コミサイトを作っておりエリアにての絞り込みとタグでの検索機能を追加したのですが各それぞれの絞り込み検索はできるのですが指定したエリア内でのタグの検索機能がうまく実装できず困っております。 whereやjoinなど色々試しましたが上手く使えず
現状controllerでのindexのコードはこの様になっております。
コントローラーでの定義の仕方が悪いと思うのですがどの様に定義したらよろしいのでしょうか？
お店とタグは中間テーブルで紐付けていますがエリアはお店としか紐付けていないです。この場合エリアのidも中間テーブルで紐付けが必要なのでしょうか？
ご教授していただけると幸いです。

該当のソースコード

def index
    if params[:area_id].blank? && params[:tag_id].blank?
      @gyms = Gym.all.order("created_at DESC").page(params[:page]).per(4)
    else
      @gyms = Area.find(params[:area_id]).gyms if params[:area_id].present?
      @gyms = Tag.find(params[:tag_id]).gyms if params[:tag_id].present?
      @gyms = @gyms.order("created_at DESC").page(params[:page]).per(4)
    end
end

viewではこの様にしておりeachで取り出しindex pageに表示させています。

.ml-auto.gyms__linkBox
    = form_tag gyms_path, method: :get, class: 'gyms__searchForm' do
      = select_tag :tag_id,
      options_from_collection_for_select(Tag.all, :id, :name, params[:tag_id]),
        { prompt: 'こだわりから探す', class: 'form-control gyms__select'}
      = form_tag gyms_path, method: :get, class: 'gyms__searchForm' do
        = select_tag :area_id,
        options_from_collection_for_select(Area.all, :id, :name, params[:area_id]),
          { prompt: 'エリアを選択', class: 'form-control gyms__select'}
        .form-group
          = submit_tag('検索', class: 'btn btn-primary')
.row
  - @gyms.each do |gym|
    .col-lg-3.col-md-4.col-sm-6.col-6
      .card style="width: auto;"
        a href="/gyms/#{gym.id}"
          = image_tag gym.picture.url, class: "card-img-top rounded" , width: 'auto', height: 170
        .card-body
          h6.card-title = link_to gym.name, gym
          h6 = gym.area.name
          a.average-review-rating data-score="#{gym.reviews.average(:rating)}"
          = gym.reviews.average(:rating)
          .number-of-reviews
            = gym.reviews.count
            | 件のレビュー

class Gym < ApplicationRecord
  belongs_to :user
  belongs_to :area
  has_many :reviews, dependent: :delete_all
  has_many :gym_tag_relations, dependent: :delete_all
  has_many :tags, through: :gym_tag_relations
end

class Area < ApplicationRecord
  has_many :gyms
end

class Tag < ApplicationRecord
  has_many :gym_tag_relations, dependent: :delete_all
  has_many :gyms, through: :gym_tag_relations
end

class GymTagRelation < ApplicationRecord
  belongs_to :gym
  belongs_to :tag
end

補足情報（FW/ツールのバージョンなど）

rails 5.1

Answer 1

３つのモデルの関連部分の定義を載せたほうが良いですね。
「私ならこうする」と同じである前提で。
Ransack という gem を使うと楽なのですが。

@gyms = Gym.all.order("created_at ESC")
@gyms = @gyms.where(area_id: params[:area_id]) if params[:area_id].present?
@gyms = @gyms.where(tag_id: params[:tag_id]) if params[:tag_id].present?

@gyms = gims.page(params[:page]).per(4)

で。

Comments

[Pichan.]

ご回答ありがとうございます。
ご提示して頂けたコードを試したところtag_idのカラムが存在しないとエラーになってしまいました。
PG::UndefinedColumn: ERROR: column gyms.tag_id does not exist
この場合view側の定義に問題がありtag_idのデータを拾えてないんでしょうか？
それともそのransackを使った方がいいでしょうか？初心者で申し訳ありません。。ransackも調べてみます。
３つのモデルも追記させて頂いたのでお時間あればまたアドバイスよろしくお願いします。