Rails、updateのエラーの解決。

Question

投稿機能のサイト作成しています。
Edit→Updateに実行した際に添付した画像のエラーが出てきます。解決方法を教えていただきたく、お願い申し上げます。

def edit
    @book = Book.find(params[:id])
  end

  def update
    if @book.update(book_params[:id])
       redirect_to public_books_path(@book), notice: "You have updated book successfully."
    else
       render "edit"
    end
  end

<div class='container'>
  <div class='row'>
    <div class="col-sm-12 col-md-8 col-lg-5 px-5 px-sm-0 mx-auto">
        
  		<h1>Editing Book</h1>
  		  <%= form_with model:[:public, @book], local:true do |f| %>
          <div class="form-group">
            <%= f.label :title %>
            <%= f.text_field :title, class: 'form-control', size: "20x10" %>
          </div>
          <div class="form-group">
            <%= f.label :image %>
            <%= f.attachment_field :image, class: 'form-control', size: "20x10" %>
          </div>
          <div class="form-group">
            <%= f.label :place %>
            <%= f.text_area :place, class: 'form-control', size: "20x10" %>
          </div>
          <div class="form-group">
            <%= f.label :explanation %>
            <%= f.text_area :explanation, class: 'form-control', size: "20x10" %>
          </div>
          <div class="form-group">
            <%= f.submit "編集内容を保存する", class: "btn btn-dark save col-xs-6" %>
            <%= link_to 'Show', public_book_path(@book), class: "book_#{@book.id} btn btn-dark back col-xs-6" %> |
  	        <%= link_to 'Back', public_books_path, class: "back" %>
          </div>
        <% end %>
        
  </div>
</div>

namespace :public do
    root to: 'homes#top'
    
    resources :books, only: [:new, :index, :show, :edit, :create, :destroy, :update]
   
   
    resources :customers, only:[:show, :edit, :index, :update]
    get 'customers/unsubscribe'
    patch 'customers/withdraw'

  end

Answer 1

エラーメッセージの通り、@bookがnilです。@bookを取得するコードを追加しましょう。

Comments

[sattaku1538]

申し訳ないのですが、もう少し、具体的な回答をいただけないでしょうか。
@bookを取得するコード(@book = Book.find(params[:id]))を記述しましたが、エラーが起きています。その件も含めて質問をさせていただいています。お手数ですが、ご回答よろしくお願いします。

[maisumakun]

> @bookを取得するコード(@book = Book.find(params[:id]))を記述しましたが

updateにはありません。

[sattaku1538]

失礼しました。質問内容には載せておりませんが、小職の方で(@book = Book.find(params[:id]))を記述してRails sを実行しています。その点も含めて、回答をいただけたらと思います。

[maisumakun]

どこに書いたのですか？

（冒頭のエラーメッセージ部分にもその行は出ていないようです）

[sattaku1538]

Controllerに記述しました。

def update
@book = Book.find(params[:id])
if @book.update(book_params[:id])
redirect_to public_books_path(@book), notice: "You have updated book successfully."
else
render "edit"
end
end

このように記述しました。
エラー分は
ArgumentError in Public::BooksController#update
When assigning attributes, you must pass a hash as an argument.
上記のようになりました。

お手数ですが、ご教示お願いします。

[maisumakun]

book_paramsはどのように定義していますか？

[sattaku1538]

プロのプログラマーに質問したところ、以下の回答が返ってきました。

修正前
def update
@book = Book.find(params[:id])
if @book.update(book_params[:id])
redirect_to public_books_path(@book), notice: "You have updated book successfully."
else
render "edit"
end
end

修正後
def update
@book = Book.find(params[:id])
if @book.update(book_params)
redirect_to public_books_path(@book), notice: "You have updated book successfully."
else
render "edit"
end
end

book_paramsの定義方法が間違えているとご教示いただきました。
Controller内のparams[:id]は投稿した内容の記号A,B,C,et.....
一方でbook_paramsは投稿内容→タイトル、写真などの内容についてを定義しているとのこと。

よって、
@book = Book.find(params[:id])で投稿の大まかな分類を定義する。
if @book.update(book_params)で@book内の詳細な記事の内容を分類をする。

→maisumakun　ご回答ありがとうございました。
小職も勉強になりました。

Answer 2

def update
@book = Book.find(params[:id])
if @book.update(book_params)
redirect_to public_books_path(@book), notice: "You have updated book successfully."
else
render "edit"
end
end

book_paramsの定義方法が間違えていました。