Rails 子モデルのレコードを親モデルのカラムを使って検索したい

Question

Ruby on Railsで写真とテキストを投稿できるSNSを作っています。

検索機能を実装したいのですがうまく動きません。
具体的にはALL,MEN, WOMENの3つのボタンを作り、ALLの時には全てのレコード、MENの時には男性のレコード、WOMENの時には女性のレコードを表示する機能です。

現在のコードでは、ALLの時には全てのレコード、MENの時には男性のレコード、なぜかWOMENの時にも男性のレコードが表示されてしまいます。

ユーザーはusersテーブルという名前で、投稿はpostsテーブルという名前を使っていて、ユーザーはdeviseを使っています。
gender_idはActive Hashを使っています。
userとpostは、1:Nの関係です。
検索のためにsearchアクションを定義しています。
search.html.erbはindex.htl.erbと同じコードにしています。

###ソースコード

-----省略

def search
    @posts = Post.search(params[:gender_id])
  end

-----省略

Rails.application.routes.draw do
  devise_for :users
  root "posts#index"
  resources :posts, only: [:new, :create, :show, :destroy, :edit, :update] do
    resources :comments, only: :create
    collection do
      get "search"
    end
  end
  resources :users, only: :show
end

class Post < ApplicationRecord
  has_many_attached :images
  has_many         :comments, dependent: :destroy
  belongs_to       :user

  with_options presence: true do
    validates :text
    validates :images
    validates :user
  end

  validates :text, length: { maximum: 150, message: "は150文字以下にしてください" }

  def self.search(search)
    if search = 2
      Post.joins(:user).where("users.gender_id = 2")
    elsif search = 3
      Post.joins(:user).where("users.gender_id = 3")
    end
  end
end

-------省略

<ul class="gender-lists">
     <li>
      <%= link_to "ALL", root_path, class:"gender" %>
     </li>
     <li>
      <%= link_to "MEN", search_posts_path(gender_id:2), class:"gender" %>
     </li>
     <li>
      <%= link_to "WOMEN", search_posts_path(gender_id:3), class:"gender" %>
     </li>
    </ul>

-------省略

Answer 1

if search = 2 ですと search に 2 を代入し、その返り値 2 がnilでもfalseでもないので、if が成立　いつも成立します。

  def self.search(search)
    if search == 2
      Post.joins(:user).where("users.gender_id = 2")
    elsif search == 3
      Post.joins(:user).where("users.gender_id = 3")
    end
  end
end

ですね

Comments

[75ks]

ありがとうございます。
そのように`==`を使うと以下のエラーが出てしまいます。
`NoMethodError in Posts#search`
`Showing /Users/sekinekenta/projects/boast/app/views/posts/search.html.erb where line #19 raised:`
`undefined method `each' for nil:NilClass`

以下の部分を指摘されています。
19 `<% @posts.each do |post| %>`

RequestのParametersには、`{"gender_id"=>"2"}`が入っています。

[winterboum]

search == 2　search == 3　を　search.to_i 　にして

[75ks]

できました！本当に助かりました。ありがとうございます！
2、3といった数値比べているので、引数で渡したparamsを数値に直さなければいけないのですね。勉強になりました。