本環境でおきたエラーを直したい

Question

herokuにアプリを更新したあと
We're sorry, but something went wrong.
If you are the application owner check the logs for more information
というエラーが出たのでターミナルheroku logs --tailをしたところ
以下のエラー文が出ました（した１０行だけ書きます）

2020-10-26T09:36:36.741765+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659]
2020-10-26T09:36:36.741766+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659] ActionView::Template::Error (No route matches {:action=>"show", :controller=>"users", :id=>nil}, missing required keys: [:id]):
2020-10-26T09:36:36.741767+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659]      5:     <p class="card__summary">
2020-10-26T09:36:36.741767+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659]      6:       <%= "プロトタイプのキャッチコピー" %>
2020-10-26T09:36:36.741768+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659]      7:     </p>
2020-10-26T09:36:36.741768+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659]      8:     <%= link_to "by#{prototype.user.name}", user_path(current_user), method: :get, class: :card__user  %>
2020-10-26T09:36:36.741769+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659]      9:   </div>
2020-10-26T09:36:36.741769+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659]     10: </div>
2020-10-26T09:36:36.741769+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659]
2020-10-26T09:36:36.741770+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659] app/views/prototypes/_prototype.html.erb:8
2020-10-26T09:36:36.741770+00:00 app[web.1]: [faaca404-a2ec-4d82-a673-3f1e008ef659] app/views/prototypes/index.html.erb:20
2020-10-26T10:11:20.851973+00:00 heroku[web.1]: Idling
2020-10-26T10:11:20.854855+00:00 heroku[web.1]: State changed from up to down
2020-10-26T10:11:21.811353+00:00 heroku[web.1]: Stopping all processes with SIGTERM
2020-10-26T10:11:21.841059+00:00 app[web.1]: - Gracefully stopping, waiting for requests to finish
2020-10-26T10:11:21.843060+00:00 app[web.1]: === puma shutdown: 2020-10-26 10:11:21 +0000 ===
2020-10-26T10:11:21.843062+00:00 app[web.1]: - Goodbye!
2020-10-26T10:11:21.843063+00:00 app[web.1]: Exiting
2020-10-26T10:11:21.962814+00:00 heroku[web.1]: Process exited with status 143

気になったのは
ActionView::Template::Error (No route matches {:action=>"show", :controller=>"users", :id=>nil}, missing required keys: [:id]):
というところですがどのファイルをどうやって直せばいいかわかりません

関係ありそうなコードを載せます
config/routes.rb

Rails.application.routes.draw do
  devise_for :users
  root to: "prototypes#index"
  resources :users, only: [:new, :edit, :update, :show]
  resources :prototypes, only: [:index, :new, :create, :show, :edit, :update, :destroy,] do
    resources :comments, only: [:create]
  end

  devise_scope :user do
    get '/users/sign_out' => 'devise/sessions#destroy'
  end

end

app/controllers/prototypes_controller.rb

class PrototypesController < ApplicationController
  before_action :authenticate_user!, only: [:index, :show, :new]
  before_action :move_to_index, expect: [:index]

  def index
     @prototypes = Prototype.all
  end
    
  def new
    @prototype = Prototype.new
  end

  def create
    @prototype = Prototype.new(prototype_params)
    if @prototype.save
       redirect_to root_path
    else
      render :new
    end
  end

  def show
    @prototype = Prototype.find(params[:id])
    @comment = Comment.new
    @comments = @prototype.comments.includes(:user)

  end

  def edit
    @prototype = Prototype.find(params[:id])
    if @prototype.user_id == current_user.id
      render :edit
    else
      redirect_to root_path 
  end
  end 

  def update
    prototype = Prototype.find(params[:id])
    if prototype.update(prototype_params)
       redirect_to prototype_path
   else
      render :edit
   end
  end

  def destroy
    @prototype = Prototype.find(params[:id])
    if @prototype.destroy
       redirect_to root_path
    end
  end

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

  private
  def prototype_params
    params.require(:prototype).permit(:title, :catct_copy, :concept, :image).merge(user_id: current_user.id)
  end  

  
end

app/controllers/users_controller.rb

class UsersController < ApplicationController
  before_action :move_to_index, except: [:index, :show]
  
  
  def index
  
  end
  
  def new
    @user = User.new 
  end

  def edit
  end

  def show
    @user = User.find(params[:id])
    @prototypes =  @user.prototypes
  end


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


  # private
  # def user_params
  #     params.require(:user).permit(:name, :profile, :occupation, :position).merge(user_id: current_user.id)
  # end  

end

app/views/prototypes/show.html.erb

<main class="main">
  <div class="inner">
    <div class="prototype__wrapper">
      <p class="prototype__hedding">
        <%= "プロトタイプのタイトル"%>
      </p>
      
      <%= link_to "by #{@prototype.user.name}さん" ,  user_path(@prototype.user), class: :prototype__user %>
      <%# 上の記述で＠prototypeにuserモデルからnameカラムを持ってくるコードができた %>
      <%# プロトタイプの投稿者とログインしているユーザーが同じであれば以下を表示する %>
      <% if current_user == @prototype.user%>
        <div class="prototype__manage">
          <%= link_to "編集する", edit_prototype_path(@prototype), class: :prototype__btn %>
          <%= link_to "削除する", prototype_path(@prototype), method: :delete, class: :prototype__btn %>
        </div>
      <% end %>
      <%# // プロトタイプの投稿者とログインしているユーザーが同じであれば上記を表示する %>
      <div class="prototype__image">
        <%= image_tag @prototype.image %>
      </div>
      <div class="prototype__body">
        <div class="prototype__detail">
          <p class="detail__title">キャッチコピー</p>
          <p class="detail__message">
            <%= @prototype.catct_copy%>
          </p>
        </div>
        <div class="prototype__detail">
          <p class="detail__title">コンセプト</p>
          <p class="detail__message">
            <%= @prototype.concept%>
          </p>
        </div>
      </div>
      <div class="prototype__comments">
        <%# ログインしているユーザーには以下のコメント投稿フォームを表示する %>
          <% if current_user %>
            <%= form_with model: [@prototype, @comment], local: true do |f|%> 
            <%# (model: [@prototype, @comment], local: true)にするといいかも %>
            <div class="field">
              <%=  f.label :text, "コメント" %><br />
              <%=  f.text_field :text %>
            </div>
            <div class="actions">
              <%= f.submit "送信する", class: :form__btn  %>
            </div>
            <% end %> 　
           <% end %> 
        <%# // ログインしているユーザーには上記を表示する %>
        <ul class="comments_lists">
          <%# 投稿に紐づくコメントを一覧する処理を記述する %>
            <% if @comments %>
              <% @comments.each do |comment| %>
            <li class="comments_list">
              <%= link_to comment.text + "  (#{@prototype.user.name})", user_path(current_user), class: :comment_user %>
             　<% end %>
             <% end %>
            </li>
          <%# // 投稿に紐づくコメントを一覧する処理を記述する %>
        </ul>
      </div>
    </div>
  </div>
</main>

app/views/users/show.html.erb

<div class="main">
  <div class="inner">
    <div class="user__wrapper">
      <h2 class="page-heading">
        <%= @user.name + "さんの情報" %>
      </h2>
      <table class="table">
        <tbody>
          <tr>
            <th class="table__col1">名前</th>
            <td class="table__col2"><%= @user.name %></td>
          </tr>
          <tr>
            <th class="table__col1">プロフィール</th>
            <td class="table__col2"><%= @user.profile %></td>
          </tr>
          <tr>
            <th class="table__col1">所属</th>
            <td class="table__col2"><%= @user.occupation %></td>
          </tr>
          <tr>
            <th class="table__col1">役職</th>
            <td class="table__col2"><%= @user.position %></td>
          </tr>
        </tbody>
      </table>
      <h2 class="page-heading">
        <%= @user.name + "さんのプロトタイプ" %>
      </h2>
      <div class="user__card">
        <%# 部分テンプレートでそのユーザーが投稿したプロトタイプ投稿一覧を表示する %>
       <%= render partial: "prototypes/prototype", collection: @prototypes %>
      </div>
    </div>
  </div>
</div>

app/views/prototypes/_prototype.html.erb

<div class="card">
  <%= image_tag prototype.image, class: 'card__img' if prototype.image.attached? %>
  <div class="card__body">
    <%= link_to 'プロトタイプのタイトル', prototype_path(prototype.id), method: :get, class: :card__title%>
    <p class="card__summary">
      <%= "プロトタイプのキャッチコピー" %>
    </p>
    <%= link_to "by#{prototype.user.name}", user_path(current_user), method: :get, class: :card__user  %>
  </div>
</div>

Answer 1

<%= link_to "by#{prototype.user.name}",
ということは、prototype　に関連するuserの詳細を表示させたいのでしょうから
user_path(current_user) は？？？
user_path(prototype.user) では？

Answer 2

ActionView::Template::Error (No route matches {:action=>"show", :controller=>"users", :id=>nil}, missing required keys: [:id]):

とのことで、引数にIDが必須だけどパス(=user_path(current_user))のcurrent_userがnilであり、IDが渡っていないために起きているエラーかと思われます。

ログインをして、current_userがnilにならないようにする
またはcurrent_userがあるときのみ表示するようにするなどで解決できるかと思いました。

<% if current_user %>
  <%= link_to "by#{prototype.user.name}", user_path(current_user), method: :get, class: :card__user  %>
<% end %>

Comments

[yozakura10]

<% if current_user %>
<%= link_to "by#{prototype.user.name}", user_path(current_user), method: :get, class: :card__user %>
<% end %>
にしたのですがエラー内容が変わりませんでしたログインをして、current_userがnilにならないようにする
またはcurrent_userがあるときのみ表示するようにするなどで解決するには
どうしたら良いでしょうか

[hatsu]

ちなみにローカル環境の非ログイン状態でも再現しますでしょうか？
あとroutes.rbファイルの中身も質問欄に追記いただけると参考になるかもです。

[yozakura10]

非ログイン時での各プロトタイプタイトルを押したらActionController::UrlGenerationError in Prototypes#show
Showing /Users/kamiyaryota/Desktop/projects/protospace-31035/app/views/prototypes/show.html.erb where line #55 raised:

No route matches {:action=>"show", :controller=>"users", :id=>nil}, missing required keys: [:id]
Extracted source (around line #55):
53
54
55
56
57
58

<% @comments.each do |comment| %>
<li class="comments_list">
<%= link_to comment.text + " (#{@prototype.user.name})", user_path(current_user), class: :comment_user %>
　<% end %>
<% end %>
</li>
というエラーが出たのですがこれのことでしょうか、これのことでしたら本文に記載いたします
ローカル環境の非ログイン時に出ました
routuesは追加いたしました