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

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

ただいまの
回答率

87.37%

2つのモデル間のリレーションがはれない

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 849

score 15

以下のようなモデル間(主:TargetServer, 副:Disk)で
リレーションを貼ってひとつのフォームで複数のモデルをDBへ登録できるようにしようとしているのですが、TargetServerをAddしようとすると

undefined method `disk' for #<Targetserver:0x00007f7c25709358>

と出てしまいます。。

いろいろと調べたのですがうまく解決できなかったため どなたか対応法をご教授いただければ幸いです。 よろしくお願いいたします。

  • app/models/targetserver.rb
class Targetserver < ApplicationRecord
  has_many :disks
  accepts_nested_attributes_for :disks
end
  • app/models/disk.rb
class Disk < ApplicationRecord
  belongs_to :targetserver
end
  • app/controllers/targetservers_controller.rb
class TargetserversController < ApplicationController
  before_action :set_targetserver, only: [:show, :edit, :update, :destroy]

  # GET /targetservers
  # GET /targetservers.json
  def index
    @targetservers = Targetserver.all
  end

  # GET /targetservers/1
  # GET /targetservers/1.json
  def show
  end

  # GET /targetservers/new
  def new
    @targetserver = Targetserver.new
    @targetserver.disks.build
  end

  # GET /targetservers/1/edit
  def edit
  end

  # POST /targetservers
  # POST /targetservers.json
  def create
    @targetserver = Targetserver.new(targetserver_params)

    respond_to do |format|
      if @targetserver.save
        format.html { redirect_to @targetserver, notice: 'Targetserver was successfully created.' }
        format.json { render :show, status: :created, location: @targetserver }
      else
        format.html { render :new }
        format.json { render json: @targetserver.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /targetservers/1
  # PATCH/PUT /targetservers/1.json
  def update
    respond_to do |format|
      if @targetserver.update(targetserver_params)
        format.html { redirect_to @targetserver, notice: 'Targetserver was successfully updated.' }
        format.json { render :show, status: :ok, location: @targetserver }
      else
        format.html { render :edit }
        format.json { render json: @targetserver.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /targetservers/1
  # DELETE /targetservers/1.json
  def destroy
    @targetserver.destroy
    respond_to do |format|
      format.html { redirect_to targetservers_url, notice: 'Targetserver was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_targetserver
      @targetserver = Targetserver.find(params[:id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def targetserver_params
      params.require(:targetserver).permit(:hostname, :cpumemory, :os, disks_attributes: [:disk])
    end
end

(TargetServerをAddする際に呼び出す_form)

<%= form_with(model: targetserver, local: true) do |form| %>
  <% if targetserver.errors.any? %>
    <div id="error_explanation">
      <h2><%= pluralize(targetserver.errors.count, "error") %> prohibited this targetserver from being saved:</h2>

      <ul>
      <% targetserver.errors.full_messages.each do |message| %>
        <li><%= message %></li>
      <% end %>
      </ul>
    </div>
  <% end %>

  <div class="field">
    <%= form.label :hostname %>
    <%= form.text_field :hostname, id: :targetserver_hostname %>
  </div>

  <div class="field">
    <%= form.label :cpumemory, "CPU & Memory" %>
    <%= form.text_field :cpumemory, id: :targetserver_cpumemory %>
  </div>

  <div class="field">
    <%= form.label :os, "OS" %>
    <%= form.text_field :os, id: :targetserver_os %>
  </div>

  <%= @targetserver.disk.each do |disk| %>
    <%= hidden_field_tag :partitionname, id: :disk_partitionname, :value => "\/" %>
    <%= disk.label :disk_size %>
    <%= disk.text_field :size, id: :disk_size %>
  <% end %>

  <div class="actions">
    <%= form.submit %>
  </div>
<% end %>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

フォームのビューに

@targetserver.disk.each do |disk|

という箇所がありますが,この disk が正しくは disks と複数形なのではないでしょうか。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/03/08 01:19

    scivolaさん


    アドバイスありがとうございます!
    ご指摘いただいた修正を施したところ
    当該箇所はパスしたようなのですが、
    以下のエラーが出てしまう状態です。。。

    Diskのコントローラやモデルを
    さらに修正する必要があるのでしょうか。。。


    立て続けの質問で大変恐縮ですが
    もし修正箇所わかりましたらご指摘いただけますと幸いです。。。


    undefined method `label' for #<Disk:0x00007f58904d4690>
    Extracted source (around line #31):

    <%= @targetserver.disks.each do |disk| %>
    <%= hidden_field_tag :partitionname, id: :disk_partitionname, :value => "\/" %>
    <%= disk.label :disk_size, "Disk Size" %> <<<<<
    <%= disk.text_field :size, id: :disk_size %>
    <% end %>

    キャンセル

  • 2018/03/08 02:29 編集

    入れ子の form の画面をつくるには fields_forを使います。
    @targetserver.disks.each do |disk| %> で
    disk,label や disk.text_field としたでは、エラーになります。 disk は form の要素でなく、Activerecord の要素なので label や text_filed なんてメソッドは持っていませんから。
    参考情報
    [Railsでaccepts_nested_attributes_forとfields_forを使ってhas_many関連の子レコードを作成/更新するフォームを作成]
    http://ruby-rails.hatenadiary.com/entry/20141208/1418018874#accepts_nested_attributes_for-form

    キャンセル

  • 2018/03/08 18:36

    katoy-san

    --
    <%= form.fields_for :disks do |disk| %>
    <%= hidden_field_tag :partitionname, id: :disk_partitionname, :value => "\/" %>
    <%= disk.label :disk_size, "Disk Size" %>
    <%= disk.text_field :size, id: :disk_size %>
    <% end %>
    --

    こちらのコードに変更したところ、
    disksが無事にDBにレコードされました!
    ありがとうございます。
    まだ実際の値が入らないので、
    コントローラなどの修正が必要なようなのですが
    もう少し確認してみます。

    ご指摘ありがとうございました。

    キャンセル

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

  • ただいまの回答率 87.37%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る