写真投稿サイトの検索機能実装の際に 検索結果が全ての投稿表示される

Question

写真のタイトルに対して検索をかけるようにしたいのですが
検索した時にveiwにて全てが反映されてしまいます。
データベースには写真のタイトルはtextカラムに保存するようにしています。

ファイル名<search.html.haml>
= text_field_tag :search, params[:keyword]
にparams[:keyword]追加して見ましたが検索結果は全て反映で効果ありませんでした。

tweet.rbのif文がうまく読めてないように思います。

原因教えてくださいよろしくお願いします。

ファイル名<tweets.controller.rb>

 def search
    @tweets = Tweet.search(params[:keyword])
 end

ファイル名<tweet.rb>

def self.search(search)
    if search
      Tweet.where('text LIKE(?)', "%#{search}%")
    else
      Tweet.all
    end
end

ファイル名<search.html.haml>

.top-page
    .menu
      = form_tag(tweets_searches_path,method: 'get') do
        = text_field_tag :search
        = submit_tag 'Search', name: nil
    - @tweets.each do |tweet|
      .main-photo{style: "background-image: url(#{tweet.image});"}

ファイル<routes.rb>
Rails.application.routes.draw do

  devise_for :users
  root to: "tweets#index"
  namespace :tweets do
    resources :searches, only: :index
  end
  resources :tweets do
    resources :comments, only: [:create, :destroy]
  end  
  resources :users, only: :show
  
end

ファイル名<tweets.controller.rb>
class TweetsController < ApplicationController
  before_action :set_tweet, only: [:edit, :show]
  before_action :move_to_index, except: [:index, :show]

  def index
    @tweets = Tweet.includes(:user).order("created_at DESC").page(params[:page]).per(6)
    @random = Tweet.order("RAND()").limit(1)

  end

  def new
    @tweet = Tweet.new
  end

  def create
    Tweet.create(tweet_params)
    
  end  

  def destroy
    tweet = Tweet.find(params[:id])
    tweet.destroy
 
  end

  def edit
    
  end

  def update
    tweet = Tweet.find(params[:id])
    tweet.update(tweet_params)
  end

  def show 
    @comment = Comment.new
    @comments = @tweet.comments.includes(:user) 
  end

  def search
    @tweets = Tweet.search(params[:keyword])
  end

  private
  def tweet_params
    params.require(:tweet).permit(:image, :text).merge(user_id: current_user.id)
  end

  def set_tweet
    @tweet = Tweet.find(params[:id])
  end

  def move_to_index
    redirect_to action: :index unless user_signed_in?
  end

  
end

Answer 1

search.html.hamlのtext_field_tag :search の部分はtext_field_tag :keywordではないですか？

Comments

[MISIAN-MISIAN]

コメントありがとうございます。
text_field_tag :keyword
にしてもやはり検索結果全て表示されます。。

[peperoncino000]

コントローラでbinding.pryなどしてparamsにパラメータが入っているか確認できますか？

[MISIAN-MISIAN]

どの部分にbinding.pryかけるのが正しいのでしょうか？

def search
@tweets = Tweet.search(params[:keyword])
end

@tweets = Tweet.search(params[:keyword])の前と後ろにbinding.pryかけて見ましたが
ターミナルでは止まらないので..
params入力しようがないです。
どうすればいいでしょうか？

[peperoncino000]

止まらないということは別のコントローラにリクエストが行ってそうですね。
routes.rbとtweets_controller.rbの全体を見せていただけますか？

[MISIAN-MISIAN]

元の質問編集して
routes.rbとtweets_controller.rb

コード書かせていただきました

よろしくお願いします

[peperoncino000]

routes.rbの
```
namespace :tweets do
resources :searches, only: :index
end
```
を
```
namespace :tweets do
get :searches, to: 'tweets#search'
end
```
に変更するとどうでしょう？

[MISIAN-MISIAN]

Routing Error
uninitialized constant Tweets::TweetsController
が出ます。

検索かけた際のURLは以下です。
http://localhost:3000/tweets/searches?utf8=%E2%9C%93&search=%E3%83%9A%E3%83%B3%E3%82%AE%E3%83%B3

[peperoncino000]

ではnamaspace部分を消して
resources :tweets do
resources :comments, only: [:create, :destroy]
get 'searches', to: 'tweets#search', on: :collection
end
に変更してください

[MISIAN-MISIAN]

= form_tag(tweets_searches_path,method: 'get') do
の部分でNameError in Tweets#search
が発生してしまします。

[peperoncino000]

bin/rails routesで確認していただくとわかるのですが、tweets_searches_pathではなくsearches_tweets_pathになるかと思います。

[MISIAN-MISIAN]

http://localhost:3000/tweets
で検索かけた場合は全ての投稿表示されます。
そして下記のurlに飛んで
http://localhost:3000/tweets/searches?utf8=%E2%9C%93&search=%E3%83%96%E3%83%AA%E3%83%83%E3%82%B8
でもう一度検索かけると検索結果に引っかかる写真だけアップされるようになりました。

[peperoncino000]

http://localhost:3000/tweets/searchesで検索をするのではないんですか？

[MISIAN-MISIAN]

全て元のコードに戻して試して見ましたが
http://localhost:3000/tweetsで検索した際には全ての投稿表示されて
検索先の
http://localhost:3000/tweets/searches?utf8=%E2%9C%93&search=%E3%83%96%E3%83%AA%E3%83%83%E3%82%B8
で検索した際は検索に引っかかった写真のみ表示されました。

[MISIAN-MISIAN]

http://localhost:3000/tweetsこれがtopページでここで検索すれば
検索先のurlに飛んで検索でヒットした写真のみ表示させるのが理想ですが。

現在はhttp://localhost:3000/tweetsで検索
検索先のurlに飛んで全ての投稿が表示されます。
そして検索先のurlのページにも検索formがありここで検索すると
検索にヒットした写真が表示されるのが現状です。

[peperoncino000]

index.html.hamlにも検索フォームがあるんですね。そちらは修正しましたか？

[MISIAN-MISIAN]

解決しましたありがとうございました

= text_field_tag :keyword
:keywordの部分が一箇所:searchになっていました。