レコードがすでに存在する場合は、作成せずに更新したい

Question

作家、作品、フレーズを投稿するフォームがあり、ユーザーがフォームにそれらを投稿する際、
すでにその作家が存在する場合、
・その作家を新規作成せずに、その作品だけを新規作成。
・このとき、もし作品も存在していたら、作品を新規作成せずに、そのフレーズのみを新規作成。

以上のことがしたいのですが、
現状、
①既存の作家に新規の作品をひもづける。
②既存の作家の、既存の作品に新規のフレーズをひもづける。
というところまではできているのですが、
②をする際に、新規に、既存の名前と同じ作品が登録されてしまいます。
これを解決するにはどうしたらよいでしょうか？

以下に、該当する箇所のコードを記載いたします。

class AuthorsController < ApplicationController
  def new
    @author = Author.new
    @writing = @author.writings.build
    @word = @writing.words.build(author_id: @author.id)
  end

  def create
    @author = Author.find_or_initialize_by(name: params[:author][:name])
    @author.update_attributes(author_params)
  end

  private

    def author_params
      params.require(:author).permit(:id, :name,
      writings_attributes: [:id, :title,
        words_attributes: [:id, :content, :author_id]])
    end

作家、作品、フレーズのモデル

class Author < ActiveRecord::Base
  has_many :writings
  has_many :words, through: :writings
  accepts_nested_attributes_for :writings


class Writing < ActiveRecord::Base
  belongs_to :author, :autosave => true
  has_many :words
  accepts_nested_attributes_for :words

  def autosave_associated_records_for_author
     if new_author = Author.find_by(name: author.name)
       self.author = new_author
     else
       self.author.save!
     end
  end


class Word < ActiveRecord::Base
  belongs_to :writing, :autosave => true
  has_one :author, through: :writing

  def autosave_associated_records_for_writing
     if new_writing = Author.find_by(name: author.name).writings.find_by(title: writing.title)
       self.writing = new_writing
     else
       self.writing.save!
     end
  end
end

よろしくお願いいたします。

Answer 1

参考情報：

• find_or_create_by http://railsdoc.com/references/find_or_create_by

...
検索条件を指定して初めの1件を取得し、1件もなければ作成(find_or_create_by)
...

• seeds.rbの使い方とマスタデータの取り扱いついて考える http://totutotu.hatenablog.com/entry/2015/09/21/seeds.rb%E3%81%AE%E4%BD%BF%E3%81%84%E6%96%B9%E3%81%A8%E3%83%9E%E3%82%B9%E3%82%BF%E3%83%87%E3%83%BC%E3%82%BF%E3%81%AE%E5%8F%96%E3%82%8A%E6%89%B1%E3%81%84%E3%81%A4%E3%81%84%E3%81%A6%E8%80%83%E3%81%88

...
なるほどidの値も含めてfirst_or_createを呼べばいいのか。
ってこれならfirst_or_createじゃなくてfind_or_createでも一緒では、って思ったら、どうやらfind_or_createはdeprecateになるらしい。
...

• いいねと思った Rails メソッド 10http://qiita.com/pekepek/items/c57d4859e6e1ec8b1ad1

Answer 2

1. uniqeキーを設定する
2. first_or_create メソッドで取得する

詳細はググってくださいませ。

Comments

[jamsyrup]

uniqueキーを設定した場合、
たとえば、すでに
author：「川端康成」、writing：「雪国」が登録されていて、
「川端康成」の「雪国」の別のフレーズを登録しようとしたとき、
not uniqueになります。
このとき、そのフレーズを、すでに存在している「雪国」にひもづけて、新規のレコードを作成しなければ、not uniqueにはならないと思うのですが、それをするにはどうすればよいでしょうか？
よろしくお願いします。
また、find_or_createについては、authorのレコードについては適用していますが、accepts_nested_attributes_forのwritingにはどのように適用できますでしょうか？
重ねてお願いします。

[sokha]

回答が遅くなり申し訳ありません。
なるほど、ちょっと複雑ですね。

同じ要件を果たすために、以下のようなテーブル構造の方がいいかと思いました。

```
---
id, author_name, writings, words
---
find_or_create( author_name, writings, words)
```

もしくは

```
- authors
id , name
- writings
id, name
- words
id, authors_id, writings_id, word

find_or_create( aurhors_id, wrtings_id, word )
```