アソシエーションについてn側のnewに行くとエラーが発生する

Question

こんにちは。

現在プログラミングを行なっている中で、アソシエーションを使ってcompany:carrier = 1:多

の関係を構築しております。

その中で、companies/showからcompaniesの情報を引き継いだママcarriers/newにページを移し、createする方法がわからずサポート頂きたいです。

以下、エラー内容となります。

ActionView::Template::Error (undefined method `carriers' for nil:NilClass):

    1: <%= form_for  ([:mypage, @company, @company.carriers.build]) do |f| %>
    2: <div><%= hidden_field_tag 'carrier[company_id]', session[:company_id] %></div>
    3:  <table class="company">
    4:     <tbody>
  
app/views/mypage/carriers/_form.html.erb:1:in `_app_views_mypage_carriers__form_html_erb__2390391367751555058_70312819412600'
app/views/mypage/carriers/new.html.erb:6:in `_app_views_mypage_carriers_new_html_erb___3009204651057680812_70312819282440'

一応エラーで言われていること自体は理解しておりますが、じゃあどう修正していけばいいかが不明です。

<%= form_for  ([:mypage, @company, @company.carriers.build]) do |f| %>

  def new
    @carrier = Carrier.new(company: @company)
  end

  def create
    @company = Company.find(carrier_params[:company_id])
    if @carrier.save
      redirect_to mypage_companies_path
    else
      render 'new'
    end
  end

ご教示下さい。

よろしくお願い致します。

Answer 1

def new
    @carrier = Carrier.new(company: @company)
end

これは cmpany の controllerではなく carrierのcontrpollerなのですね？

＞ companies/showからcompaniesの情報を引き継いだママ
の引き継ぐ情報が @company　だと思うのですが、この new の中で定義されていないので、　@carrier = Carrier.new(company: nil) という意味になります。これは望みと違うかと。
まずここを解決しましょう。

で、@carrier を定義したのなら、
<%= form_for ([:mypage, @company, @company.carriers.build])
ここは
<%= form_for ([:mypage, @carriers)
で良いように思うのですが、 @company のscopeに入れたい理由が有るのでしょうか

Answer 2

def newの部分ですが、

@carrier = Carrier.new(company: @company)

ではなく

@company = Company.new()

とかだと思ったのですが汗
carrierをformで使われていないので...

Comments

[KOO_]

この形でcarrierが生成されるのでしょうか？

[bamboo-nova]

すみません、これは私なりの記述になりますが、
def new
@carrier = Company.carriers.build
end

def create
@carrier = Company.carriers.build(carrier_params[:company_id])
以下略
end

carriers/newページ
<%= form_for (@carrier) do |f| %>
<div><%= hidden_field_tag 'carrier[company_id]', session[:company_id] %></div>
以下略

と書くかなと思います(他の部分は記載されていないので分からないです)。createアクションとかでもインスタンスが間違っていたりとミスも多いので、そちらにも気をつけて欲しいと思います。

[KOO_]

ご丁寧にありがとうございます。この場合
NoMethodError (undefined method `carriers' for #<Class:0x00007f9f1c842390>
Did you mean? cattr_writer):

app/controllers/mypage/carriers_controller.rb:28:in `new'

とcarriersに対してエラーが出るのですが、何が考えられるでしょうか？

よろしくお願い致します。

[bamboo-nova]

モデル部分にbelongs_toとhas_manyは記載していますか？

[KOO_]

はい。記載されております！

[bamboo-nova]

今更なんですが、
_form.html.erbなので、パーシャルなんですね。パーシャルの場合は@をつけないです。どのようにrenderされたかは表示されていないのでわかりませんが、

<%= render '_form', carrier: @carrier %>として、パーシャルでは
<%= form_for (carrier) do |f| %>ですね。

最初の質問でのエラーは、そもそもパーシャルで@としているのが原因ですね...気づかずすみませんでした汗