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

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

ただいまの
回答率

91.02%

  • Ruby

    6361questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    6190questions

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

  • MySQL

    5090questions

    MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

Railsで入力フォームからのデータを外部キーを持たせて保存する方法を教えてください。

解決済

回答 1

投稿

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

koume

score 187

Rails5.1.3でWebアプリケーション制作の勉強をしています。CustomersテーブルとPointsテーブルがあるのですが
Pointsテーブルに入力フォームで入力したデータを保存する際に外部キー(customer_id)の値も入れたいのですが
どこにどのように記述したらいいのでしょうか?コードは以下になります。

points_controller.rb

def create
    @point_form = Customer::PointForm.new
    @point_form.assign_attributes(params[:form])
    if @point_form.save
      flash.notice = 'ポイントを登録しました。'
      redirect_to action: 'show'
    else
      flash.now.alert = '入力に誤りがあります。'
      render actionn: 'new'
    end
  end
point.rb (モデル)

class Point < ActiveRecord::Base
  belongs_to :customer
end
new.html.erb

<% @title = 'ポイントの新規登録' %>
<h1><%= @title %></h1>

<div id="generic-form">
  <%= form_for @point_form, as: 'form', url: :customer_point do |f| %>
    <%= render 'form', f: f %>
    <div class="buttons">
      <%= f.submit '登録' %>
      <%= link_to 'キャンセル', :customer_root %>
    </div>
  <% end %>
</div>
_form.html.erb

<%= FormPresenter.new(f, self).notes %>
<fieldset id="Point-fields">
  <legend>ポイント情報(正確に入力してください。)</legend>
  <%= render 'point_fields', f: f, confirming: false %>
</fieldset>
_point_fields.html.erb

<%= f.fields_for :point, f.object.point do |ff| %>
  <%= markup do |m|
    p = confirming ? ConfirmingPointFormPresenter.new(ff, self) :  PointFormPresenter.new(ff, self)
    p.with_options(required: true) do |q|
      m << q.1game_block(:1game, '1回目')
      m << q.2game_block(:2game, '2回目')
      m << q.3game_block(:3game, '3回目')
      m << q.4game_block(:4game, '4回目')
      m << q.5game_block(:5game, '5回目')
    end
  end %>
<% end %>


createアクションに以下のように記述したらいいのでしょうか?

points_controller.rb

def create
  @point_form = Customer::PointForm.new
  @point_form.assign_attributes(params[:form])
 
#①のコード
 customer = Customer.find_by(id: current_customer.id)
  point = customer.build_point(customer_id: current_customer.id)
  point.save

#②のコード
 customer = Customer.find_by(id: current_customer.id)
  customer.point.update(customer_id: current_customer.id)

  if @point_form.save
    flash.notice = 'ポイントを登録しました。'
    redirect_to action: 'show'
  else
    flash.now.alert = '入力に誤りがあります。'
    render actionn: 'new'
  end
end


①のコード、②のコードのどちらかのような感じでいいのでしょうか?
それとも別のコード(方法)があるのでしょうか?
どなたか教えていただけないでしょうか?宜しくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

@point_form.asign_attributes(customer_id: current_sustomer.id)


これでいいのでは?

なお、1、2でcustomer = Customer.find_by(id: current_customer.id)とやっていますが、customerとcurrent_customerは同じものですので不必要です。

 1の問題点
@point_form.assign_attributes(params[:form])


で出来たPointとは別のPointを作成しているのでおかしい

 2の問題点
customer = Customer.find_by(id: current_customer.id)
customer.point.update(customer_id: current_customer.id)


ってやってるけど、customer.id = current_customer.idなんだから、2行目の意味はないです。customer.pointで拾えるpointのcustomer_idには既にcustomer.idが入っています。
やるなら、customer.point = @point_form.pointみたいな書き方です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/18 19:56

    お礼が遅くなり大変申し訳ございませんでした。回答ありがとうございます。うまく行きました。
    今後とも宜しくお願いします。

    キャンセル

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

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

関連した質問

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

  • Ruby

    6361questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    6190questions

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

  • MySQL

    5090questions

    MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。