画像投稿時のエラーの解消

Question

前提・実現したいこと

Cloud9を用いてRuby on railsでのWebアプリケーション制作を学んでいます。

画像投稿画面(send_pictures.html.erb)から画像を投稿した後、詳細画面(detail_request.html.erb)に戻り投稿した画像が表示されているという形にしようとしています。
投稿画面で画像を選択し送信ボタンを押すと、以下のようなエラーが出てしまい
色々試してみましたが解決方法が見つからず困っています。
解決方法をどなたか教えてください。よろしくお願いします。

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

エラーメッセージは以下の通りです。

NoMethodError in RequestsController#upload
undefined method `date' for nil:NilClass

FileUtils.mkdir_p("public/pictures/#{@request.date.year}/#{@request.date.month}/#{@request.date.day}") unless FileTest.exist?("public/pictures/#{@request.date.year}/#{@request.date.month}/#{@request.date.day}")

該当のソースコード

画像投稿画面(send_pictures.html.erb)

<h4><%= @request.date %></h4>
             
<%= form_tag("/requests/upload", {multipart: true}) do %>
 <input type = "file" name = "pic1">
 <input class="button_signup" type="submit" value="送信">
<% end %>

詳細画面(detail_request.html.erb)

<h4><%= @request.date %></h4>
                
<% if @request.pic1 != nil %>
　<% 5.times do |num| %>
　　<img src="<%= "/pictures/#{@request.date.year}/#{@request.date.month}/#{@request.date.day}/#{@request.date}_#{num+1}.jpg" %>" width=20%>
 <% end %>
<% end %>
<br>
                
<%= link_to("画像送信", "/requests/upload/#{@request.id}") %>

requests_controller.rb

class RequestsController < ApplicationController
  def detail_request
    @request = Request.find_by(id: params[:id])
  end
  
  def send_pictures
    @request = Request.find_by(id: params[:id])
  end
  
  def upload
    @request = Request.find_by(id: params[:id])
    
    FileUtils.mkdir_p("public/pictures/#{@request.date.year}/#{@request.date.month}/#{@request.date.day}") unless FileTest.exist?("public/pictures/#{@request.date.year}/#{@request.date.month}/#{@request.date.day}")
    
    if params[:pic1]
      @request.pic1 = "#{@request.date}_1.jpg"
      pic1 = params[:pic1]
      File.binwrite("public/pictures/#{@request.date.year}/#{@request.date.month}/#{@request.date.day}/#{@request.pic1}", pic1.read)
    end
    
    if @request.save
      flash[:notice] = "画像を登録しました"
      redirect_to("/requests/detail/#{@request.id}")
    else
      render("requests/send_picture")
    end  
  end
  
end

routes.rb

Rails.application.routes.draw do
 get 'requests/detail/:id' => 'requests#detail_request'
 get 'requests/upload/:id' => 'requests#send_pictures'
 post 'requests/upload' => 'requests#upload'
end

試したこと

1. 画像投稿画面に@request.dateを表示させてみたが、問題なく表示されていた。
2. 保存先のフォルダ名を@request.idに変えてみたが

"undefined method `id' for nil:NilClass"
とエラーの対象が変わるだけでエラー内容は変わらなかった。

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

Rails 5.1.4

Answer 1

form_tag("/requests/upload", {multipart: true})
の指定パスが違っていて、idが渡せていないんじゃないでしょうか？

正しいパスはroutes.rbの記載内容に合わせてだと思いますが、@requestのidを入れる必要があるかと。

以下でデバッグしてみては？

def upload
    @request = Request.find_by(id: params[:id])

    # 以下を追加
    p @request
    p params[:id]

    FileUtils.mkdir_p("public/pictures/#{@request.date.year}/#{@request.date.month}/#{@request.date.day}") unless FileTest.exist?("public/pictures/#{@request.date.year}/#{@request.date.month}/#{@request.date.day}")

    if params[:pic1]
      @request.pic1 = "#{@request.date}_1.jpg"
      pic1 = params[:pic1]
      File.binwrite("public/pictures/#{@request.date.year}/#{@request.date.month}/#{@request.date.day}/#{@request.pic1}", pic1.read)
    end

    if @request.save
      flash[:notice] = "画像を登録しました"
      redirect_to("/requests/detail/#{@request.id}")
    else
      render("requests/send_picture")
    end  
  end

end

---

追記

Rails.application.routes.draw do
 get 'requests/detail/:id' => 'requests#detail_request'
 get 'requests/upload/:id' => 'requests#send_pictures'
 # 以下に修正
 post 'requests/upload/:id' => 'requests#upload'
end

<!-- form_tagのpath変更 -->
<%= form_tag("/requests/upload/#{@request.id}", {multipart: true}) do %>
 <input type = "file" name = "pic1">
 <input class="button_signup" type="submit" value="送信">
<% end %>

Comments

[whiteN]

ご教示いただきありがとうございます。
パスの確認を行いましたが、間違っていませんでした。
また、追加してデバッグしてみましたが、エラーの場所にもないようにも変わりがありませんでした。

[h_daido]

本当ですか？？？この記述方式だと間違いなくparams[:id]の値は渡せていないはずですが...。ちなみにエラー自体は変わらないはずですし、p xxx の部分は今まで表示されているデバッグログとは違う場所にでますよ。

ターミナルなどで
> rails server
した画面に表示されます。

もしわかりづらければ、
p '================================'
p @request
p params[:id]
とかってやっておくと表示箇所がわかりやすくなると思います。

[whiteN]

本当です。
上記と変わらないエラーが表示されてしまい、画像の投稿ができません。

[h_daido]

いや...そりゃそうですよ。上記はデバッグするためのコードで、修正するためのコードじゃないでしょ？
本当にちゃんとログ見ました？標準出力されているものが、
================================
nil
nil
となっていませんでしたか？

そこまで分かれば修正ポイントもわかりそうなものですが、一応追記していおいたのでよかったら見てみてください。

[whiteN]

ご教示ありがとうございます。
デバッグのためのコードというものを理解できていませんでした。
勉強不足で申し訳ありません。
追記いただいた内容でエラー表示が消え、画像の投稿ができました。ありがとうございました。

[h_daido]

いえ、わかっていただけたならよかったです。
ちなみに、routingの設計がrailsの標準に則していない気がします。別にこのままでも動作はするのですが、あまり見ないroutingなので、一度
https://railsguides.jp/routing.html
とか確認されるとよいかと思います。