質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

1回答

1074閲覧

インスタンス変数を定義しているのにNoMethodErrorが表示されてしまいます

shuta-a

総合スコア5

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2021/12/14 23:26

編集2021/12/15 01:03

shutaコントローラのcreateアクション、eachメソッドでエラーが起きました。
createアクション内で、インスタンス変数@booksは定義しています。
何が原因なのでしょうか。

エラー文↓
イメージ説明

shutaコントローラ↓

rails

1class ShutaController < ApplicationController 2 def new 3 @books = Book.new 4 5 end 6 7 8 def index 9 @books = Book.all 10 @book = Book.new 11 12 end 13 14 def show 15 @book = Book.find(params[:id]) 16 end 17 18 def create 19 @books = Book.new(book_params) 20 if @books.save 21 redirect_to shutax_path(book.id) 22 flash[:notice] = "Book was successfully created" 23 else 24 render :index 25 end 26 end 27 28 def edit 29 @book = Book.find(params[:id]) 30 end 31 32 def update 33 book = Book.find(params[:id]) 34 book.update(book_params) 35 redirect_to shutax_path(book.id) 36 flash[:notice] = "Book was successfully updated" 37 end 38 39 def destroy 40 book = Book.find(params[:id]) 41 book.destroy 42 redirect_to index_path 43 end 44 45 46 47 private 48 49 def book_params 50 params.permit(:title, :body) 51 end 52 53end 54

index.html.erb↓

rails

1<h1>Books</h1> 2 3<table class="table"> 4 <thead> 5 <tr> 6 <th>Title</th> 7 <th>Body</th> 8 <th colspan="3"></th> 9 </tr> 10 </thead> 11 <tbody> 12 <% @books.each do |book| %> 13 <tr> 14 <td> 15 <%= book.title %> 16 </td> 17 <td> 18 <%= book.body %> 19 </td> 20 <td> 21 <%= link_to "Show", shutax_path(book.id) %> 22 <%= link_to "Edit", edit_shutax_path(book.id) %> 23 <%= link_to "Destroy", destroy_shutax_path(book.id), method: :delete %> 24 </td> 25 </tr> 26 <% end %> 27 </tbody> 28</table> 29 30<h1>New book</h1> 31 32<% if @book.errors.any? %> 33 <%= @book.errors.count %> 34 <% @book.errors.full_messages.each do |message| %> 35 <%= message %> 36 <% end %> 37<% end %> 38 39 40<%= form_with model: @book, url:'/books', local:true do |f| %> 41 <h4>Title</h4> 42 <%= f.text_field :title %> 43 <h4>Body</h4> 44 <%= f.text_area :body %> 45 <%= f.submit 'Create Book' %> 46<% end %>

ご回答よろしくお願いいたします。

見本の投稿・一覧画面↓
イメージ説明

空白でCreate Bookを押した際の、見本のエラー表示↓
イメージ説明

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

エラーメッセージのとおり、@booksnilです。

cr4eateでエラーになった際に、render :indexとしていますが、この場合に@booksはセットされていません。

投稿2021/12/14 23:44

maisumakun

総合スコア146018

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

maisumakun

2021/12/14 23:45

エラーになった際の再入力に、他の項目の一覧は不要かと思うので、専用のビューを用意する(あるいはnew用のビューを使う)ことをおすすめします。
maisumakun

2021/12/14 23:49

なぜ@booksにBook.newの結果を入れようとしているのでしょうか?
maisumakun

2021/12/14 23:52 編集

コントローラーのコードは、本当にご提示のコードで正しいですか? (書き換えたけれど反映されていない、なんてことはありませんか?保存、再起動などして確認してみましょう)
shuta-a

2021/12/15 00:24

現在とあるアプリケーションを見本に作成しているのですが、見本では一覧画面(index)と投稿画面(new)が同じViewにあるのです(index.html.erb参照)。また投稿フォームでの空欄の際のエラーメッセージもindex.html.erbで表示されるようにしたいのです。 indexアクションでの@book = Book.newのことでしょうか。とりあえず@bookを定義するために書いたので詳しい根拠もないです。すみません。 コントローラのコードは提示したコードそのままです。サーバーも逐一再起動して確認しております。
maisumakun

2021/12/15 00:36

提示のコードで提示のエラーは出るはずがないのですが、何か間違っていませんか? (本当に@booksがnilなら、ビューに辿り着く前に、@books.saveでエラーとなります)
maisumakun

2021/12/15 00:37

> また投稿フォームでの空欄の際のエラーメッセージもindex.html.erbで表示されるようにしたいのです。 本当に「入力している項目と無関係な案件の一覧」も必要なのですか?
shuta-a

2021/12/15 00:58

わかりました。詳しく調べてみます。 上手く言語化ができないので質問の追記欄に見本の画像を貼ります。 すみません。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問