newアクションから入力されたデータがうまく反映されない。

よろしくお願いいたします。
Rails初学者です。
Railsで自作の牛群管理のウェブアプリを作っています。

管理画面 indexアクション（縦に長いものを一枚に画像にしています。）

データがすでに入っているセルのボタンを押すとshowアクションで個体情報を閲覧できるようになっています。

管理画面から新規作成ボタンを押すと牛のデータを入力するフォームが表示され、フォームに内容を記述してデータを送信したら、フォームに記述された内容を反映したshowアクションに飛ぶようにしたいのですがうまくいきません。

内容を記述したフォーム送信画面

Create Cowボタンをクリックした後に表示される画面

stalls_controller.rb

ruby
1class StallsController < ApplicationController
2  def index
3        @stall = Stall.includes(:cow).order("stallnumber")
4      end
5      
6      def show
7        @stall = Stall.includes(:cow).find(params[:id])
8      end
9      
10      def new
11        @cow = Cow.new(birthday: Date.new(2010,1,1))
12        render template:'cows/new'
13
14      end
15      
16      def edit
17        @stall = Stall.includes(:cow).find(params[:id])
18      end
19      
20      def create
21        @cow = Cow.new(params[:cow])
22        
23          if @cow.save
24            redirect_to @cow, notice: "登録しました。"
25            render template:'cows/show'
26            else
27            render "cows/new"
28          end
29      end
30      
31      def update
32        @stall = Stall.includes(:cow).find(params[:id])
33        @stall.assign_attributes(params[:cow])
34          if @stall.save
35              redirect_to @stall, notice: "更新しました"
36          else
37              render "edit"
38          end
39      end
40      
41      def destroy
42        @stall = Stall.find(params[:id])
43        @stall.destroy
44        redirect_to :stall, notice: "削除しました。"
45      end
46
47end
48

cows_controller.rb

ruby
1class CowsController < ApplicationController
2  def index
3    @stall = Stall.includes(:cow).order("number")
4  end
5        
6        def show
7          @cow = Cow.find(params[:id])
8        end
9        
10        def new
11          @cow = Cow.new(birthday: Date.new(2010,1,1))
12        end
13        
14        def edit
15          @cow = Cow.find(params[:id])
16        end
17        
18        def create
19          @cow = Cow.new(params[:cow])
20            if @cow.save
21              redirect_to @cow, notice: "登録しました。"
22              else
23              render "new"
24            end
25        end
26        
27        def update
28          @cow = Cow.find(params[:id])
29          @cow.assign_attributes(params[:cow])
30            if @cow.save
31              redirect_to @cow, notice: "更新しました"
32            else
33              render "edit"
34            end
35        end
36        
37        def destroy
38          @cow = Cow.find(params[:id])
39          @cow.destroy
40          redirect_to :cow, notice: "削除しました。"
41        end
42
43end
44

stalls/index.html.erb

html
1<% @page_title = "搾乳牛管理" %>
2<h1><%= @page_title %></h1>
3
4
5
6
7  <table class = "index" align = "left">
8  
9     
10     
11     <% @stall.each do |cell|%>
12     <% if cell.cow.present? %>
13     <tr><td><%= link_to cell.cow.number, cell.cow %></td></tr>
14     <% else %>
15     <tr><td><%= link_to "新規作成",new_stall_path(stall_id: cell.id) %></td></tr>
16     <% end %>
17     <% end %>
18     
19     
20  </table>
21

cows/show.html.erb

html
1<% @page_title = "詳細" %>
2
3<h1><%= @page_title %></h1>
4
5<td><%= link_to "編集", [:edit, @cow] %>/
6    <%= link_to "削除",  @stall, method: :delete %></td>
7<td><%= link_to "一覧へ戻る", :stalls%></td>
8
9<table class = "attr">
10  <tr>
11    <th>個体識別番号</th>
12    <th>名号</th>
13    <th>生年月日</th>
14    <th>状態</th>
15  </tr>
16  <tr>
17    <td><%= @cow.number %></td>
18    <td><%= @cow.name %></td>
19    <td><%= @cow.birthday %></td>
20    <td><%= @cow.status == 1 ? "経産牛" : "未経産牛" %></td>
21  </tr>
22</table>

cows/new.html.erb

html
1<% @page_title = "新規作成" %>
2
3<h1><%= @page_title %></h1>
4
5<%= form_for @cow do |form| %>
6  <%= render "form", form: form %>
7  <div><%= form.submit %></div>
8<% end %>

cows/_form.html.erb

html
1<table class = "form">
2   <tr>
3     <th><%= form.label :number, "個体識別番号" %></th>
4     <td><%= form.text_field :number, size: 8 %></td>
5   </tr>
6
7   <tr>
8     <th><%= form.label :name, "名号" %></th>
9     <td><%= form.text_field :name, size: 50 %></td>
10   </tr>
11
12   <tr>
13     <th><%= form.label :birthday, "生年月日",
14             for: "cow_birthday_1i" %></th></th>
15     <td><%= form.date_select :birthday,start_year: 2010, end_year: Time.current.year,
16             use_month_numbers: true %></td>
17   </tr>
18   <tr>
19     <th>状態</th>
20       <td>
21         <%= form.radio_button :status, 1 %>
22         <%= form.label :status_1, "経産牛" %>
23         <%= form.radio_button :status, 2 %>
24         <%= form.label :status_2, "未経産牛" %>
25       </td>
26</table>

フォームを送信する前と後でURLが変わっているので何か起ってはいるのでしょうが具体的にどこがどう間違っているのかわからないです。
フォームを送信したらフォームに記述された内容を反映したshowアクションへ行けるようにするにはどうすればいいか教えていただきたいです。

2019.11.27 21:49更新

_form.html.erb

html
1<%= form.hidden_field :stall_id %>
2
3<table class = "form">
4   <tr>
5     <th><%= form.label :number, "個体識別番号" %></th>
6     <td><%= form.text_field :number, size: 8 %></td>
7   </tr>
8
9   <tr>
10     <th><%= form.label :name, "名号" %></th>
11     <td><%= form.text_field :name, size: 50 %></td>
12   </tr>
13
14   <tr>
15     <th><%= form.label :birthday, "生年月日",
16             for: "cow_birthday_1i" %></th></th>
17     <td><%= form.date_select :birthday,start_year: 2010, end_year: Time.current.year,
18             use_month_numbers: true %></td>
19   </tr>
20   <tr>
21     <th>状態</th>
22       <td>
23         <%= form.radio_button :status, 1 %>
24         <%= form.label :status_1, "経産牛" %>
25         <%= form.radio_button :status, 2 %>
26         <%= form.label :status_2, "未経産牛" %>
27       </td>
28</table>

2019.11.27 22:49　追記

stalls_controller.rb

ruby
1class StallsController < ApplicationController
2  def index
3        @stall = Stall.includes(:cow).order("stallnumber")
4      end
5      
6      def show
7        @stall = Stall.includes(:cow).find(params[:id])
8      end
9      
10      def new
11        @cow = Cow.new(birthday: Date.new(2010,1,1),stall_id: params[:stall_id])
12        render template:'cows/new'
13
14      end
15      
16      def edit
17        @stall = Stall.includes(:cow).find(params[:id])
18      end
19      
20      
21       def cow_params
22          params.require(:cow).permit(:number, :name, :birthday, :status, :stall_id)
23        end
24      def create
25        @cow = Cow.new(cow_params)
26        
27          if @cow.save
28            redirect_to @cow, notice: "登録しました。"
29            render template:'cows/show'
30            else
31            render "cows/new"
32          end
33      end
34      
35      def update
36        @stall = Stall.includes(:cow).find(params[:id])
37        @stall.assign_attributes(params[:cow])
38          if @stall.save
39              redirect_to @stall, notice: "更新しました"
40          else
41              render "edit"
42          end
43      end
44      
45      def destroy
46        @stall = Stall.find(params[:id])
47        @stall.destroy
48        redirect_to :stall, notice: "削除しました。"
49      end
50      
51      def cow_params
52        params.require(:cow).permit(:number, :name, :birthday, :status, :stall)
53      end
54end
55

cows_controller.rb

ruby
1class CowsController < ApplicationController
2  def index
3    @stall = Stall.includes(:cow).order("number")
4  end
5        
6        def show
7          @cow = Cow.find(params[:id])
8        end
9        
10        def new
11          @cow = Cow.new(birthday: Date.new(2010,1,1),stall_id: params[:stall_id])
12        end
13        
14        def edit
15          @cow = Cow.find(params[:id])
16        end
17        
18        def cow_params
19          params.require(:cow).permit(:number, :name, :birthday, :status, :stall_id)
20        end
21        
22        def create
23          @cow = Cow.new(cow_params)
24            if @cow.save
25              redirect_to @cow, notice: "登録しました。"
26              else
27              render "new"
28            end
29        end
30        
31        def update
32          @cow = Cow.find(params[:id])
33          @cow.assign_attributes(params[:cow])
34            if @cow.save
35              redirect_to @cow, notice: "更新しました"
36            else
37              render "edit"
38            end
39        end
40        
41        def destroy
42          @cow = Cow.find(params[:id])
43          @cow.destroy
44          redirect_to :cow, notice: "削除しました。"
45        end
46
47
48end
49

2019.11.28 11:56　追記
新規作成画面（newアクション）

フォームの内容が反映された詳細画面（showアクション）

行動規範の内容に同意します

回答1件

ベストアンサー

createで失敗してます、多分。
newのviewに

<% unless @cow.errors.empty? %>
<%= @cow.errors.fullmassages %>
<% end %>

するとその理由が表示されます。

@cow = Cow.new(params[:cow])
を
@cow = Cow.new(cow_params)
とし

def cow_params
   params.requirequire(:cow).permit(:number,:name,:birthday,:status)
end

してみて下さい

投稿2019/11/27 04:24

winterboum

総合スコア23567

ayumu0622

2019/11/27 05:14

winterboum様前回に引き続きご回答ありがとうございます。 newのviewにご指摘いただいたコードを入力してフォームを送信したところ ["Stall must exist"] と表示されました。ご指摘いただいた@cowの変更、cow_paramsの定義も追加してフォームの送信を行ってみましたが質問時の状況と変わりませんでした（質問の画像に載せた”Create Cowボタンをクリックした後に表示される画面”と同様の状況です）。

winterboum

2019/11/27 05:41

ごめんなさい見落とした、 params.requirequire(:cow).permit(:number,:name,:birthday,:status) に :stall を追加して下さい

ayumu0622

2019/11/27 06:50

すいません params.require(:stall).require(:cow).permit(:number,:name,:birthday,:status) ということでよろしいですか？

winterboum

2019/11/27 06:54

params.require(:cow).permit(:number,:name,:birthday,:status,:stall) です

ayumu0622

2019/11/27 11:59

ご指摘いただいたように変更しましたが質問時の状況と変わりませんでした。 newのviewでも["Stall must exist"]がまだ表示されています。記述間違いもあるかもしれないので一応質問の方に変更後のcows_controller.rbとstalls_controller.rbを追加しましたので見ていただきたいです。

winterboum

2019/11/27 12:35

ああ、、、いかん cows/_form.html.erb から stallについての情報が送られてきていないですね。 def new @cow = Cow.new(birthday: Date.new(2010,1,1), stall_id: params[:stall_id]) とし、 cows/_form.html.erb　の最初に <%= form.hidden_field :stall_id %> をいれ、 params.require(:cow).permit(:number,:name,:birthday,:status,:stall) を params.require(:cow).permit(:number,:name,:birthday,:status,:stall_id) に修正

ayumu0622

2019/11/27 12:54

変更しましたが状況変わらずです．．．変更後の_form.html.erbを質問の方に追記しました。

winterboum

2019/11/27 13:24

def new、params.require　も直しましたか？ logを見せて下さい

ayumu0622

2019/11/28 00:25 編集

Started GET "/stalls/new?stall_id=27" for ::1 at 2019-11-27 21:52:48 +0900 Processing by StallsController#new as HTML Parameters: {"stall_id"=>"27"} Rendering cows/new.html.erb within layouts/application Rendered stalls/_form.html.erb (2.3ms) Rendered cows/new.html.erb within layouts/application (13.4ms) Completed 200 OK in 48ms (Views: 46.9ms | ActiveRecord: 0.0ms) Started POST "/cows" for ::1 at 2019-11-27 21:52:52 +0900 Processing by CowsController#create as HTML Parameters: {"utf8"=>"✓", "authenticity_token"=>"cMR8XdCaH2Q2+vAeh2MALO0ZgJQAW16Hspquqn9R3YNlbYS4iUCWaCdUgeaOgLFVeouf3s9GkBsXXkXWm5XvHw==", "cow"=>{"number"=>"9978", "name"=>"テスト　テスト　テスト", "birthday(1i)"=>"2010", "birthday(2i)"=>"1", "birthday(3i)"=>"1", "status"=>"1"}, "commit"=>"Create Cow"} [1m[35m (0.1ms)[0m [1m[36mbegin transaction[0m ↳ app/controllers/cows_controller.rb:24 [1m[35m (0.1ms)[0m [1m[31mrollback transaction[0m ↳ app/controllers/cows_controller.rb:24 Rendering cows/new.html.erb within layouts/application Rendered cows/_form.html.erb (3.2ms) Rendered cows/new.html.erb within layouts/application (11.0ms) Completed 200 OK in 53ms (Views: 48.3ms | ActiveRecord: 0.2ms)

ayumu0622

2019/11/27 14:10

def newとparams.requireは変更してあります。変更後のものを質問の方に追加しました。 logは上のものでいいでしょうか？質問の方に追加しようとすると文字数超過するのでとりあえずこちらにコピーしたものを張りました。見づらくて申し訳ありません。

winterboum

2019/11/27 14:37

Started GET "/stalls/new?stall_id=27" for :: から下だけ残して下さい。さて、最後のPOSTのParametersを見ると、stall_idが無いですね、nilが渡ってるっぽい。が、なぜ？ codeもう一度おいかけますが、ayumuさんも　ミスタイプ無いか　ブラウザのnew画面のｈｔｍｌのソースをみて、 name=cow[stall_id]　な　idden にどんな値が入っているか確認してください。

ayumu0622

2019/11/28 00:56

確認しましたがミスタイプはおそらくないと思います。 new画面のhtmlのソースですが、name=cow[stall_id]という文言が見つかりませんでした。 <%= form.hidden_field :stall_id %>が表示されてるあたりは <form class="new_cow" id="new_cow" action="/cows" accept-charset="UTF-8" method="post"><input name="utf8" type="hidden" value="✓" /><input type="hidden" name="authenticity_token" value="jyx/DHduwS59kXwf/8sMxX4edZh/QW2mHflMDE0U7RfRyyhLpTokLayLR25xMK6XEbp5Jpq8lhuhgbtKDfO/Yg==" /> となっていました。それとこの件に直接関係あるかはわからないのですが、newアクションを最初に表示したときのURLが http://localhost:3000/stalls/new?stall_id=26 と表示されるんですが、Create Cowボタンを押すと http://localhost:3000/cows CowControllerのindexアクションになぜか飛んでいます。見た目上はnewアクションのフォームが表示されているのですがページを更新すると ActionController::UnknownFormat in CowsController#indexが出てきます。何か解決の糸口になるでしょうか．．．

winterboum

2019/11/28 01:03

すると、hidden_fieldでなにか間違えてるかな。。。で、URLが変わるのは、始めは StallsControllerのnewですが、書き直し画面は Cowのcreateからになっているからです

ayumu0622

2019/11/28 02:55

URLについて、ありがとうございます。今試しに<%= form.hidden_field :stall_id %>を最初にご指摘いただいていた_form.html.erbではなくcows/new.html.erbへ以下のように書き換えて記述してみたところフォームの内容が反映されたshowアクションにレンダリングされました。 <%= form_for @cow do |form| %> <%= form.hidden_field :stall_id %> <%= render "form", form: form %> <div><%= form.submit %></div> <% end %> 質問の方にフォームの画面、反映されたshowアクションの画面を載せます。 hidden_fieldの記述する場所が違ったからうまく反映されていなかったということだったんでしょうか？

ayumu0622

2019/11/28 02:58

logもこちらに載せます。 Started POST "/cows" for ::1 at 2019-11-28 11:51:11 +0900 Processing by CowsController#create as HTML Parameters: {"utf8"=>"✓", "authenticity_token"=>"E6W/BVKISo2cqqDnc5vzoM7LKZQvyYMGjZGrV3tktUFZu/gJXo4sKcjb0hFXiDWHrLKfPZwyfT8teOJJG7+Rhg==", "cow"=>{"stall_id"=>"40", "number"=>"11111", "name"=>"テスト　テスト　テスト", "birthday(1i)"=>"2010", "birthday(2i)"=>"1", "birthday(3i)"=>"1", "status"=>"1"}, "commit"=>"Create Cow"} (0.1ms) begin transaction ↳ app/controllers/cows_controller.rb:24 Stall Load (0.3ms) SELECT "stalls".* FROM "stalls" WHERE "stalls"."id" = ? LIMIT ? [["id", 40], ["LIMIT", 1]] ↳ app/controllers/cows_controller.rb:24 Cow Create (725.9ms) INSERT INTO "cows" ("stall_id", "number", "name", "birthday", "created_at", "updated_at") VALUES (?, ?, ?, ?, ?, ?) [["stall_id", 40], ["number", 11111], ["name", "テスト　テスト　テスト"], ["birthday", "2010-01-01"], ["created_at", "2019-11-28 02:51:11.035494"], ["updated_at", "2019-11-28 02:51:11.035494"]] ↳ app/controllers/cows_controller.rb:24 (169.4ms) commit transaction ↳ app/controllers/cows_controller.rb:24 Redirected to http://localhost:3000/cows/25 Completed 302 Found in 914ms (ActiveRecord: 895.8ms) Started GET "/cows/25" for ::1 at 2019-11-28 11:51:11 +0900 Processing by CowsController#show as HTML Parameters: {"id"=>"25"} Cow Load (0.7ms) SELECT "cows".* FROM "cows" WHERE "cows"."id" = ? LIMIT ? [["id", 25], ["LIMIT", 1]] ↳ app/controllers/cows_controller.rb:7 Rendering cows/show.html.erb within layouts/application Rendered cows/show.html.erb within layouts/application (1.1ms) Completed 200 OK in 62ms (Views: 47.0ms | ActiveRecord: 0.7ms)

winterboum

2019/11/30 12:07

動いてよかったです。ただ、partialの外でも中でも form の中なら同じはずなので、疑問です

ayumu0622

2019/11/30 12:26

自分でもしっかり理解できているかわからないまま解決してしまっているので正直不安ではあります．．． ”hidden_fieldとhidden_field_tagの違い”のような解説記事で出てきた記述方法を真似して書いてみたらなぜか動いた、というような印象です．．．

行動規範の内容に同意します