NoMethodError in ContentsController#createが解決できない

Question

前提・実現したいこと

rails で情報共有掲示板を作成中です。
ある金融機関のあるカテゴリーに対して、その内容を登録したいのですが、
エラーが発生し、解決できていない状況です。
※テーブルはユーザー・金融機関・カテゴリー・コンテンツの4つです。
イメージとしては、「A銀行」の「残高」は「5000円」と登録したいのですが、できていません。

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

該当のソースコード

class ContentsController < ApplicationController

  def index
    @banks = Bank.all #サイドバーに必要（共通）
    @content =Content.new
    @bank = Bank.find(params[:bank_id])
    @contents = @bank.contents
    @categories = @bank.categories.select(:subject)
  end

  def new
    @content = Content.new
    @banks = Bank.all #サイドバーに必要（共通）
    @bank = Bank.find(params[:bank_id])
    @categories = @bank.categories.select(:subject)
    @category = Category.find(params[:category_id]) 
  end

  def create
    @bank = Bank.find(params[:bank_id]),Category.find(params[:category_id])
    @test = Bank.find(params[:bank_id])
    @categories = @test.categories.select(:subject)
    @content = @bank.contents.new(content_params)
    if @content.save
      redirect_to bank_contents_path(@bank),notice: "登録しました。"
    else
      @contents = @bank.contents.includes(:category)
      redirect_to bank_contents_path(@bank),notice: "登録に失敗しました。"
    end
  end

  private
  def content_params
    params.require(:content).permit(:detail,:bank_id,:category_id,:user_id).merge(user_id: current_user.id)
  end
end

登録フォームのあるHTML

<div class="wrapper">

  <div class="side-bar">
    <%= render "categories/side_bar" %>
  </div>

  <div class="page-header">
  <div class="header-title">
      <%= @bank.name %>
  </div>
    <% if flash[:notice] %>
      <p><%= flash[:notice] %></p>
    <% end %>
  </div>
</div>

  <div class="content">
    <div class="category_detail">
          <div class="group">
            <%= @category.subject %>
            <%= form_with model: [@bank,@category,@content], class: 'form', local: true do |f| %>
              <div class="form-input"> 
                <%= f.text_field :detail, class: 'form-message', placeholder: '最新情報を登録' %>
                <%= f.submit '登録', class: 'form-submit' %>
              </div> 
            <% end %>
          </div>
    </div>
  </div>

登録フォームへ遷移するHTML

<div class="wrapper">

  <div class="side-bar">
    <%= render "categories/side_bar" %>
  </div>

  <div class="page-header">
  <div class="header-title">
      <%= @bank.name %>
  </div>
    <% if flash[:notice] %>
      <p><%= flash[:notice] %></p>
    <% end %>
  </div>
</div>

  <div class="content">
    <div class="category_detail">
      <% @bank.categories.each do |category| %>
          <div class="group">
            <%=link_to category.subject,new_bank_category_content_path(@bank.id ,category.id) %>
            <%= @content.detail %>
          </div>
      <% end %>
    </div>
  </div>

ルーティング

Rails.application.routes.draw do
  devise_for :users
  root to: "categories#index"
  resources :users, only: [:edit, :update]
  resources :categories, only: [:new, :create]
  resources :banks, only: [:new, :create, :destroy, :edit, :update] do
    resources :contents, only: [:index, :create, :new]
  end

  resources :banks, only: [:new, :create, :destroy, :edit, :update] do
    resources :categories, only: [:new, :create] do
      resources :contents, only: [:index, :create, :new]
    end
  end
end


```テーブル設計
banks（金融機関）テーブル
|id|name|
|:--:|:--:|
|1|A銀行|

categories（カテゴリー）テーブル
|id|subject|
|:--:|:--:|
|1|残高|
|2|商品名|

bank_categories（中間）テーブル
|id|bank_id|category_id|
|:--:|:--:|:--:|
|1|1|1|
|2|1|2|

contents（内容）テーブル
|id|detail|bank_id|category_id|user_id|
|:--:|:--:|:--:|:--:|:--:|
|1|5000円|1|1|1|
|2|Aローン|1|2|1|

モデル
```bank
class Bank < ApplicationRecord
  has_many :bank_categories
  has_many :categories, through: :bank_categories
  has_many :contents

  validates :name, presence: true
  validates :name, uniqueness: true
end

class Category < ApplicationRecord
  has_many :bank_categories
  has_many :banks, through: :bank_categories
  has_many :contents
  
  validates :subject, uniqueness: true
  validates :number, uniqueness: true
end

class BankCategory < ApplicationRecord
  belongs_to :bank
  belongs_to :category
end

class Content < ApplicationRecord
  belongs_to :bank
  belongs_to :category
  belongs_to :user

  validates :detail, presence: true
  validates :bank_id, presence: true
  validates :category_id, presence: true
  validates :user_id, presence: true
end

※上記テーブルのように情報が登録できればベストだと考えています。

試したこと

＠contentが正しく設定できていないのではないかと思いましたがうまくいきませんでした。
※IDをとる必要があると考え、ルーティングを今回のように設定しております。

Answer 1

Bankモデルに

has_many :contents

で関連付けはできていますでしょうか？この関連付けがないと@bank.contentsは呼び出せないですね

あと

@bank = Bank.find(params[:bank_id]),Category.find(params[:category_id])

の

,Category.find(params[:category_id])

の部分は誤字ですかね？もし意図的に加えたものであるならば、外しておきましょう

>>>>>>>>>追記

has_many :contentsはあるのですね

new.html.erb
<%= form_with model: [@bank,@category,@content], class: 'form', local: true do |f| %>
↓
<%= form_with model: @content, class: 'form', local: true do |f| %>

contents_controller.rb
※testは不要なコードだとお見受けしたので省略
def create
@bank = Bank.find(params[:bank_id])
@content = @bank.contents.new(content_params)

一旦、ここまで修正してエラーがどう変化するか見てみたいです。
おそらく、ここまで変更すると、残りはcontent_paramsをどうするか、という課題だけが残るかと思います。

当初、
@bank = Bank.find(params[:bank_id]),Category.find(params[:category_id])
と書いていたので、@bankにはBankのオブジェクトだけでなくCategoryオブジェクトも入ってしまっていたため、@bankにはオブジェクトの配列ができてしまっていました。
その証拠にundefined methods contents for <Array xxxxxx....>とエラー文が示すように、Array(=配列)にはcontentsはないですよ、というエラーになってしまっています。

ので、一旦コードをスッキリさせるために、Bankだけを取得するように書いてみる、ということです。

form_withの修正については、modelに複数書くことに意味がないため削除した、という理由だけです。

上記、一旦確認してみてください。お願いいたします。

>>>>>>>>>>追記の追記 2021.3.22 08:30

次のエラーが
'''ruby
undefined method `contents_path' for #<#Class:0x00007fb61254a4f0:0x00007fb61d13f8c8>
'''
ですね。
次は

<%= form_with model: @content, url: bank_category_contents_path, class: 'form', local: true do |f| %>

に直せばたぶんいけるんじゃないかなと思います。もしくは、

<%= form_with model: @content, url: [@bank, @category, :contents], method: :post, class: 'form', local: true do |f| %>

かなぁ。

こちらはform_withで生成されたformのURLがルーティングで見つからないよ、というエラーです。
今回のようにmodelに@contentと設定すると、contents_pathが自動で設定されるようになります。

しかし、routes.rbの設定を伺うと、contents_pathに該当するパスはなく、かつ、ご希望を叶えるとするならばbanks/id/categories/id/contentsのパスを生成しなければなりません。
そのため、別途urlオプションにURLパスを指定してあげる必要があります。

後者のほうはパスの別の書き方になります。上の方法がうまくいかなければお試しください。

これで、次にどういったエラーが出るかまた確認したいです。よろしくお願いいたします。

>>>>>>>>>>>>>>追記の追記の追記 2021.3.22 21:30
前回の修正により次の課題がcontent_paramsのエラー解消となったみたいですので、

params.require(:content).permit(:detail,:bank_id,:category_id,:user_id).merge(category_id: category.id ,user_id: current_user.id)

を

params.require(:content).permit(:detail).merge(bank_id: params[:bank_id], category_id: params[:category_id] ,user_id: current_user.id)

に変更して見てください。これで登録できるようになるはず…

エラーコードにParameterとして

{"authenticity_token"=>"Tdl19kGEpFdkQQsgEFHAKhHV4ef1idrapjM73+whvV89uRcWbR2njvyCGwwn3cbU8sqFiwfgQWd8arMK8sdQMw==",
"content"=>{"detail"=>"5000"},
"commit"=>"登録",
"bank_id"=>"2",
"category_id"=>"1"}

が記載されていたと思います。こちらがparamsの値の正体です。Railsを使い始めたばかりでparamsにどんな値が入っているのかわからない、という場合にはエラーを出した際にこちらの値を確認するとわかりやすいです。他の機能を実装する際の参考にしてみてください。

Comments

[kaaaawaaaa]

ご回答ありがとうございます。
Bankモデルへのhas_many :contentsの記載はできていましたので、問題はございませんでした。
「,Category.find(params[:category_id])」についてはcategory_idを取得する上で必要であろうと思い記載しておりました。
※外して試したところ登録に失敗してしまいました。（contents_controller内のストロングパロメーターの部分にmergeでカテゴリIDを追加するように記載してみました（category_id: category.idと追記））

[yoshi_10_11]

has_many :contentsはあるのですね

new.html.erb
<%= form_with model: [@bank,@category,@content], class: 'form', local: true do |f| %>
↓
<%= form_with model: @content, class: 'form', local: true do |f| %>

contents_controller.rb
※testは不要なコードだとお見受けしたので省略
def create
@bank = Bank.find(params[:bank_id])
@content = @bank.contents.new(content_params)

一旦、ここまで修正してエラーがどう変化するか見てみたいです。
おそらく、ここまで変更すると、残りはcontent_paramsをどうするか、という課題だけが残るかと思います。

当初、
@bank = Bank.find(params[:bank_id]),Category.find(params[:category_id])
と書いていたので、@bankにはBankのオブジェクトだけでなくCategoryオブジェクトも入ってしまっていたため、@bankにはオブジェクトの配列ができてしまっていました。
その証拠にundefined methods contents for <Array xxxxxx....>とエラー文が示すように、Array(=配列)にはcontentsはないですよ、というエラーになってしまっています。

ので、一旦コードをスッキリさせるために、Bankだけを取得するように書いてみる、ということです。

form_withの修正については、modelに複数書くことに意味がないため削除した、という理由だけです。

上記、一旦確認してみてください。お願いいたします。

[kaaaawaaaa]

大変ご丁寧にありがとうございます。下記内容に修正してみました。
new.html.erb
<%= form_with model: @content, class: 'form', local: true do |f| %>
contents_controller.rb
　def create
@bank = Bank.find(params[:bank_id])
@content = @bank.contents.new(content_params)
と修正しました。内容登録画面に入るところでエラーとなりました。

NoMethodError in Contents#new
Showing /Users/kawanishimasaya/ai-app/ai-hosho-ebb/app/views/contents/new.html.erb where line #21 raised:

undefined method `contents_path' for #<#<Class:0x00007fb61254a4f0>:0x00007fb61d13f8c8>
Did you mean? content_tag
Extracted source (around line #21):
19　<div class="group">
20 　<%= @category.subject %>
21　　<%= form_with model: @content, class: 'form', local: true do |f| %>
22　　　<div class="form-input">
23　　　　<%= f.text_field :detail, class: 'form-message', placeholder: '最新情報を登録' %>
24　　　　<%= f.submit '登録', class: 'form-submit' %>

Rails.root: /Users/kawanishimasaya/ai-app/ai-hosho-ebb
Application Trace | Framework Trace | Full Trace
app/views/contents/new.html.erb:21
Request
Parameters:
{"bank_id"=>"2", "category_id"=>"1"}

[yoshi_10_11]

ありがとうございます。

ちょっとコメントだとコードが読みにくくなるので、回答の部分を編集して次のコードをお伝えしますのでお待ち下さい！

[yoshi_10_11]

修正しました！

[kaaaawaaaa]

二通り試してみました。双方ともに同じエラー文でした。
内容を見るとおそらくストロングパロメーターに問題があるのではないかと思っています。

NameError in ContentsController#create
undefined local variable or method `category' for #<ContentsController:0x00007ff205fd9e00> Did you mean? categories_url
Extracted source (around line #34):
32　 private
33 def content_params
34　 params.require(:content).permit(:detail,:bank_id,:category_id,:user_id).merge(category_id: category.id ,user_id: current_user.id)
35　 end
36 end

Rails.root: /Users/kawanishimasaya/ai-app/ai-hosho-ebb
Application Trace | Framework Trace | Full Trace
app/controllers/contents_controller.rb:34:in `content_params'
app/controllers/contents_controller.rb:23:in `create'
Request
Parameters:
{"authenticity_token"=>"Tdl19kGEpFdkQQsgEFHAKhHV4ef1idrapjM73+whvV89uRcWbR2njvyCGwwn3cbU8sqFiwfgQWd8arMK8sdQMw==", "content"=>{"detail"=>"5000"}, "commit"=>"登録", "bank_id"=>"2", "category_id"=>"1"}

[yoshi_10_11]

更新しました！

[kaaaawaaaa]

夜分遅くに申し訳ございません。
yoshi_10_11さんのおかげで解決することができました！
また、パラメーターの意味やエラーの意味もよく理解することができ、
私自身の学習にもなりました。感謝でいっぱいです！ありがとうございました！