NoMethodError in Users#show のエラーから抜けられません

Question

undefined method `name' for nil:NilClass　となりエラーになります
4行目の@user　が定義されてないのでしょうか？

Request

Parameters:

{"id"=>"signup"}　

新規登録のリンクを押すとエラーが出てきます。ユーザー一覧やユーザー詳細を押したら問題なく表示されております。
この場合　show 周辺より　new　が怪しいでしょうか？

<div class="main user-show">
  <div class="container">
    <div class="user">
        <h2><%= @user.name  %></h2>
        <p> <%= @user.email %></p>
        <%= link_to("編集", "/users/#{@user.id}/edit") %>
    </div>
  </div>
</div>

class UsersController < ApplicationController
  def index
    @users=User.all
  end

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

  def new
    @user=User.new
  end

  def create
    @user=User.new(name: params[:name], email: params[:email])

    if  @user.save
        redirect_to("/users/#{@user.id}")
        flash[:notice]="ユーザー登録が完了しました"
    else
      render("users/new")
    end
  end
  
  def edit
    @user=User.find_by(id: params[:id])
  end

  def update
    @user=User.find_by(id: params[:id])
    @user.name = params[:name ]
    @user.email= params[:email]
    
    if  @user.save
        redirect_to("/users/#{@user.id}")
        flash[:notice]="ユーザー情報を編集しました"
    else
        render("users/edit")
    end
  end

end

post "users/:id/update" => "users#update"
  get  "users/:id/edit" => "users#edit"
  post "users/create" => "users#create"
  get  "signup" => "users#new"
  get  "users/index" => "users#index"
  get  "users/:id" => "users#show"

Answer 1

User.find_by(id: params[:id])が見つからない場合にnilを返すのが直接の問題です。

nilを返された時にどうするか決めていないのならば
@user = User.find(params[:id])で404エラーにしてしまった方がよいように思います。

---

params[:id]が見つからないのがそもそもの原因です。
なぜそのようなパラメータが渡されているのか、リンクを生成している側のviewを探る必要があります。

---

{"id"=>"signup"}　

なので、ルーティングがまずいですね。

routes.rbの上から設定されるので

get "/users/:id"   => "users#show"
get "/users/signup" => "users#new"

と設定すると/users/signupはusers#showに引っかかりusers#newには飛びません。
先にnewにするか、そもそもresources :users, path_names: {new: :signup}使った方が楽です。

Comments

[ke-si]

ルーティングは上から実行されるということでしょうか？一応newは上にしております、resources...は
signup newのルーティングを消して書き換えるということでしょうか？

[ke-si]

書き換えると新規登録は復活しました！が今度はユーザー一覧に同じエラーが出て　今度は{"id"=>"index"}となっております。

[ke-si]

get "signup" => "users#new"　→　get "users/signup" => "users#new"　に変えると復活できました。これはどういうことだったのでしょうか？

[asm]

index/updateも標準とは違うので

resources :users, path_names: {new: :signup, index: :index}, except: [:update, :destroy] do
post "update" => "users#update", on: :member
end

ですね

[asm]

> "signup"→"users/signup"

routes.rbでは/signupを利用しているのに
「新規登録のリンク」を生成しているViewファイルでは/users/signupをリンク先に指定していました。

[ke-si]

そういうことでした、リンクの指定先を/users/signupにしてました。ＵＲＬを確認すればすぐに分かることでした。適切な回答いただきありがとうございました。

Answer 2

4行目の@user　が定義されてないのでしょうか？

はい、@userがnilとなっているようです。

showアクション内の@user=User.find_by(id: params[:id])は、見つからない場合にnilを返します。

Comments

[ke-si]

回答ありがとうございます、idが入っていないということでしょうか？ユーザー詳細を見るとURLにid
は含まれております。

[maisumakun]

> idが入っていないということでしょうか？ユーザー詳細を見るとURLにid
は含まれております。

当該IDが見つからない状態です。間違えて別なモデルのIDを供給していたりしませんか？

[ke-si]

ありがとうございます、そういうケースもあるんですね、確認してみます。