Ruby on rails で画像の検索機能を実装したい

Question

前提

検索機能の付いた画像一覧を作成中です。
画像それぞれに詳細ページがあります。

実現したいこと

「検索」ボタンで、キーワードから該当画像を一覧で出したいです。

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

ActiveRecord::RecordNotFound in DProductionsController#show

Couldn't find Dproduction with 'id'=search

Extracted source (around line #63):
61
62  def set_dproduction
63    @d_production = Dproduction.find(params[:id])
64  end
65
66  def move_to_index
              

該当のソースコード

(d_productions_controller.rb)
class DProductionsController < ApplicationController 
  before_action :set_dproduction, only: [:show, :edit]
  before_action :authenticate_user!, only:[:edit, :new, :destroy]
  before_action :move_to_index, except: [:index, :show, :search]
 

  def index
    @d_production = Dproduction.includes(:user).order("created_at DESC")
  end

  def new
    @d_productions = Dproduction.new
  end

  def create
    @d_productions = Dproduction.create(d_production_params)
    if 
      @d_productions.save
      redirect_to root_path
      else
        render :new
    end
  end

  def show
    
  end

  def destroy
    d_production = Dproduction.find(params[:id])
    if d_production.destroy
      redirect_to root_path
    end
  end

  def update
    d_production = Dproduction.find(params[:id])
    if
      d_production.update(d_production_params)
      redirect_to d_production_path(dproduction.id)
    else
      redirect_to request.referer
    end
  end

  def edit
    unless user_signed_in? && current_user.id == @d_production.user_id
      redirect_to action: :index
    end
  end

  def search
    @d_production = Dproduction.search(params[:keyword]).order("created_at DESC")
  end

  private

  def d_production_params
    params.permit(:title, :catch_copy, :concept, :image).merge(user_id: current_user.id)
  end

  def set_dproduction
    @d_production = Dproduction.find(params[:id])
  end

  def move_to_index
    unless user_signed_in?
      redirect_to action: :index
    end
  end

end

(models > dproduction.rb)
class Dproduction < ApplicationRecord
  belongs_to :user
  has_many :comments, dependent: :destroy
  has_one_attached :image

  validates :title, presence: true
  validates :catch_copy, presence: true
  validates :concept, presence: true
  validates :image, presence: true

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

<%= form_with(url: search_d_productions_path, local: true, method: :get, class: "search-form") do |form| %>
  <%= form.text_field :keyword, placeholder: "投稿を検索する", class: "search-input" %>
  <%= form.submit "検索", class: "search-btn" %>
<% end %>
<div class="contents row">
  <% @d_productions.each do |d_production| %>
    <%= render partial: "d_production", locals: { d_production: d_production } %>
  <% end %>
</div>

試したこと

コントローラーの
def set_dproduction
@d_production = Dproduction.find(params[:id])
end
がおかしいのかと色々調べてやってみたのですが、どれも上手くいきませんでした。

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

rails _6.0.0です。

Comments

[yuma.inaura]

params[:id] の値はどうなってますか？

[yuma.inaura]

どんなタイミングでエラーが起こるのでしょうか？

[tamtamtime]

「検索」を押した時にエラーが出ます。

[tamtamtime]

Dproduction.find(params[:id])で

Traceback (most recent call last):
1: from (irb):1
NameError (undefined local variable or method `params' for main:Object)

と出ます。SequelProでは各画像にidが入っているのですが、やはりidがおかしいのでしょうか？
初歩的で申し訳ございません。ご教示いただけますと幸いです。

[tamtamtime]

@d_productionとすると「nil」が返ってきますが
「Dproduction.find(3)」とすると
中身が出てきます。

[yuma.inaura]

どの行でそのデバッグプリントを実行したんでしょう？

show でエラーが起きているように見えるんですが、showに検索フォームがあるのでしょうか？

[yuma.inaura]

>@d_productionとすると「nil」が返ってきますが

「とすると」というのはどんなことなんでしょうか？

[tamtamtime]

検索フォームはindexにあります。
```
<main class="main">
<div class="inner">
<%= form_with(url: search_d_productions_path, local:true, method: :get, class:"search-form") do |form| %>
<%= form.text_field :keyword, placeholder:"検索する", class:"search-input" %>
<%= form.submit "検索",class: "search-btn" %>
<% end %>
<div class="card__wrapper">
<%= render partial: 'dproduction', collection: @d_production %>
</div>
</div>
</main>
```

[yuma.inaura]

検索実行時にコントローラーのsearchアクションは実行されてそうですか？

[tamtamtime]

失礼しました、ターミナルのコマンドで@d_productionとすると、nilが返ってきます。

[tamtamtime]

ターミナルには
ActiveRecord::RecordNotFound (Couldn't find Dproduction with 'id'=search):

app/controllers/d_productions_controller.rb:63:in `set_dproduction'
Started GET "/d_productions/search?keyword=%E6%9C%9D%E9%A3%9F&commit=%E6%A4%9C%E7%B4%A2" for ::1 at 2023-01-07 16:42:50 +0900
Processing by DProductionsController#show as HTML
Parameters: {"keyword"=>"朝食", "commit"=>"検索", "id"=>"search"}
Dproduction Load (0.4ms) SELECT `dproductions`.* FROM `dproductions` WHERE `dproductions`.`id` = NULL LIMIT 1
↳ app/controllers/d_productions_controller.rb:63:in `set_dproduction'
Completed 404 Not Found in 1ms (ActiveRecord: 0.4ms | Allocations: 1389)

と出ておりますので、searchアクションは実行されているのではないかと思います。

[yuma.inaura]

>Processing by DProductionsController#show as HTML

showが実行されてませんか？

[tamtamtime]

ActiveRecord::RecordNotFound in DProductionsController#show

すみません、エラー画面で上記が出ますので、showを実行してエラーになったという事かもしれません。
とすると、searchアクションそのものが動いていないという事でしょうか？

[yuma.inaura]

>searchアクションそのものが動いていないという事でしょうか？

っぽいですよね
$ rails routes でルーティングを再確認するなどしたら良いかもしれません

[tamtamtime]

ありがとうございます！ルート確認しますと

new_user_session GET /users/sign_in(.:format) devise/sessions#new
user_session POST /users/sign_in(.:format) 　devise/sessions#create
destroy_user_session DELETE /users/sign_out(.:format) devise/sessions#destroy
new_user_password GET /users/password/new(.:format) devise/passwords#new
edit_user_password GET /users/password/edit(.:format) devise/passwords#edit
user_password PATCH /users/password(.:format) devise/passwords#update
PUT /users/password(.:format) devise/passwords#update
POST /users/password(.:format) devise/passwords#create
cancel_user_registration GET /users/cancel(.:format) devise/registrations#cancel
new_user_registration GET /users/sign_up(.:format) devise/registrations#new
edit_user_registration GET /users/edit(.:format) devise/registrations#edit
user_registration PATCH /users(.:format) devise/registrations#update
PUT /users(.:format) devise/registrations#update
DELETE /users(.:format) devise/registrations#destroy
POST /users(.:format) devise/registrations#create
root GET / d_productions#index
d_productions GET /d_productions(.:format) d_productions#index
POST /d_productions(.:format) d_productions#create
new_d_production GET /d_productions/new(.:format) d_productions#new
edit_d_production GET /d_productions/:id/edit(.:format) d_productions#edit
d_production GET /d_productions/:id(.:format) d_productions#show
PATCH /d_productions/:id(.:format) d_productions#update
PUT /d_productions/:id(.:format) d_productions#update
DELETE /d_productions/:id(.:format) d_productions#destroy
search_d_productions GET /d_productions/search(.:format) d_productions#search
GET /d_productions(.:format) d_productions#index
POST /d_productions(.:format) d_productions#create
GET /d_productions/new(.:format) d_productions#new
GET /d_productions/:id/edit(.:format) d_productions#edit
GET /d_productions/:id(.:format) d_productions#show
PATCH /d_productions/:id(.:format) d_productions#update
PUT /d_productions/:id(.:format) d_productions#update
DELETE /d_productions/:id(.:format) d_productions#destroy
user GET /users/:id(.:format) users#show
d_productions_new POST /d_productions/new(.:format) d_productions#create

rails_postmark_inbound_emails POST /rails/action_mailbox/postmark/inbound_emails(.:format) action_mailbox/ingresses/postmark/inbound_emails#create
rails_relay_inbound_emails POST /rails/action_mailbox/relay/inbound_emails(.:format) action_mailbox/ingresses/relay/inbound_emails#create
rails_sendgrid_inbound_emails POST /rails/action_mailbox/sendgrid/inbound_emails(.:format) action_mailbox/ingresses/sendgrid/inbound_emails#create
rails_mandrill_inbound_health_check GET /rails/action_mailbox/mandrill/inbound_emails(.:format) action_mailbox/ingresses/mandrill/inbound_emails#health_check
rails_mandrill_inbound_emails POST /rails/action_mailbox/mandrill/inbound_emails(.:format) action_mailbox/ingresses/mandrill/inbound_emails#create
rails_mailgun_inbound_emails POST /rails/action_mailbox/mailgun/inbound_emails/mime(.:format) action_mailbox/ingresses/mailgun/inbound_emails#create
rails_conductor_inbound_emails GET /rails/conductor/action_mailbox/inbound_emails(.:format) rails/conductor/action_mailbox/inbound_emails#index
POST /rails/conductor/action_mailbox/inbound_emails(.:format) rails/conductor/action_mailbox/inbound_emails#create
new_rails_conductor_inbound_email GET /rails/conductor/action_mailbox/inbound_emails/new(.:format) rails/conductor/action_mailbox/inbound_emails#new
edit_rails_conductor_inbound_email GET /rails/conductor/action_mailbox/inbound_emails/:id/edit(.:format) rails/conductor/action_mailbox/inbound_emails#edit
rails_conductor_inbound_email GET /rails/conductor/action_mailbox/inbound_emails/:id(.:format) rails/conductor/action_mailbox/inbound_emails#show
PATCH /rails/conductor/action_mailbox/inbound_emails/:id(.:format) rails/conductor/action_mailbox/inbound_emails#update
PUT /rails/conductor/action_mailbox/inbound_emails/:id(.:format) rails/conductor/action_mailbox/inbound_emails#update
DELETE /rails/conductor/action_mailbox/inbound_emails/:id(.:format) rails/conductor/action_mailbox/inbound_emails#destroy
rails_conductor_inbound_email_reroute POST /rails/conductor/action_mailbox/:inbound_email_id/reroute(.:format) rails/conductor/action_mailbox/reroutes#create
rails_service_blob GET /rails/active_storage/blobs/:signed_id/*filename(.:format) active_storage/blobs#show
rails_blob_representation GET /rails/active_storage/representations/:signed_blob_id/:variation_key/*filename(.:format) active_storage/representations#show
rails_disk_service GET /rails/active_storage/disk/:encoded_key/*filename(.:format) active_storage/disk#show
update_rails_disk_service PUT /rails/active_storage/disk/:encoded_token(.:format) active_storage/disk#update
rails_direct_uploads POST /rails/active_storage/direct_uploads(.:format) active_storage/direct_uploads#create

と出ました。search_d_productionsは確認できたのですが…。

[winterboum]

config/routes.rb に問題があります。
載せてください

[tamtamtime]

回答遅くなり申し訳ございません。以下がroutes.rbになります。

Rails.application.routes.draw do
devise_for :users
root to: 'd_productions#index'
resources :d_productions
resources :d_productions do
collection do
get 'search'
end
end
resources :users, only: :show

post '/d_productions/new', to: 'd_productions#create'

end

Answer 1

code はコメントにではなく。質問を編集してそちらに載せ替えてください。
インデントもなくなってしまってる。

resources :d_productions  # <= ここと
resources :d_productions do #<= ここで
collection do
get 'search'
end

ダブって定義されてます。
このため 2つ目の get search より 1つ目の get show の方が先にhitするので、そういうけっかになります。
2つ目だけにしましょう

Comments

[tamtamtime]

失礼いたしました、ご回答ありがとうございました。