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

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

ただいまの
回答率

87.50%

【Rails】リレーションのあるマスターテーブルを同時に追加したい

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 880

score 20

下記Modelを定義していて

class Maker < ApplicationRecord
  has_many :notes, inverse_of: :maker
  validates :name, presence: true, uniqueness: true
end

class Tag < ApplicationRecord
  has_many :relations, class_name: 'TagRelation'
  has_many :notes, through: :relations, source: :note
  validates :name, presence: true, uniqueness: true
end

class TagRelation < ApplicationRecord
  belongs_to :note
  belongs_to :tag
end

class Note < ApplicationRecord
  belongs_to :maker
  accepts_nested_attributes_for :maker

  has_many :relations, class_name: 'TagRelation'
  has_many :tags, through: :relations, source: :tag
  accepts_nested_attributes_for :tags

  validates :name, presence: true, uniqueness: true
end

下記データが登録されている時に

@Maker Table     @Tag Table        @Note Table
 id | name         id | name        id | name  | maker_id
------------     -------------     -----------------------
  1 | maker1        1 | tag1         1 | note1 |        1
  2 | maker2        2 | tag2         2 | note2 |        2
                    3 | tag3

@TagRelation Table
 id | note_id | tag_id
-----------------------
  1 |       1 |      1
  2 |       2 |      2
  3 |       2 |      3

コントローラ上で(permit後の)下記パラメータでNoteにsaveしたい

params = {
  name: 'note3',
  maker_attributes: {
    name: 'maker1',
  },
  tags_attributes: [
    {name: 'tag1'},
    {name: 'tag4'},
  ],
}

発生するバリデーションエラー

{
  :"maker.name" => ["has already been taken"]
  :"tags[0].name" => ["has already been taken"]
}

具体的に実行したいこと

Note.new(params).save すると、上記のバリデーションエラーが発生します。
※ maker.name も tags[x].name も uniqueness: true なので当然ですが。。。

実施したいのは、Maker Tag の両テーブルに登録済のデータ(nameカラム)があれば、
レコードを増やす事なく、勝手にNoteからリレーションを貼ってもらいたいのです。

動作イメージとしては、下記のようになります。

note = Note.new(params)
# バリデーションに成功する
note.valid? # => true

# Makerは登録済のレコードなので、対象のレコードを持ってくる
note.maker # => Maker(id: 1, name: 'maker1')
# tags[0]は登録済のレコードなので対象を持ってくるが、 tags[1]は新規
note.tags # => [Tag(id: 1, name: 'tag1'), Tag(id: nil, name: 'tag4')]

note.save # => true

現状、考えられるのは Note の before_validation で各モデルに find_or_initialize するしかないかなと考えているのですが、
validates や accepts_nested_attributes_for に有効なオプションや、他に手法があれば教えていただきたいです!
よろしくおねがいします :bow:

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • ykp_yk

    2019/04/05 11:38

    申し訳ないのですがやっぱり何をしたいのか理解できませんんでした。
    validates :name, presence: true, uniqueness: true
    ユニーク制約というのはこのことですか?これを外せば良いんじゃないでしょうか。
    Noteモデルのnameはユニークじゃなく、同じものが入る可能性があるということだと思いますが、ご質問全体に主語と詳しいエラーが無いのでよくわからないです。

    キャンセル

  • aa37971

    2019/04/05 11:44

    > Noteモデルのnameはユニークじゃなく同じものが入る可能性があるということだと思いますが
    Note/Maker/Tag の name はすべてユニークにしたいです。

    質問のparamsを改めて見ていただきたいです。
    Noteに入れる name は別値です。

    Tagに限って話をしますが、
    初回のsaveでは tag0, tag1 をsaveします。
    → これはtags_attributesなので当然できますよね?

    次は tag1, tag2 の saveをしたいのですが、
    tag1もsaveしようとするので、ユニーク制約にかかりますよね?

    もちろん、NoteとTagを別々にsaveすればいいのですが、それはaccepts_nested_attributes_forを使うメリットがないですし、コントローラも複雑化してしまうのでRailsに機能があるはずと思って質問しています。

    キャンセル

  • ykp_yk

    2019/04/05 12:37

    > 質問のparamsを改めて見ていただきたいです。
    このparamsとご質問を見ただけでエラーもやりたいことも書いていないあなたのやりたいことが分かる人ってそうそういないと思いますよ。
    before_validationでユニークかどうかのチェックをやると良いです。
    https://apidock.com/rails/ActiveRecord/Callbacks/before_validation

    キャンセル

回答 1

checkベストアンサー

0

accepts_nested_attributes_for を使って一気に作るのを諦めるという方法はいかがでしょう

note = Note.new(params) もしくは create のあと

note.maker= Maker.find_or_initialize_by 
もしくは Maker.find_or_create_by

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 87.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る