【Ruby on Rails】gem 'ransack'を利用した複数ワード検索について

Question

###前提・実現したいこと
フリーワード検索でスペースを利用した複数ワード検索をand条件で検索したいです。

（例）新宿営業支店　野呂
↑新宿営業支店に所属している野呂さんを検索する場合

###該当のソースコード

def search
    words = params[:q].delete(:lastname_or_firstname_or_lastname_kana_or_firstname_kana_or_email_or_section_name_or_offices_name_cont)
    if words.present?
      params[:q][:groupings] = []
      words.split(/[ 　]/).each_with_index do |word, i| #全角空白と半角空白で切って、単語ごとに処理します
        params[:q][:groupings][i] = { lastname_or_firstname_or_lastname_kana_or_firstname_kana_or_email_or_section_name_or_offices_name_cont: word }
      end
    end
    @q = User.search(search_params)
    @users = @q.result(distinct: true).order(id: :desc).page(params[:page])
end

def search_params
      params.require(:q).permit(
        :lastname_or_firstname_or_lastname_kana_or_firstname_kana_or_email_or_user_id_or_section_name_department_name_or_division_name_or_position_name_or_offices_name_cont,
        {:offices_id_in => [], :section_id_in => [], :department_id_in => [], :division_id_in => [], :position_id_in => []}
      )
end

###試したこと
非常に不恰好ですが、以下のコードで動いてはいます。
しかし、.deleteしているため？検索後のフリーワードのinputに検索値が入りません。

また、手探りのコードで非常に不恰好のような気がしています。
スッキリできるコードの書き方があれば、そちらも助言いただければ幸いです。

def search
    if params[:q][:lastname_or_firstname_or_lastname_kana_or_firstname_kana_or_email_or_user_id_or_section_name_or_department_name_or_division_name_or_position_name_or_offices_name_cont].nil?
      @q = User.search(search_params)
      @users = @q.result(distinct: true).order(id: :desc).page(params[:page])
    else
      words = params[:q].delete(:lastname_or_firstname_or_lastname_kana_or_firstname_kana_or_email_or_user_id_or_section_name_or_department_name_or_division_name_or_position_name_or_offices_name_cont)
      if words.present?
        params[:q][:groupings] = []
        words.split(/[ 　]/).each_with_index do |word, i| #全角空白と半角空白で切って、単語ごとに処理します
          params[:q][:groupings][i] = { lastname_or_firstname_or_lastname_kana_or_firstname_kana_or_email_or_user_id_or_section_name_or_department_name_or_division_name_or_position_name_or_offices_name_cont: word }
        end
      end
      @q = User.ransack(params[:q])
      @users = @q.result(distinct: true).order(id: :desc).page(params[:page])
    end
end

###補足情報(言語/FW/ツール等のバージョンなど)
Rails (5.1.2)
ransack （1.8.4）

■参照URL
https://qiita.com/asukiaaa/items/a877534b8155819e22a9

Comments

[gouf]

delete メソッドを使っている意図が掴めませんでした、その理由はなんですか? (dup メソッドを挟まないのですか?)

Answer 1

問題の解決になるか検証できないのでわかりませんが: コードを書き換えてみました

def search
  search_target_keys = %w[
    lastname
    firstname
    lastname_kana
    firstname_kana
    email
    user_id
    section_name
    department_name
    division_name
    position_name
    offices_name
  ].freeze # 「このあと変更されるかもしれない」考慮負担の軽減(ただ、定数を使う場面かもしれない)

  # TODO: この変数を用いた処理の部分、文字列型のままか/シンボル型で動くか/動かないか調べる
  # 対応が必要であれば .to_sym などの変換を試す
  fuzzy_search_keys = (search_target_keys.join('_or_') + '_cont').freeze

  # if は最後に置いた値を代入できる
  @q =
    if params[:q][fuzzy_search_keys].nil?
      User.search(search_params)
    else
      words = params[:q].dup.delete(fuzzy_search_keys)
      if words.present?
        params[:q][:groupings] = []
        words.split(/[ 　]/).each_with_index do |word, i| #全角空白と半角空白で切って、単語ごとに処理します
          params[:q][:groupings][i] = { fuzzy_search_keys => word }
        end
      end
      User.ransack(params[:q])
    end

  @users = @q.result(distinct: true).order(id: :desc).page(params[:page])
end

「if words.present?」全体の書き換えもしてみたかったのですが、コードの全体像がつかめないのでできませんでした
それと、この部分についても適切な処理名を付けてメソッドとして切り出して上げる必要があるかもしれません(メソッド全体が20行を越えているため)

何か参考になれば幸いです

Comments

[ssk]

gouf様

返信が遅くなってしまい、申し訳ございません。
ご回答ありがとうございます。

メソッド部分を切り出して、モデルに任せる形で実装することができました。