undefined method `name=' for nil:NilClass というエラーが出てしまいます。

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
プロフィールを編集して、名前、所属、画像をアップデートできるようにしたいのですが、エラーが出てしまい困っています。

イメージ
名前所属画像を変更したら、アップデート後に/users/indexに飛ぶ

拙い質問で申し訳ございません。何卒よろしくお願いします。

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

undefined method `name=' for nil:NilClass

該当のソースコード

<div class="main users-edit">
  <div class="container">
    <div class="form-heading">
      プロフィール編集
    </div>
    <div class="form-body">
      <%= form_tag("/users/:id/update", {multipart: true}) do %>
      <p>名前</p>
      <input name="name" value="<%= @user.name %>">
      <p>所属パート</p>
      <input name="part" value="<%= @user.part %>">
      <p>画像</p>
      <input name="image" type="file">
      <input type="submit" value="保存">
      </div>
      <% end %>
  </div>
</div>

def update
    @user = User.find_by(id: params[:id])
    @user.name = params[:name]
    @user.part = params[:part]
    
    if params[:image]
      @user.image_name = "#{@user.id}.jpg"
      image = params[:image]
      File.binwrite("/public/user_images/#{@user.image_name}", image.read)
    end
    if @user.save
      flash[:notice]="プロフィールを変更しました"
      redirect_to("/users/#{@user.id}")
    else
      render("users/edit")
    end
  end

試したこと

エラーの出る、@user.name = params[:name]の部分を消したら一個下の@user.part = params[:part]でエラーが出ます。またこれも消したら@user.image_name = "#{@user.id}.jpg"で同じエラーが出ます。

https://teratail.com/questions/61224などを参考にしたところ、name,part,idを見つけれていないのかと思います。

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

Progateを参考に進めています

Answer 1

フォームの飛び先に"/users/:id/update"と書いていますが、この表記では書いた通りに解釈されますので、updateでのparams[:id]には、「:id」が代入されてしまいます。当然ながら、こんなidのUserは見つかりませんので、@userがnilになります。

nilに対してプロパティの代入をしようとしているので、エラーになっています。

Comments

[sn.jr]

ご回答ありがとうございます！なるほど、どのように書きかえればよろしいのでしょうか？もしありましたら教えていただけると幸いです！

[siruku6]

せっかくベストアンサーにしていただいたので横やり失礼します...

どうしても手っ取り早くやるなら、
<%= form_tag("/users/:id/update", {multipart: true}) do %>
の部分を
<%= form_tag("/users/" + @user.id.to_s + "/update", {multipart: true}) do %>
のように修正する感じになります。

可能なら、コンソールで
　bundle exec rake routes か、または bundle exec rails routes
　（どちらもだめなら rake routes か rails routes）
を実行して、updateアクションのpathを呼び出せるメソッド名を確かめるとよいと思います。

rake routesの結果をどう使ったらいいかわからない場合は、この質問かコメント欄に追記してみてください。

[sn.jr]

ご親切にありがとうございます！！ 少しやってみてわからなかったらまた質問させていただきます！

Answer 2

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

が
@user = nil
となっているので、nameやpartがないというのがエラーの原因かと。
params[:id]が取得できているか確認してみてください。

Comments

[sn.jr]

ご回答ありがとうございます！確認いたします！

Answer 3

ここでuserを見つけられず、@userの中身がnilになっていることがエラーの原因です。

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

表示されているエラーは、nil.name= を呼ぼうとしたけど呼べなかったよ、というメッセージになります。

なので、

1. params[:id] に、期待した値が入っているかどうか確認
2. @user が正しく取得できているか確認

をしたらよいと思います

方法としては、以下のような記述をして、

logger.debug(params)
logger.debug(params[:id])
@user = User.find_by(id: params[:id])

実際に動作させて、log/development.log ファイルなどを確認するとよさそうです。

---

個人的には、 pry-rails と pry-byebug や better-errors という gem を使用して変数の中身を確かめるのがおすすめです。
余裕があれば、これらの導入方法も調べるとよいと思います

Comments

[sn.jr]

ご回答ありがとうございます！導入方法調べてみます！