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

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

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

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

Ajax

Ajaxとは、Webブラウザ内で搭載されているJavaScriptのHTTP通信機能を使って非同期通信を利用し、インターフェイスの構築などを行う技術の総称です。XMLドキュメントを指定したURLから読み込み、画面描画やユーザの操作などと並行してサーバと非同期に通信するWebアプリケーションを実現することができます。

Q&A

解決済

1回答

1057閲覧

form_forでcreateしたいのにupdateになってエラーになってしまう

mikannOishiii

総合スコア6

Ruby on Rails

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

Ajax

Ajaxとは、Webブラウザ内で搭載されているJavaScriptのHTTP通信機能を使って非同期通信を利用し、インターフェイスの構築などを行う技術の総称です。XMLドキュメントを指定したURLから読み込み、画面描画やユーザの操作などと並行してサーバと非同期に通信するWebアプリケーションを実現することができます。

0グッド

0クリップ

投稿2020/04/15 17:37

編集2020/04/15 17:39

日付に対してその日の血圧を記録するアプリを作っています。
フォーム(form_for)の日付フィールドにはデフォルトで今日の日付が入っていて、
今日のデータがあれば update, なければ create されます。
日付フィールドを変更すれば、過去のデータも編集・更新できます。

大体の実装は終わった、、、と思いきや、
日付フィールドを別のまだデータがない日付に変更してデータを登録しようとすると、
新規作成ができないことがわかりました。
どなたかお知恵を貸していただけると嬉しいです。

前提・実現したいこと

日付フィールドからデータが登録されていない日付に変更したあと、create したい。

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

create したいのに update に飛ばされてしまい、
新規作成だけエラーになってしまう。
(データがある日付に変更した場合は編集・更新できる。)

server

1Started PATCH "/records/89" for ::1 at 2020-04-16 01:11:04 +0900 2Processing by RecordsController#update as JS 3 Parameters: {"authenticity_token"=>"xBV8SEFWbXWM+y+oTTdaJErTh/cd9EVCfuzZYYSU6Adi6mNqYQ7p47K8pJim0pw6Kp2yCffkbXpeg85kL6ok 4nQ==", "record"=>{"date"=>"2020-04-04", "m_sbp"=>"140", "m_dbp"=>"", "m_pulse"=>"", "n_sbp"=>"", "n_dbp"=>"", "n_pulse"=>" 5", "memo"=>""}, "commit"=>"記録する", "id"=>"89"} 6 User Load (0.4ms) SELECT "users".* FROM "users" WHERE "users"."id" = ? ORDER BY "users"."id" ASC LIMIT ? [["id", 1], [ 7"LIMIT", 1]] 8 Record Load (0.3ms) SELECT "records".* FROM "records" WHERE "records"."user_id" = ? AND "records"."date" = ? LIMIT ? [ 9["user_id", 1], ["date", "2020-04-04"], ["LIMIT", 1]] 10 ↳ app/controllers/records_controller.rb:35:in `update' 11Completed 500 Internal Server Error in 22ms (ActiveRecord: 0.8ms | Allocations: 3254) 12 13NoMethodError (undefined method `update_attributes' for nil:NilClass):

@record が nil のためエラーになっています。
一行目にある通り、submit の飛び先が PATCH になっているのが原因と思われます。

DBに登録されているか否かで form_for がよしなに create と update を切り分けてくれると思ったのですが、うまくできませんでした。

なお、入力フォームはモーダルで実装、日付変更もAjaxで実装しています。
以下、routes, controller, view と ajax のコードです。

該当のソースコード

ruby

1# config/routes.rb 2 3Rails.application.routes.draw do 4 resources :records, only: [:new, :create, :edit, :update] do 5 get :search, on: :collection 6 end 7end

ruby

1# app/controllers/records_controller.rb 2 3class RecordsController < ApplicationController 4 before_action :authenticate_user! 5 6 def new 7 user = current_user 8 if user.records.exists?(date: Time.now) 9 @record = user.records.find_by(date: Time.now) 10 else 11 @record = Record.new(date: Time.now) 12 end 13 end 14 15 def create 16 @record = Record.new(record_params) 17 respond_to do |format| 18 if @record.save 19 format.html{ redirect_to root_url, notice: 'Record was successfully created.'} 20 else 21 format.js { render :new } 22 end 23 end 24 end 25 26 def edit 27 user = current_user 28 @record = user.records.find_by(date: params[:record][:date]) 29 end 30 31 def update 32 user = current_user 33 @record = user.records.find_by(date: params[:record][:date]) 34 respond_to do |format| 35 if @record.update_attributes(record_params) 36 format.html { redirect_to root_url, notice: "Record was successfully updated" } 37 else 38 format.js { render :edit } 39 end 40 end 41 end 42 43 def search 44 user = current_user 45 @record = user.records.find_by(date: params[:date]) 46 respond_to do |format| 47 if @record.nil? 48 @record = Record.new(date: params[:date]) 49 format.json { render json: @record } 50 else 51 format.json { render json: @record } 52 end 53 end 54 end 55 56 private 57 58 def record_params 59 params.require(:record).permit(:date, :m_sbp, :m_dbp, :m_pulse, 60 :n_sbp, :n_dbp, :n_pulse, :memo,) 61 .merge(user_id: current_user.id) 62 end 63end

Ajax で変更された日付の値を返すようにしています。

javascript

1$(document).on('change', '#inputDate','turbolinks:load', function(){ //日付フィールド変更時 2 var inputDate = $("#inputDate").val(); 3$.ajax({ //ajax通信で以下を実行 4 url: '/records/search', 5 type: 'GET', 6 data: ('date=' + inputDate), //コントローラーに渡すデータ 7 processData: false, 8 contentType: false, 9 dataType: 'json' 10 }) 11 .done(function(data){ 12 $("#inputM_SBP").val(data.m_sbp); 13 $("#inputM_DBP").val(data.m_dbp); 14 //省略 15 }) 16 .fail(function(data){ 17 $("#inputM_SBP").val(""); 18 $("#inputM_DBP").val(""); 19 //省略 20});

erb

1<!-- Modal --> 2<div class="modal-dialog" role="document"> 3 <div class="modal-content"> 4 <%= form_for @record, remote: true do |f| %> 5 <% if @record.errors.any? %><!--省略--><% end %> 6 7 <div class="form-group"> 8 <%= f.label :date, for: "inputDate" %> 9 <%= f.date_field :date, class: "form-control", id: "inputDate", value: @record.date %> 10 </div> 11 12 <p class="rcd-lbl">朝:</p> 13 <div class="form-row"> 14 <div class="form-group col-md-4"> 15 <%= f.label :m_sbp, for: "inputM_SBP", class: "sr-only" %> 16 <%= f.telephone_field :m_sbp, placeholder: "最高血圧", class: "form-control", id: "inputM_SBP", value: @record.m_sbp %> 17 </div> 18 19 <div class="form-group col-md-4"> 20 <%= f.label :m_dbp, for: "inputM_DBP", class: "sr-only" %> 21 <%= f.telephone_field :m_dbp, placeholder: "最低血圧", class: "form-control", id: "inputM_DBP", value: @record.m_dbp %> 22 </div> 23 24 <!--省略--> 25 26 <div class="form-group"> 27 <%= f.submit "記録する", class: "btn btn-block btn-blue", id: "record-btn" %> 28 </div> 29 <% end %> 30 </div> 31</div>

補足情報(FW/ツールのバージョンなど)

・ruby 2.6.5
・rails 6.0.2

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

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

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

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

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

guest

回答1

0

ベストアンサー

「DBに登録されているか否かで form_for がよしなに create と update を切り分けてくれる」のは正しい認識です。
ただしそれを行ってくれるのは ブラウザに送り返すとき でブラウザから戻ってくるときは、form_for は通らないのでそうは行きません。

動きを想像するに、
edit 画面で日付を変えて別の日の情報を入力する
ということかと思います。
editでは存在するRecordを読んで送り出しますから、form_for は PATCH と判断します。ですので update に送られる。
@record = user.records.find_by(date: params[:record][:date])
「がnilだったら、新しい日付の入力として create の処理を行う」様にしましょう

投稿2020/04/16 02:31

winterboum

総合スコア23284

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

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

mikannOishiii

2020/04/16 04:26

ご確認ありがとうございます。 edit に下記を追加して試みましたが、結果は同じでした。 ``` if @record.nil? @record = Record.new(date: params[:record][:date]) end ``` 「新しい日付の入力として create の処理を行う」というのは具体的にどのような処理を書けば、create に飛ばせるのでしょうか??? ヒントだけでもご教授いただけると大変嬉しいです。
winterboum

2020/04/16 04:33

いえ、updateに行ってしまうのですから、updateに書くのです、
mikannOishiii

2020/04/16 05:00

ありがとうございます! nilなら new, nil出ないなら update にすることで無事できました! find_by_create を検討しかけていましたが、これなら空のデータを生成せずに好きな日付で作りたいデータだけ作れそうです!!!
winterboum

2020/04/16 05:13

使うなら、find_or_initialize ですね
winterboum

2020/04/16 05:14

使うなら、find_or_initialize_by でしょうね
mikannOishiii

2020/04/16 06:17

ありがとうございます! 勉強になりました!!!!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問