[Rails6.0]ループ時のlabelの表記が変わらない

Question

一つのフォームで複数のモデルのレコードを更新できるように実装したいのですが、labelの表記がうまく反映されません。
time_tablesテーブルのレコードとartistsテーブルのレコードを同時に保存したいため、モデルとコントローラーは以下のコードにしています。

class TimeTable < ApplicationRecord
  has_many :schedules # 複数のScheduleを持っている(中間テーブル)
  has_many :artists, through: :schedules # 複数のArtistを持っていて、中間モデルにSchedulesがある
  belongs_to :user # Userモデルに属している

  validates_associated :artists # Artistモデルのバリデーションを実行
  accepts_nested_attributes_for :artists, allow_destroy: true # Artistモデルを更新、削除刷ることができる
end

def new
    @time_table = TimeTable.new
    10.times do
      @time_table.artists.build
    end
  end

一括登録の仕方がわからなかったのでtimes.doを使い、artistテーブルのモデルを１０回作成しようとしています。
erbファイルは以下のように記載しているのですが、ループ文が使われているため、フォーム欄は複数できるのですが、ラベルの表記が全部同じになってしまいます。

# 抜粋
     <div class="form-row">
       <%= f.fields_for :artists do |a| %>
        <div class='form-group'>
          <%= a.label :name, '10:00' %>
          <%= a.text_field :name, class:'form-control' %>
        </div>
      <% end %>

ラベルだけループさせずに、それぞれのフォームに合わせたラベルを作成したいのですが、どのように実装したらよいかわからないので、教えていただきたいです。

Comments

[asm]

> それぞれのフォームに合わせたラベル
とは何を期待しており、現状はどのようになっていますか？

[ta_ka]

https://i.gyazo.com/cbc099a650a2ef5c0b4286b4c06b0230.png

このようになっており、現状、全部同じ時間になってるのですが、これを１時間ごとのラベルにしたいです。

Answer 1

Viewはとりあえずできたようですが、これですとどの時刻にどのArtistなのか、がわかる様にsaveできますか？
TimeTableとArtistが多対多ということは、あるArtistが他の(日の)TimeTableに登録されることもある、ということかと思います。
viewに乗せるのは artists ではなく schedules ではないのでしょうか？

その場合、

(0..9).each do | index |
   @time_table.schedules.build(stage_on_at: asmさんの式)
end

としておいて
<%= a.label :name, a.object.stage_on_at %>
で行けるかと
Scheduleのcolumn名わからないので適当に書きました。
あ、「asmさんの式」そのままではだめか
"#{10 + index / 2}:#{index % 2 ? '00' : '30'}"
ですね。

Comments

[ta_ka]

ご指摘ありがとうございます。
仰る通り、schedulesテーブルに保存にどのArtistが保存されているかという実装です。

```ruby
class TimeTablesController < ApplicationController
def new
@time_table = TimeTable.new
(0..9).each do |index|
@time_table.schedules.build(appearance_time: "#{10 + index / 2}:#{index % 2 ? '00' : '30'}")
end
end
```

TimeTablesコントローラーにてこのようにし、TimeTableモデルは、

```ruby
class TimeTable < ApplicationRecord
has_many :schedules # 複数のScheduleを持っている(中間テーブル)
has_many :artists, through: :schedules # 複数のArtistを持っていて、中間モデルにSchedulesがある
belongs_to :user # Userモデルに属している

validates_associated :artists # Artistモデルのバリデーションを実行
accepts_nested_attributes_for :artists, allow_destroy: true # Artistモデルを更新、削除刷ることができる
accepts_nested_attributes_for :schedules, allow_destroy: true # Schdulesモデルを更新、削除刷ることができる
end
```

viewでは

```ruby
<div class="form-row">
<%= f.fields_for :artists do |a| %>
<div class='form-group'>
<%= a.label :name, a.object.appearance_time %>
<%= a.text_field :name, class:'form-control' %>
</div>
<% end %> <%# fields_for %>
</div>

```
このように書きました。
しかし、viewにアーティストを入力するフォームが反映されなくなってしまいました。
field_for の引数が間違っているのでしょうか。

[winterboum]

field_for は artistsではなくschedules ですね。
それと入力fieldのnameはScheduleに合わせないと。
Scheduleにはartist_id でnameでは無いと思うので、そこをどうやるか、はひと工夫ですね。

[ta_ka]

この場合、通常の多対多の受け渡しでレコードの登録はできるものなのでしょうか？
まだ技術不足なところで、実装方法が見えてきていないので、どのように実装したらよいか教えていただけたら幸いです。申し訳ございませんが、よろしくおねがいします。

[winterboum]

create と なんとかparamsのcodeを

[ta_ka]

いくつか方法を試してみたのですが、現状、
```ruby
(0..9).each do | index |
@time_table.schedules.build(appearance_time: "#{10 + index / 2}:#{index % 2 ? '00' : '30'}")
end
```
で、time_tablesテーブルに紐づくschesulesテーブルのカラムを保存しようとする。
schesulesテーブルは中間テーブルなので、artist_idが無いと保存できない。
フォームのappearance_time(出演時間カラム)として値を受け取っているため、Artistを保存することができない。
このような問題に詰まり、うまく実装ができませんでした。
rails c では@time_table.artist_idsにArtistのインスタンスを代入して保存することはできましたが、ローカルでの実装方法がわからないので教えていただけるとありがたいです。
何度もすみませんが、よろしくお願いします。

[winterboum]

こういう場合2つのcaseが有ります。
Artistのデータが揃っていて、そこから選択する。載っていない人は登録出来ない
Artistのデータにない人も登録できるようにする。その場合はArtistの登録も同時に行う
前者の方が格段に楽ですが、そうしますか？

[ta_ka]

前者のArtistデータが揃っている場合に登録できる実装をしてみたいです。

[winterboum]

では、<%= a.text_field :name, class:'form-control' %> の所を
Artistのデータをつかって、
https://api.rubyonrails.org/classes/ActionView/Helpers/FormBuilder.html#method-i-collection_select
にて実装しましょう。

[ta_ka]

ありがとうございます。
一応テストデータとしてArtistのデータから保存することはできました。
ただ、

```ruby
<%= a.collection_select :artist_id, @artists, :id, :name, { prompt: true } %>
```
としているためにappearance_time(出演時間カラム)にいつ出演するかのデータが登録されません。
引数に複数のモデルを渡すことはできそうにないのでしょうか…？

[winterboum]

hidden field で appearance_time を載せて渡しましょう

[ta_ka]

ありがとうございます！！
なんとか実装することができました！！☺

ちなみになんですが、
> Artistのデータにない人も登録できるようにする。その場合はArtistの登録も同時に行う

こちらの方法で実装しようとしたらどういった手順で実装するべきなのでしょうか。
いまいちこちらの実装方法がわからないので参考までに教えていただきたいです。

[winterboum]

これは私も模範解答はないです

[ta_ka]

承知いたしました。
長々とご教授いただきありがとうございました！

まだまだ技術不足なので、理解を深めていけるように頑張っていきたいと思います！

Answer 2

fields_forを見る限り

          <%= a.label :name, "#{10 + a.index}:00" %>

にて対処できそうですが、いかがでしょうか？

Comments

[ta_ka]

<%= a.label :name, "#{10 + a.index}:00" %>

これで実装することができました！！(ドキュメントをよく読んでいませんでした…)
ちなみになんですが、これを３０分ごとに区切る場合(例えば10:00→10:30→11:00など)もできますでしょうか？

[asm]

"#{10 + a.index / 2}:#{a.index % 2 ? '00' : '30'}"
ですかね

[ta_ka]

"#{10 + a.index / 2}:#{a.index % 2 == 0 ? '00' : '30'}"

このコードで実装することができました！
きちんとドキュメント読むことの大切さがわかりました、ありがとうございますm(_ _)m