RubyonRailsにて、右記の正しい記述方法を知りたい。→ where.not(id: ◯◯, id: □□) ※◯◯は整数で□□はハッシュ

Question

前提・実現したいこと

独学の初学者です。RubyonRailsで個人アプリを作成しております。学習のためRubocopを初導入した所、どうしても自己解決出来ない点が生じてしまいました。

"user"をインクリメンタルサーチ出来るように記述していた所、開発完了後にRubocopから以下のお叱りを受けました。

#.rubocop_todo.yml

Offense count: 1
Lint/DuplicateHashKey:
  Exclude:
    - 'app/controllers/users_controller.rb'

確かに記述を見ると、以下の5行目のように重複したハッシュキー"id:"が存在していました。

#app/controllers/users_controller.rb

 1: class UsersController < ApplicationController
 2:  def index
 3:    return nil if params[:keyword] == ""
 4:
 5:    @users = User.where(['nickname LIKE?', "%#{params[:keyword]}%"]).where.not(id: current_user.id, id: params[:member]).limit(10)
 6:    respond_to do |format|
 7:      format.html
 8:      format.json
 9:    end
10:  end
11:end

ハッシュキー"id:"をまとめる記述方法を調べたものの、自己解決に至ることが出來ませんでした。基本的なことで大変恐れ入りますが、御教授いただければ幸いです。

試したこと

失敗例1)　〜　.where.not(id: [current_user.id, params[:member]])
失敗例2)　〜　.where.not(id: current_user.id, params[:member])
失敗例3)　〜　.where.not(id: current_user.id AND params[:member])
失敗例4)　〜　.where.not(id: (current_user.id, AND params[:member]))
失敗例5)　〜　.where.not(id: current_user.id.to_h, AND params[:member])
失敗例6)　〜　.where.not(id: {current_user.id.to_h, params[:member]})
　　　　...etc

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

実装状況自体は問題が無く、あくまでRubocopからのお叱りに対応したいという内容なので、関連するJavascriptの記述等は省略させていただきます。

Answer 1

.whereの列の条件として複数を渡したいときは、配列として渡すことができます。

.where.not(id: [current_user.id, params[:member]])

Comments

[maisumakun]

「失敗例」として上がっていますが、どのようになったのですか？

[maisumakun]

もっと原始的に、「.where.not(id: current_user.id).where.not(id: params[:member])」というようなやり方も考えられます。

[YoheiKobayashi]

御教授、ありがとうございます。
ご回答頂いた記述で当初私も問題ないと考えていたのですが、記述後にインクリメンタルサーチ部分の実装状況を確認した所、検索結果が何も表示されなくなる…という結果になりました。

[maisumakun]

params[:member]にnilが来ることはありますか？

[YoheiKobayashi]

「.where.not(id: current_user.id).where.not(id: params[:member])」
確かにこちらはうまくいきました！ただ、where.notが二重になっているため、rubocopからまた違う内容のお叱りを受けました。

rubocopに全て従う…ということ自体が、セオリーから外れているという考え方もあるのでしょうか^^;

[YoheiKobayashi]

nilが来ること、あります！

[maisumakun]

> rubocopに全て従う…ということ自体が、セオリーから外れているという考え方もあるのでしょうか^^;

両者の「お叱り」は、別レベルのものと考えたほうがいいです。Lint/系のものは、「文法的によろしくない」ものに出ます。

[YoheiKobayashi]

> 両者の「お叱り」は、別レベルのものと考えたほうがいいです。Lint/系のものは、「文法的によろしくない」ものに出ます。
御教授ありがとうございます！
「.where.not(id: current_user.id).where.not(id: params[:member])」
でやることで、rubocopからのLint系の指摘は無くなりました。
rubocopの基本をしっかりと抑える必要があると振り返りました…本当にありがとうございます！