【Rails】検索機能の実装について

Question

Railsの検索機能についてわからない点がございます。
現在Railsにて書籍のレビューアプリを作っており、検索機能も実装しております。

現状だと検索ページに遷移すると登録のある書籍が全て反映する状態であり、そこから検索をして絞っていくような形です。

しかし見栄え的にあまり良くないので、
検索をかけるまでは何も書籍が反映しない状況にしたいと考えております。

■現状
https://gyazo.com/1e31eaa9441c97d30e05f1b2709f3c7e

■理想
https://gyazo.com/893c489a3b68eca7eaa0d6efcb6eac5a

##products_controller
class ProductsController < ApplicationController
  before_action :authenticate_user!
  
  def index
    @products = Product.includes(:user).page(params[:page]).per(5).order("created_at DESC")
  end
  
  def new
    @product = Product.new
  end
  
  def create
    @products = Product.create(bookname: product_params[:bookname], author: product_params[:author], image: product_params[:image], user_id: current_user.id)
  end
  
  def show
    @product = Product.find(params[:id])
    @review = @product.reviews.all.count
    @products = Product.page(params[:id]).per(10).order("created_at DESC")
    @reviews = Review.order("created_at ASC").page(params[:page]).per(5)
  end
  
  def search
    @products = Product.where('bookname LIKE(?)', "%#{params[:keyword]}%")
  end
  
  private
  def product_params
    params.require(:product).permit(:bookname, :author, :image)
  end

end

##products_search.html.erb
<div class="contents-box">
  <div class="container-left">
    <div class="container-search-box">
      <h3>書籍を検索する</h3>
      <%= form_tag('/products/search', method: :get) do %>
        <input id="sbox" name="keyword" type="text" placeholder="キーワードを入力" />
        <input id="sbtn" type="submit" value="検索" />
      <% end %>
      <form accept-charset="utf-8" action="/" class="js-search-submit" method="get">
      </form>
    </div>
    <div class="container-search-result">
      <h3>検索一覧</h3>
      <div class="no-search">
        <p>検索でレビューを書きたい書籍が見つからなかった場合はまず書籍をご登録ください</p>
        <a class="right-btn" href="/products/new">書籍を登録する</a>
      </div>
      <br>
      <% @products.each do |product| %>
      <a class="container-search-result-box" href="/products/<%= product.id %>" title="<%= product.bookname %>">
        <div class="container-search-result-box-left" >
          <% if product.image.attached? %>
          <%= image_tag product.image %>
        <% end %>
        </div>
        <div class="container-search-result-box-right">
          <h2><%= product.bookname %></h2>
          <%= product.author %>
        </div>
      </a>
      <% end %>
    </div>
  </div>
  <div class="container-right">
    <%= link_to "書籍を検索する", "/products/search", class: "right-btn" %>
    <%= link_to "マイページ", user_path(current_user), class: "right-btn" %>
    <a class="right-btn" href=""></a>
  </div>
</div>

##product.rb
class Product < ApplicationRecord
  
  has_one_attached :image
  
  has_many :reviews
  belongs_to :user
  has_many :comments
  
end

どなたかご教示いただけますと幸いです。

Answer 1

def search
  @products = params[:keyword].present? ? Product.where('bookname LIKE(?)', "%#{params[:keyword]}%") : []
end

とすると、キーワードが空の時に全データが引っかかるってことはなくなります。

また空の時に <% @products.each do |product| %>~<% end %>は@productsがからのとき不要なので、if文で空のときはEachしないとかするといいかもしれないです。

Comments

[yastinbieber]

ご回答ありがとうございました！
無事希望通りの実装ができました！！