newアクションについて

Question

よろしくお願いします。
自作のウェブアプリケーションを作っているのですがnewアクションを自分の考えるような設定にできません。

一覧表示（indexアクション）

新規作成フォーム（newアクション）

新規作成をクリック（仮に最上部646の右隣のセルの新規作成をクリックしたとする）
↓
フォームに内容を記述、フォームを送信
↓
646の右隣のセルに新しく作成したデータが入る

というものにしたいのですがうまくいきません。

main_cows_controller.rb

class MainCowsController < ApplicationController
  def index
      @maincow = MainCow.order("number")
    end
    
    def show
      @maincow = MainCow.find(params[:id])
    end
    
    def new
      @maincow = MainCow.new(birthday: Date.new(2010,1,1))
    end
    
    def edit
      @maincow = MainCow.find(params[:id])
    end
    
    def create
      @maincow = MainCow.new(params[:main_cow])
        if @maincow.save
          redirect_to @maincow, notice: "登録しました。"
        else
          render "new"
        end
    end
    
    def update
      @maincow = MainCow.find(params[:id])
      @maincow.assign_attributes(params[:main_cow])
        if @maincow.save
          redirect_to @maincow, notice: "更新しました"
        else
          render "edit"
        end
    end
    
    def destroy
      @maincow = MainCow.find(params[:id])
      @maincow.destroy
      redirect_to :main_cow, notice: "削除しました。"
    end

end

index.html.erb

<% @page_title = "搾乳牛管理" %>
<h1><%= @page_title %></h1>


<% if @maincow.present? %>

  <table class = "index" align = "left">
  
    <% 20.downto(1).to_a.each do |i|%>
    <% @cow = @maincow.find_by line: i %>
     <% if @cow.present? %>
     <tr><td><%= link_to @cow.number, @cow %></td></tr>
     <% else %>
     <tr><td><%= link_to "新規作成",new_main_cow_path %></td></tr>
     <% end %>
   <% end %>
  </table>
 
  <table class = "index" align = "left">
      <% 40.downto(21).to_a.each do |i|%>
    <% @cow = @maincow.find_by line: i %>
     <% if @cow.present? %>
     <tr><td><%= link_to @cow.number, @cow %></td></tr>
     <% else %>
     <tr><td><%= link_to "新規作成",new_main_cow_path %></td></tr>
     <% end %>
   <% end %>
   
  </table>
  
<% end %>

new.html.erb

<% @page_title = "新規作成" %>

<h1><%= @page_title %></h1>

<%= form_for @maincow do |form| %>
  <%= render "form", form: form %>
  <div><%= form.submit %></div>
<% end %>

_form.html.erb

<table class = "form">
   <tr>
     <th><%= form.label :number, "個体識別番号" %></th>
     <td><%= form.text_field :number, size: 8 %></td>
   </tr>

   <tr>
     <th><%= form.label :name, "名号" %></th>
     <td><%= form.text_field :name, size: 50 %></td>
   </tr>

   <tr>
     <th><%= form.label :birthday, "生年月日",
             for: "maincow_birthday_1i" %></th></th>
     <td><%= form.date_select :birthday,start_year: 2010, end_year: Time.current.year,
             use_month_numbers: true %></td>
   </tr>
</table>  

マイグレーションスクリプト

class CreateMainCows < ActiveRecord::Migration[5.2]
  def change
    create_table :main_cows do |t|
      t.integer :number, null: false
      t.string :name, null: false
      t.date :birthday, null: false

      t.timestamps
    end
  end
end

マイグレーションスクリプトその2

class AlterMainCows < ActiveRecord::Migration[5.2]
  def change
    add_column :main_cows, :line, :integer
  end
end

フォームにlineカラムを入力できるようにしておけば問題自体は解決するとは思うのですが、
新規作成をクリックしたら、クリックしたセルのlineカラムが自動で設定されて、フォームではnumberカラム、 nameカラム、birthdayカラムのみを入力するようにしたいです。
何かいい方法はないでしょうか？

2019.11.25追記

stalls_controller.rb

class StallsController < ApplicationController
  def index
        @stall = Stall.includes(:cow).order("stallnumber")
      end
      
      def show
        @stall = Stall.includes(:cow).find(params[:id])
      end
      
      def new
        @cow = Cow.new(birthday: Date.new(2010,1,1))
          render template:'cows/new'
      end
      
      def edit
        @stall = Stall.includes(:cow).find(params[:id])
      end
      
      def create
        @cow = Cow.new(params[:cow])
          if @cow.save
            redirect_to @cow, notice: "登録しました。"
            render template:'cows/show'
          else
            render "new"
          end
      end
      
      def update
        @stall = Stall.includes(:cow).find(params[:id])
        @stall.assign_attributes(params[:cow])
          if @stall.save
              redirect_to @stall, notice: "更新しました"
          else
              render "edit"
          end
      end
      
      def destroy
        @stall = Stall.find(params[:id])
        @stall.destroy
        redirect_to :stall, notice: "削除しました。"
      end

end

cows_controller.rb

class CowsController < ApplicationController
  def index
          @stall = Stall.includes(:cow).order("number")
        end
        
        def show
          @cow = Cow.find(params[:id])
        end
        
        def new
          @cow = Cow.new(birthday: Date.new(2010,1,1))
        end
        
        def edit
          @cow = Cow.find(params[:id])
        end
        
        def create
          @cow = Cow.new(params[:cow])
            if @cow.save
              redirect_to @cow, notice: "登録しました。"
            else
              render "new"
            end
        end
        
        def update
          @cow = Cow.find(params[:id])
          @cow.assign_attributes(params[:cow])
            if @cow.save
              redirect_to @cow, notice: "更新しました"
            else
              render "edit"
            end
        end
        
        def destroy
          @cow = Cow.find(params[:id])
          @cow.destroy
          redirect_to :cow, notice: "削除しました。"
        end

end

stallsのindex.html.erb

<% @page_title = "搾乳牛管理" %>
<h1><%= @page_title %></h1>




  <table class = "index" align = "left">
  
     <% @stall.each do |cell|%>
     <% if cell.cow.present? %>
     <tr><td><%= link_to cell.cow.number, cell.cow %></td></tr>
     <% else %>
     <tr><td><%= link_to "新規作成",new_stall_path(stall_id: cell.id) %></td></tr>
     <% end %>
     <% end %>
     
   
  </table>
 

cowのshow.html.erb

<% @page_title = "詳細" %>

<h1><%= @page_title %></h1>

<td><%= link_to "編集", [:edit, @cow] %>/
    <%= link_to "削除",  @stall, method: :delete %></td>
<td><%= link_to "一覧へ戻る", :stalls%></td>

<table class = "attr">
  <tr>
    <th>個体識別番号</th>
    <th>名号</th>
    <th>生年月日</th>
    <th>状態</th>
  </tr>
  <tr>
    <td><%= @cow.number %></td>
    <td><%= @cow.name %></td>
    <td><%= @cow.birthday %></td>
    <td><%= @cow.status == 1 ? "経産牛" : "未経産牛" %></td>
  </tr>
</table>

①stallの一覧（indexアクション）から新規作成
↓
②stallcontrollerからcowcontrollerのnewアクションを使って新規作成のフォームを表示
↓
③フォームに内容を書いて送信
↓
④フォームの内容が反映された詳細ページ（showアクション）にリダイレクトされる

の流れで新しい牛の登録をするつもりで上のコントローラを書いたのですが、

②で表示される画面（フォームの内容を書いた後です）

②の内容をCreate Cowボタンで送信した後の画面

フォームの内容が書かれた状態のままで、URLだけが変更されてしまっています。

Answer 1

牛をどう同定するのか、がごちゃごちゃになっています。
migrationを見ると　id, number, line がそれに当たりそうですが、内lineは画面上どこに表示されるか、で牛固有のものでは無いですからその目的には使わない方が良いです。例えば100頭になって、１画面２０頭ずつ表示、なんてときに困ります。
number で管理するとすると <% @cow = @maincow.find_by line: i %>とviewの中でDBから取ってくるのではなく
controller index にて@maincow = MainCow.order("number")しているのを使います

    <% 20.downto(1).to_a.each do |i|%>
    <% @cow = @maincow.find_by line: i %>

を

    <% @maincow.each do |cow|%>
     <tr><td><%= link_to cow.number, cow %></td></tr>
     <% end %>
     <tr><td><%= link_to "新規作成",new_main_cow_path %></td>

但しこうすると 最初は１頭も居ないので、上の２０行は空で最後の新規作成だけになりますが

Comments

[ayumu0622]

ご回答ありがとうございます。
ご指摘の通りに書き換えてみましたが私の考えていたようにはなりませんでした…

私の説明不足で申し訳ないのですが
①一覧の表示は質問に載せた画像のように2列×20行のような表示になる。
②セル内のデータは番号順ではなく、任意のセルに任意のデータが入るようにしたい。
③データが入っていないセルにはnewアクションに飛ぶリンクをつけたい。
の3つができるようにしたいです。

私は酪農をやっていて、私の牧場では質問の最初の画像のセルのよう（2×20）な並びで牛をつないでいます。
牛は番号順につないでいる訳ではなく、また牛をつないでいない箇所もあります。
実際の牧場内の配置とアプリ上の牛のデータの配置を同じようにしたいです。

また質問でご指摘いただいた牛が100頭になったときの場合についてですが、自分の牧場の配置と同じようにしたいので、40頭以上にはならないです。
あくまで最初の画像のような表示で、１ページで完結した一覧表示を作りたいと考えています。

わかりづらくてすいません…
何かいい方法がありましたらご教授頂ければ幸いです。

[winterboum]

と言うことですと、Cowで管理するのではなく、牛舎というか20x2の小部屋を管理する方がわかりやすいですね。
あと元の図では20x2の小部屋ではなく２０ｘ１の小部屋に番号と名前の2列になっています。
MainCowという名前はちょっと、、、ですがとりあえずそれを小部屋として、４０でもう変わらないのなら、db/seed にて初期登録しておきましょう。
牛の管理と部屋の管理と両方やることになるでしょうから（体重の変化とか搾乳量とか管理したくなりますよね）MainCowは
1)部屋の　番号と必要なら名前
2)そこにつなぐ牛のID
を入れるようにします。
で、Cowというモデルをそこに関連つける。
そんな方向で作り変えてみてはどうでしょう。
あと、２０ｘ２で固定なら、一番最後の 新規登録 のtrは不要では？

[winterboum]

その方向だと、
indexの `@maincow = MainCow.order("number")`　を　`@maincow = MainCow.includes(:cow).order("number")`に変え
view の　indexの
```

<% 20.downto(1).to_a.each do |i|%>
<% @cow = @maincow.find_by line: i %>
<% if @cow.present? %>
<tr><td><%= link_to @cow.number, @cow %></td></tr>
<% else %>
<tr><td><%= link_to "新規作成",new_main_cow_path %></td></tr>
<% end %>
<% end %>
```
は
```
<% @maincows.each do | cell | %>
<% if cell.cow.present? %>
<tr><td><%= link_to cell.cow.number, cell.cow %></td></tr>
<% else %>
<tr><td><%= link_to "新規作成",new_main_cow_path(maincow_id: cell,id)%></td></tr>
<% end %>
<% end %>
```
で、CowsControllerのnewで受けることにします。
あ、
それは新しい牛が来た時か。
MainCowsControllerのnewか、updateとかinsert_cowとかですね。

[ayumu0622]

1、db/seedに初期登録、というのはシードデータで40頭分の適当なデータを投入してしまう、ということでしょうか？
一応最初の画像の一覧にはシードデータで下のようなデータは入っています。

MainCow.create(number: 323,name: "ヒロタ　ニッコリ　ホルスタイン",birthday: "2015-3-9", line: 1)
MainCow.create(number: 689,name: "ヒロタ　プリプリ　ブリッコ",birthday: "2014-8-9", line: 2)
MainCow.create(number: 2628,name: "ヒロタ　ラフロイグ　ラガヴーリン",birthday: "2016-6-22",line: 3)
MainCow.create(number: 1486,name: "ヒロタ　ブラントン　フォアローゼス",birthday: "2015-7-9", line: 4)
．．．．

2、今あるMainCowでは
①牛をつなぐ場所（winterboum様のおっしゃる部屋）
②つなぐ牛のid
を管理し、
新しく作ったCowモデルで牛の個体番号、生年月日を管理するよう設定し、MainCowとCowで関連付けを行う
のような解釈でよろしいでしょうか？
理解力乏しくてすいません．．．

（補足、というわけではないのですが、当初は搾乳している牛をMainCow、まだ出産していない牛をYoungCowで分けて管理しようとしていたためMainCowという名前を付けました。）

[winterboum]

今日明日ちょっと気ぜわしいのでとりあえず、ですが
初期登録は牛ではなくつなぐ場所です。
ついでに牛もやるのもありですが
なおすならmainCowを止めてつなぎ場所らしい名前にしましょうよ
搾乳できるか否かはCowの属性でよいですね

[ayumu0622]

とりあえず
①つなぐ場所を管理するStallscontroller,Stallmodel（牛舎で牛をつなぐ場所のことをストール、と呼ぶため）を作成
def change
create_table :stalls do |t|
t.integer :stallnumber, null: false
t.integer :（下記の理由のためまだ未記入）

t.timestamps
end

②牛を管理するためのCowscontroller、Cowmodelを作成
def change
create_table :cows do |t|
t.integer :number, null: false
t.string :name, null: false
t.date :birthday, null: false
t.integer :status,null: false, default: 1

t.timestamps
end

statusカラムで搾乳牛か若牛かの区別をする予定。

この二つを関連付けるにあたって、どちらにhas_oneを書きどちらにbelong_toを書くべきか迷ったのでとりあえずここまで書いて作業を止めてあります．．．
当初の予定では部屋を管理する方（現在のStall）に牛のidをいれて管理する、とのことだったのでStallmodelにbelong_to :cow、Cowmodelにhas_one :stall
になるんでしょうか

[winterboum]

自然言語感覚的では StallにCawをつなぐ（割り当てる）という感じですので、
Stall が　has_one :cow かなと思いますが、　belongs_to も無いわけではない。

ただ、rails的には、　belongns_to にするとdefaultではCowが繋がっていないをvalidationエラーを起こします（空の時もあるよ、という宣言が必要）

ということを考慮すると、Rails語的には 相手が必ず必要な牛の方をbelongs_to にするのかな、と思います

[ayumu0622]

class StallsController < ApplicationController
def index
@stall = Stall.includes(:cow).order("stallnumber")
end

def show
@stall = Stall.includes(:cow).find(params[:id])
end

def new
@cow = Cow.new(birthday: Date.new(2010,1,1))
render template:'cows/new'
end

def edit
@stall = Stall.includes(:cow).find(params[:id])
end

def create
@cow = Cow.new(params[:cow])
if @cow.save
redirect_to @cow, notice: "登録しました。"
render template:'cows/show'
else
render "new"
end
end

def update
@stall = Stall.includes(:cow).find(params[:id])
@stall.assign_attributes(params[:cow])
if @stall.save
redirect_to @stall, notice: "更新しました"
else
render "edit"
end
end

def destroy
@stall = Stall.find(params[:id])
@stall.destroy
redirect_to :stall, notice: "削除しました。"
end
end

class CowsController < ApplicationController
def index
@stall = Stall.includes(:cow).order("number")
end

def show
@cow = Cow.find(params[:id])
end

def new
@cow = Cow.new(birthday: Date.new(2010,1,1))
end

def edit
@cow = Cow.find(params[:id])
end

def create
@cow = Cow.new(params[:cow])
if @cow.save
redirect_to @cow, notice: "登録しました。"
else
render "new"
end
end

def update
@cow = Cow.find(params[:id])
@cow.assign_attributes(params[:cow])
if @cow.save
redirect_to @cow, notice: "更新しました"
else
render "edit"
end
end

def destroy
@cow = Cow.find(params[:id])
@cow.destroy
redirect_to :cow, notice: "削除しました。"
end

end

関連付けは概ねうまくいき、edit、updateはうまく動くのですがnewアクションでフォームに内容を記述した後、フォームの内容を送信してもshowアクションにリダイレクトされません．．．

[winterboum]

＞newアクションでフォームに内容を記述した後、フォームの内容を送信してもshowアクションにリダイレクトされません．
が良くわかりません
（どのコントローラの？）newアクションでnewのviewを表示し
そこに入力した後フォームを（どのコントローラのどのアクションに？）送信し
成功したら（どのモデルの）showに表示させたいが、（どうなった？）

[winterboum]

あとcodeはコメント欄だと読みにくいので、質問を編集して載せていただけますか

[ayumu0622]

質問の内容がわかりづらくて申し訳ありませんでした。
質問本文を編集して改めて質問させていただきましたのでお時間あるときにご回答頂きたいです。

[ayumu0622]

最初の質問から少しそれた内容になってしまったので改めて別の質問記事を立てさせていただきます。
winterboum様ありがとうございました。

[winterboum]

はい。