Railsで多対多のコントローラの作り方が分からない

Question

前提

以下のような多対多のテーブルを有識者の方のアドバイスをもとに作成しました。

しかし、私の知識が浅く、コントローラの作り方が分からなくなってしまいました。

フレームワークは Rails 6 系、DBは PostgreSQL 13 系です。

モデルの状態

# app/models/song.rb
class Song < ApplicationRecord
  has_many :composer_songs, dependent: :destroy
  has_many :composer_song_artists, through: :composer_songs

  has_many :lyricist_songs, dependent: :destroy
  has_many :lyricist_song_artists, through: :lyricist_songs

  has_many :arranger_songs, dependent: :destroy
  has_many :arranger_song_artists, through: :arranger_songs
end

# app/models/artist.rb
class Artist < ApplicationRecord
  has_many :composer_songs, dependent: :destroy
  has_many :composer_song_songs, through: :composer_songs

  has_many :lyricist_songs, dependent: :destroy
  has_many :lyricist_song_songs, through: :lyricist_songs

  has_many :arranger_songs, dependent: :destroy
  has_many :arranger_songs_song, through: :arranger_songs
end

# app/models/lyricist_song.rb
class LyricistSong < ApplicationRecord
  belongs_to :lyricist, class_name: "Artist", foreign_key: "artist_id"
  belongs_to :song
end

# app/models/composer_song.rb
class ComposerSong < ApplicationRecord
  belongs_to :composer, class_name: "Artist", foreign_key: "artist_id"
  belongs_to :song
end

# app/models/arranger_song.rb
class ArrangerSong < ApplicationRecord
  belongs_to :arranger, class_name: "Artist", foreign_key: "artist_id"
  belongs_to :song
end

スキーマの状態

# db/schema.b

ActiveRecord::Schema.define(version: 2021_09_29_115047) do

  # These are extensions that must be enabled in order to support this database
  enable_extension "plpgsql"

  create_table "arranger_songs", force: :cascade do |t|
    t.bigint "arranger_id"
    t.bigint "song_id"
    t.index ["arranger_id"], name: "index_arranger_songs_on_arranger_id"
    t.index ["song_id"], name: "index_arranger_songs_on_song_id"
  end

  create_table "artists", force: :cascade do |t|
    t.string "name"
    t.datetime "created_at", precision: 6, null: false
    t.datetime "updated_at", precision: 6, null: false
  end

  create_table "composer_songs", force: :cascade do |t|
    t.bigint "composer_id"
    t.bigint "song_id"
    t.index ["composer_id"], name: "index_composer_songs_on_composer_id"
    t.index ["song_id"], name: "index_composer_songs_on_song_id"
  end

  create_table "lyricist_songs", force: :cascade do |t|
    t.bigint "lyricist_id"
    t.bigint "song_id"
    t.index ["lyricist_id"], name: "index_lyricist_songs_on_lyricist_id"
    t.index ["song_id"], name: "index_lyricist_songs_on_song_id"
  end

  create_table "songs", force: :cascade do |t|
    t.string "name"
    t.datetime "created_at", precision: 6, null: false
    t.datetime "updated_at", precision: 6, null: false
  end

  add_foreign_key "arranger_songs", "artists", column: "arranger_id"
  add_foreign_key "arranger_songs", "songs"
  add_foreign_key "composer_songs", "artists", column: "composer_id"
  add_foreign_key "composer_songs", "songs"
  add_foreign_key "lyricist_songs", "artists", column: "lyricist_id"
  add_foreign_key "lyricist_songs", "songs"
end

試したこと

以下のような操作はちゃんとできました。

Song.create(name: "test1")
Song.create(name: "test2")
Artist.create(name: "a1")
Artist.create(name: "a2")

中間テーブルを多分ちゃんと覗けています。。。
今はまだ中間テーブルは空です。

irb(main):012:0> Song.first.composer_songs
  Song Load (0.6ms)  SELECT "songs".* FROM "songs" ORDER BY "songs"."id" ASC LIMIT $1  [["LIMIT", 1]]
  ComposerSong Load (0.7ms)  SELECT "composer_songs".* FROM "composer_songs" WHERE "composer_songs"."song_id" = $1  [["song_id", 1]]
=> []

実現したいこと

以下のような状態を実現できるコントローラーを記述したいです。

しかし、どのようにしてSong.create(name: "test1")のような
createやupdateの処理を記述したらいいのかわかりません。

私のような多対多のテーブルにおける、コントローラ(CRUD)の記述の仕方を教えていただきたいです。
よろしくお願いします。

Comments

[shinoharat]

画面的には「アーティストを登録する画面」と「曲名を入力し、作詞者・作曲者・編曲者を選んで楽曲を登録する画面」の2種類になるってイメージでいいんですかね？
もしそうなら、Song モデルに「accepts_nested_attributes_for」を足した上で、

Song.create(
name: "test1",
composer_songs_attributes: { ... },
lyricist_songs_attributes: { ... },
arranger_songs_attributes: { ... },
)

みたいに書けば Song を登録するついでに関連付けも一緒に行えます。
実際のコントローラーでは permit とかも必要です。
これで方向性が合っていれば、もう少し詳しい回答を書きます。

--

それから、質問ではコントローラーの書き方に限定されていますが、 view の方は問題ないですか？
「fields_for」とか「_destroy」とか、ご存知なければそちらも解説しますが。

[yosse95ai]

返信が遅れました。

> 画面的には「アーティストを登録する画面」と「曲名を入力し、作詞者・作曲者・編曲者を選んで楽曲を登録する画面」の2種類になるってイメージでいいんですかね？
その認識で間違いありません。

> それから、質問ではコントローラーの書き方に限定されていますが、 view の方は問題ないですか？
APIモードで作成しているので、いまのところviewはなくても大丈夫かなと思っています。

Answer 1

多対多の中間テーブルの作成はcontrollerを作って行うというよりは、、、、

例えば
test1 を a1 の composer_songs にいれるには
a1.composer_song_songs << test1
です。
a2 の composer_songs を test1, test2 に置き換えるなら
a2.composer_songs = [test1, test2]
です。

追記

関連定義に違和感があるのです
例えば has_many :composer_song_songs
これは CompoerSongSong というmodelをhas_manyしてるというもの。

ある Song を　ComposerSong として Artistに関連付けるなら
中間tableは Artist と ComposerSong をつなぐものとして、 ArtistComposerSong で、
class Artist has_many :composer_songs, class_name: "Song", through: :artist_composer_songs
とするのがよいのでは。

Comments

[yosse95ai]

回答ありがとうございます。
以下のように行ってみましたが、なぜかうまくいきませんでした…

```
a1 = Artist.first
test1 = Song.first
a1.composer_song_songs << test1
```
すると以下のようになりました…

```
/home/user/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/activerecord-6.1.4.1/lib/active_record/reflection.rb:933:in `check_validity!': Could not find the source association(s) "composer_song_song" or :composer_song_songs in model ComposerSong. Try 'has_many :composer_song_songs, :through => :composer_songs, :source => <name>'. Is it one of composer or song? (ActiveRecord::HasManyThroughSourceAssociationNotFoundError)
```

このエラーについて調べてみたのですが、いまいち解決方法が見つかりませんでした。
そのため。返信が遅れて申し訳ありません。

[yosse95ai]

追記ありがとうございます。
命名規則について無知なところがあるため、見直していきたいと思いました。