定義されていないメソッドのエラー

Question

前提・実現したいこと

railsでwebアプリケーションを作成しています。
point_listの編集機能を実装中です。

ご教授のほどよろしくお願いいたします。

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

undefined method `update' for nil:NilClass

該当のソースコード

point_lists_controller.rb

class PointListsController < ApplicationController
  def new
    @point_list = PointList.new
  end

  def create    
    @point_list = PointList.new(point_list_params)
    
    if @point_list.save
      redirect_to card_list_url(@point_list.card_list_id), notice: "特典を追加しました"
    else
      render "new"
    end
  end

  def edit
    @point_list = PointList.find(params[:id])
  end

  def update
    if @point_list.update(point_list_params)
      redirect_to card_list_url(@point_list.card_list_id), notice: "特典内容を変更しました"
    else
      render "edit"
    end
  end


  private
    def point_list_params
      params.require(:point_list).permit(:content).merge(card_list_id: params[:card_list_id])
    end
end

試したこと

最初によく見るエラーなのでスペルミスを探しました。
しかし、自分では見つけられていません。

2つ目にpoint_listの親要素であるcard_list_idが空になっていると思いました。
なのでviews/point_lists/edit.html.hamlのformの部分に下記の記述を加えました。
しかし、うまくいきません。

=f.hidden_field :card_list_id

3つ目に@point_listが空になっていると考えました。
binding.pryを使って中身を見ると下記の通り、取得したい値が入っています。

[1] pry(#<PointListsController>)> point_list_params
Unpermitted parameter: :card_list_id
=> <ActionController::Parameters {"content"=>"いか二貫", "card_list_id"=>"1"} permitted: true>

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

rails routesの結果です
                    Prefix Verb   URI Pattern                                                                              Controller#Action
                      root GET    /                                                                                        introduction#home
                     users GET    /users(.:format)                                                                         users#index
                           POST   /users(.:format)                                                                         users#create
                  new_user GET    /users/new(.:format)                                                                     users#new
                 edit_user GET    /users/:id/edit(.:format)                                                                users#edit
                      user GET    /users/:id(.:format)                                                                     users#show
                           PATCH  /users/:id(.:format)                                                                     users#update
                           PUT    /users/:id(.:format)                                                                     users#update
                           DELETE /users/:id(.:format)                                                                     users#destroy
               users_login GET    /users/login(.:format)                                                                   users/sessions#new
                           POST   /users/login(.:format)                                                                   users/sessions#create
              users_logout DELETE /users/logout(.:format)                                                                  users/sessions#destroy
                     cards GET    /cards(.:format)                                                                         cards#index
                           POST   /cards(.:format)                                                                         cards#create
                  new_card GET    /cards/new(.:format)                                                                     cards#new
              admins_login GET    /admins/login(.:format)                                                                  admins/sessions#new
                           POST   /admins/login(.:format)                                                                  admins/sessions#create
             admins_logout DELETE /admins/logout(.:format)                                                                 admins/sessions#destroy
                    admins POST   /admins(.:format)                                                                        admins#create
                 new_admin GET    /admins/new(.:format)                                                                    admins#new
     card_list_point_lists POST   /card_lists/:card_list_id/point_lists(.:format)                                          point_lists#create
  new_card_list_point_list GET    /card_lists/:card_list_id/point_lists/new(.:format)                                      point_lists#new
 edit_card_list_point_list GET    /card_lists/:card_list_id/point_lists/:id/edit(.:format)                                 point_lists#edit
      card_list_point_list PATCH  /card_lists/:card_list_id/point_lists/:id(.:format)                                      point_lists#update
                           PUT    /card_lists/:card_list_id/point_lists/:id(.:format)                                      point_lists#update
                           DELETE /card_lists/:card_list_id/point_lists/:id(.:format)                                      point_lists#destroy
    card_list_coupon_lists POST   /card_lists/:card_list_id/coupon_lists(.:format)                                         coupon_lists#create
 new_card_list_coupon_list GET    /card_lists/:card_list_id/coupon_lists/new(.:format)                                     coupon_lists#new
edit_card_list_coupon_list GET    /card_lists/:card_list_id/coupon_lists/:id/edit(.:format)                                coupon_lists#edit
     card_list_coupon_list PATCH  /card_lists/:card_list_id/coupon_lists/:id(.:format)                                     coupon_lists#update
                           PUT    /card_lists/:card_list_id/coupon_lists/:id(.:format)                                     coupon_lists#update
                           DELETE /card_lists/:card_list_id/coupon_lists/:id(.:format)                                     coupon_lists#destroy
                card_lists POST   /card_lists(.:format)                                                                    card_lists#create
             new_card_list GET    /card_lists/new(.:format)                                                                card_lists#new
                 card_list GET    /card_lists/:id(.:format)                                                                card_lists#show
        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

views/point_lists/edit.html.haml

=render "layouts/header"
.point-list-content
  %h2 
    追加する特典情報を入力してください
  =form_with model: @point_list, url: card_list_point_list_url, method: :patch, class: "admin-point-form", local: true do |f|
    .admin-point-form__point
      = f.text_field :content, class: "admin-point-form__point-input" 
    .admin-point-form__submit
      = f.submit "特典内容を変更する", class: 'btn btn-primary'

Comments

[otn]

> 3つ目に@point_listが空になっていると考えました。
> binding.pryを使って中身を見ると下記の通り、取得したい値が入っています。

と書きながら、@point_list じゃなく point_list_params を見ている結果が載ってます。
正しい@point_listを見た結果を載せましょう。

[anguraaaa]

ご指摘、ありがとうございます！
以後、気をつけます。

Answer 1

updateアクション(def update)中において、@point_listの中に何も入っていない(nil)ようです。

原因としては、update アクションの基本的な記載方法が間違っています。
@point_list.update(point_list_params)する前に@point_list = PointList.find(params[:id])してあげないといけません。

こんな感じで。

@point_list = PointList.find(params[:id])
@point_list.update(point_list_params)

ただ、そのためには、paramsにidを含める必要があるので、point_list_params メソッドやviewファイルに変更も必要になります。

• 参考記事

【Rails】updateメソッドの使い方を徹底解説！

Comments

[anguraaaa]

ご回答いただきありがとうございます！
無事に修正できました！
参考記事は大変、勉強になりました。
editに@point_list = PointList.find(params[:id])を記述すればいいと勘違いしていました。

[siruku6]

うまくいってよかったです！