良く分からないアクションが反応してエラーになる

Question

前提・実現したいこと

gemはransackで
検索機能を実装していて、検索して、検索結果処理をして、検索結果のビューを表示したいのですが、検索ボタンを押した後にdrinks#search_drinkを反応させたいのですが
何故かdrinks#showが反応して、

Couldn't find Drink with 'id'=searchdrink

といったエラーが起きてます

もっと詳しくいうと、カテゴリ検索的なことがしたくて、
active hashを用いてカテゴリ選択をしていて、

このようにコーヒーの濃さで検索できるようにしたいです

そもそもactive hashを用いたカテゴリだと検索ができないのでしょうか？

　とにかくsearch_drinkアクションを反応させたいです

発生している問題・エラーメッセージ

ActiveRecord::RecordNotFound in DrinksController#show
Couldn't find Drink with 'id'=searchdrink
Extracted source (around line #12):

  def show
    @drink = Drink.find(params[:id]) ここがハイライトされてます

    @user = @drink.user
  end


Rails.root: /Users/soichirohara/coffee_passport

Application Trace | Framework Trace | Full Trace
app/controllers/drinks_controller.rb:12:in `show'

考えられる原因としてはroutes.rbの記述がうまくいっていない
検索フォームの記述が間違ってる

該当のソースコード

drinks/index.html.erb

<h1>
  商品検索
</h1>
<%= search_form_for @p,url: drinks_searchdrink_path do |f| %>
<%# search_form_for @p(検索オブジェクトを渡すことで
    検索フォームを生成しています)%>
  <%= f.collection_select :name_eq,@drink_body,:name,:name,include_blank: '---'%>
  <%# 第一引数: 検索したいカラム名 
      2: 配列データを指定(今回はdrinkの配列)
        いわば検索したいカラムのデータ元
      3: 表示する際に参照するDBのカラム名
      4: 実際に表示されるカラム名
      5 何も選択してない時に表示される内容%>
  <br>
  <%= f.submit '検索'%>
<% end %>

routes.rb

Rails.application.routes.draw do
  root to: 'drinks#index'
  get    '/login',   to: 'sessions#new'
  post   '/login',   to: 'sessions#create'
  delete '/logout',  to: 'sessions#destroy'
  resources :users do
    member do
      get :following,:followers
      # memberメソッドを使うと
      # ユーザーidが含まれてるURlを扱うようになる
    end
  end
  resources :drinks, only: [:index,:new,:show,:create,:destroy] do
    collection do
      get 'search'
  end
    
    
    resources :trades,only: [:index,:new,:create]
  end
  post '/drinks/:id/trades/new', to: 'trades#create'
  get '/drinks/searchdrink',  to: 'drinks#search_drink'
  resources :relationships, only: [:create,:destroy]
end

drinks_controller

before_action :searched_drink_object,only:[:index,:search_drink]

  # GET drinks/search_drinks
  def search_drink
    @results = @p.result.includes(:body)
    # 作った検索オブジェクト@pに対して.result
    # とすると検索結果を取得できる
    # 検索条件に該当した商品が@pに格納されているので、
    # その格納されている値を表示する役割があります
  end

  private
  def drink_params
    params.require(:drink_tag).permit(:name,:price,:explain,:image,:tag_name,:region_id,:body_id,:acidity_id,:processing_id).merge(user_id: current_user.id)
  end

  def searched_drink_object
    @p = Drink.ransack(params[:q])
    # キー(:q)を使ってDrinksテーブルから商品情報を探しています
    # @pという名前の検索オブジェクトを生成
  end
end

body.rb

class Body < ActiveHash::Base
  
  self.data = [
    { id: 1, name: '--' },
    { id: 2, name: 'LIGHT(軽い)' },
    { id: 3, name: 'MEDIUM(ほどよい)' },
    { id: 4, name: 'FULL(しっかり)' }
  ]
end

ログ

Started GET "/rails/active_storage/disk/eyJfcmFpbHMiOnsibWVzc2FnZSI6IkJBaDdDRG9JYTJWNVNTSWhObUpuZEhSMWRUVTNNR3huWTJwNGNqaHpaV0kyT1dWaU5HTnZZUVk2QmtWVU9oQmthWE53YjNOcGRHbHZia2tpVldsdWJHbHVaVHNnWm1sc1pXNWhiV1U5SW1sdFp5MXdZWE56Y0c5eWRDMHdOQzB4TG1wd1p5STdJR1pwYkdWdVlXMWxLajFWVkVZdE9DY25hVzFuTFhCaGMzTndiM0owTFRBMExURXVhbkJuQmpzR1ZEb1JZMjl1ZEdWdWRGOTBlWEJsU1NJUGFXMWhaMlV2YW5CbFp3WTdCbFE9IiwiZXhwIjoiMjAyMC0xMi0wOVQxMzoxMzozOC44NDVaIiwicHVyIjoiYmxvYl9rZXkifX0=--725748959cb475c186bdf86c23e9cfad96b9903d/img-passport-04-1.jpg?content_type=image%2Fjpeg&disposition=inline%3B+filename%3D%22img-passport-04-1.jpg%22%3B+filename%2A%3DUTF-8%27%27img-passport-04-1.jpg" for ::1 at 2020-12-09 22:09:46 +0900
Processing by ActiveStorage::DiskController#show as JPEG
  Parameters: {"content_type"=>"image/jpeg", "disposition"=>"inline; filename=\"img-passport-04-1.jpg\"; filename*=UTF-8''img-passport-04-1.jpg", "encoded_key"=>"eyJfcmFpbHMiOnsibWVzc2FnZSI6IkJBaDdDRG9JYTJWNVNTSWhObUpuZEhSMWRUVTNNR3huWTJwNGNqaHpaV0kyT1dWaU5HTnZZUVk2QmtWVU9oQmthWE53YjNOcGRHbHZia2tpVldsdWJHbHVaVHNnWm1sc1pXNWhiV1U5SW1sdFp5MXdZWE56Y0c5eWRDMHdOQzB4TG1wd1p5STdJR1pwYkdWdVlXMWxLajFWVkVZdE9DY25hVzFuTFhCaGMzTndiM0owTFRBMExURXVhbkJuQmpzR1ZEb1JZMjl1ZEdWdWRGOTBlWEJsU1NJUGFXMWhaMlV2YW5CbFp3WTdCbFE9IiwiZXhwIjoiMjAyMC0xMi0wOVQxMzoxMzozOC44NDVaIiwicHVyIjoiYmxvYl9rZXkifX0=--725748959cb475c186bdf86c23e9cfad96b9903d", "filename"=>"img-passport-04-1"}
Completed 304 Not Modified in 1ms (ActiveRecord: 0.0ms | Allocations: 362)


Started GET "/rails/active_storage/disk/eyJfcmFpbHMiOnsibWVzc2FnZSI6IkJBaDdDRG9JYTJWNVNTSWhPV0p2YW1jMmQyODBibXB4YmpKd2VIWjNhSEp2TXpZeWJXVm1hUVk2QmtWVU9oQmthWE53YjNOcGRHbHZia2tpQWNacGJteHBibVU3SUdacGJHVnVZVzFsUFNJbE0wWWxNMFlsTTBZbE0wWWxNMFlsTTBZbE0wWWxNMFlsTTBZZ01qQXlNQzB4TWkwd05pQXhOQzR3Tnk0ME5TNXdibWNpT3lCbWFXeGxibUZ0WlNvOVZWUkdMVGduSnlWRk15VTRNaVZDT1NWRk15VTRNaVZCUmlWRk15VTRNeVZCUVNWRk15VTRNeVZDUXlWRk15VTRNeVZDTXlWRk15VTRNaVZDTnlWRk15VTRNeVZCTnlWRk15VTRNeVU0TXlWRk15VTRNeVU0T0NVeU1ESXdNakF0TVRJdE1EWWxNakF4TkM0d055NDBOUzV3Ym1jR093WlVPaEZqYjI1MFpXNTBYM1I1Y0dWSklnNXBiV0ZuWlM5d2JtY0dPd1pVIiwiZXhwIjoiMjAyMC0xMi0wOVQxMzoxMzozOC43ODZaIiwicHVyIjoiYmxvYl9rZXkifX0=--7aaa5870ab16a67f06b3cf82f3184d68c9ba09b3/%E3%82%B9%E3%82%AF%E3%83%AA%E3%83%BC%E3%83%B3%E3%82%B7%E3%83%A7%E3%83%83%E3%83%88%202020-12-06%2014.07.45.png?content_type=image%2Fpng&disposition=inline%3B+filename%3D%22%253F%253F%253F%253F%253F%253F%253F%253F%253F+2020-12-06+14.07.45.png%22%3B+filename%2A%3DUTF-8%27%27%25E3%2582%25B9%25E3%2582%25AF%25E3%2583%25AA%25E3%2583%25BC%25E3%2583%25B3%25E3%2582%25B7%25E3%2583%25A7%25E3%2583%2583%25E3%2583%2588%25202020-12-06%252014.07.45.png" for ::1 at 2020-12-09 22:09:46 +0900
Processing by ActiveStorage::DiskController#show as PNG
  Parameters: {"content_type"=>"image/png", "disposition"=>"inline; filename=\"%3F%3F%3F%3F%3F%3F%3F%3F%3F 2020-12-06 14.07.45.png\"; filename*=UTF-8''%E3%82%B9%E3%82%AF%E3%83%AA%E3%83%BC%E3%83%B3%E3%82%B7%E3%83%A7%E3%83%83%E3%83%88%202020-12-06%2014.07.45.png", "encoded_key"=>"eyJfcmFpbHMiOnsibWVzc2FnZSI6IkJBaDdDRG9JYTJWNVNTSWhPV0p2YW1jMmQyODBibXB4YmpKd2VIWjNhSEp2TXpZeWJXVm1hUVk2QmtWVU9oQmthWE53YjNOcGRHbHZia2tpQWNacGJteHBibVU3SUdacGJHVnVZVzFsUFNJbE0wWWxNMFlsTTBZbE0wWWxNMFlsTTBZbE0wWWxNMFlsTTBZZ01qQXlNQzB4TWkwd05pQXhOQzR3Tnk0ME5TNXdibWNpT3lCbWFXeGxibUZ0WlNvOVZWUkdMVGduSnlWRk15VTRNaVZDT1NWRk15VTRNaVZCUmlWRk15VTRNeVZCUVNWRk15VTRNeVZDUXlWRk15VTRNeVZDTXlWRk15VTRNaVZDTnlWRk15VTRNeVZCTnlWRk15VTRNeVU0TXlWRk15VTRNeVU0T0NVeU1ESXdNakF0TVRJdE1EWWxNakF4TkM0d055NDBOUzV3Ym1jR093WlVPaEZqYjI1MFpXNTBYM1I1Y0dWSklnNXBiV0ZuWlM5d2JtY0dPd1pVIiwiZXhwIjoiMjAyMC0xMi0wOVQxMzoxMzozOC43ODZaIiwicHVyIjoiYmxvYl9rZXkifX0=--7aaa5870ab16a67f06b3cf82f3184d68c9ba09b3", "filename"=>"スクリーンショット 2020-12-06 14.07.45"}
Completed 304 Not Modified in 2ms (ActiveRecord: 0.0ms | Allocations: 355)


Started GET "/drinks/searchdrink?q%5Bname_eq%5D=LIGHT%28%E8%BB%BD%E3%81%84%29&commit=%E6%A4%9C%E7%B4%A2" for ::1 at 2020-12-09 22:09:49 +0900
Processing by DrinksController#show as HTML
  Parameters: {"q"=>{"name_eq"=>"LIGHT(軽い)"}, "commit"=>"検索", "id"=>"searchdrink"}
  Drink Load (0.3ms)  SELECT `drinks`.* FROM `drinks` WHERE `drinks`.`id` = NULL LIMIT 1
  ↳ app/controllers/drinks_controller.rb:12:in `show'
Completed 404 Not Found in 3ms (ActiveRecord: 0.3ms | Allocations: 1485)

と、showアクションが反応してしまいます

パラメーターは

{"q" => {"name_eq" => "MEDIUM（表される）"}、 "commit" => "検索"、 "id" => "searchdrink"}

　でした

試したこと

rails routesを実行して

    drinks_searchdrink GET    /drinks/searchdrink(.:format)                                                            drinks#search_drink

があることを確認できました

これをurl指定してると思うので、何故drinks#showが反応してしまうのか、
search_drinkアクションを反応させるにはどうしたらいいか
ご教授いただけると幸いです

Answer 1

Railsのルーティングは、ルーティングファイルの「上からの記載順に」マッチします。このため、たとえばresources :photosというルーティングがget 'photos/poll'よりも前の行にあれば、resources行のshowアクションがget行の記述よりも優先されますので、get行のルーティングは有効になりません。これを修正するには、get行をresources行 よりも上 の行に移動してください。これにより、get行がマッチするようになります。

Rails のルーティング

下記のルーティングでは上に書いてあるdrinks#show が優先されています。

resources :drinks, only: [:index,:new,:show,:create,:destroy]
get '/drinks/searchdrink',  to: 'drinks#search_drink'

Comments

[divclass123]

解決できました、ありがとうございました！