驚くほど複雑な ActiveRecord 関係

概要

ユーザーモデルがあります。ユーザーは多くの EmailAddresses を持っており、そのうちの 1 つを、primary_email_address として選択します。これが、私が電子メールを送信するアドレスです。ユーザーは常に少なくとも 1 つの電子メールアドレスを持っている必要があり、プライマリ電子メールアドレスを設定する必要があります。プライマリ電子メールアドレスは破棄できますが、その後、新しいプライマリ電子メールアドレスをユーザーに割り当てる必要があります。

これに対処するのは驚くほど難しい状況であることが判明しており、私が試した解決策にはどれも満足できない要素がありました。これは非常に一般的な問題のクラスのようです (A には多くの B があり、その B の 1 つは特別です)。それをきれいに解決する方法を知りたいと思っています。

何かのようなもの：

class User < ActiveRecord::Base
  has_many :email_addresses, inverse_of: :user

  validates :has_exactly_one_primary_email_address

  def primary_email_address
    email_addresses.where(is_primary:true).first
  end

  def has_exactly_one_primary_email_address
    # ...
  end
end

class EmailAddress < ActiveRecord::Base
  belongs_to :user, inverse_of: :email_addresses

  before_destroy :check_not_users_only_email_address
  after_destroy :reassign_user_primary_email_address_if_necessary

  # the logic for both these methods should live on the user but you get the idea
  def reassign_user_primary_email_address_if_necessary
    # ...
  end

  def check_not_users_only_email_address
    # ...
  end
end

ユーザーがプライマリ電子メールアドレスを 1 つだけ持つことが非常に重要であり、これを複数の電子メールアドレスレコードにわたって検証する必要があるのは良くないと思われるため、これは概念的に厄介です。また、ActiveRecord トランザクションは、ユーザーがプライマリ電子メールアドレスなしで行き詰まることのないように意味するものであることはわかっていますが、それは災難を招くレシピのように思えます。プライマリ電子メールアドレスは基本的にユーザーに属するものであり、このロジックを EmailAddress モデルに組み込むのは理想的ではありません。

何かのようなもの：

class User < ActiveRecord::Base
  has_many :email_addresses, inverse_of: :user
  belongs_to :primary_email_address

  validates_presence_of :primary_email_address
end

class EmailAddress < ActiveRecord::Base
  belongs_to :user, inverse_of: :email_addresses

  before_destroy :check_not_users_only_email_address
  after_destroy :reassign_user_primary_email_address_if_necessary

  # the logic for both these methods should live on the user but you get the idea
  def reassign_user_primary_email_address_if_necessary
    # ...
  end

  def check_not_users_only_email_address
    # ...
  end
end

これは、プライマリ電子メールアドレスを 1 つだけ持つユーザーの検証がはるかに簡単で、ユーザーモデルとより緊密に結合されているため、より優れています。しかし、現在、逆関数に関して厄介な問題が発生しています。 user.primary_email_address は、user.email_addresses 配列内の同じレコードと同じインスタンスを参照しないため、メモリ内のインスタンスに正しいデータがあることを確認するには、大量の再ロードが必要です。

> u = User.last
> u.email_addresses.map(&:email)
=> ["[email protected]", "[email protected]"]
> u.primary_email_address.destroy
=> true
> u.email_addresses.map(&:email)
=> ["[email protected]", "[email protected]"]
> u.reload
> u.email_addresses.map(&:email)
=> ["[email protected]"]

これにより、after_destroy フックやその他の状況で多くの問題が発生します。これは、User モデルの少し厄介なbelongs_to :primary_email_address 行が原因であるようです。 EmailAddresses と Users が、これら 2 つの異なる ActiveRecord 関係 (has_many :email_addresses/belongs_to :user およびbelongs_to :primary_email_address) によって関連付けられるのは少し奇妙です。

技術的には両方とも機能する 2 つのソリューション (現在 2 番目のソリューションを使用しています) ですが、どちらも直感的ではなく、時間がかかる欠陥があります。これを適切に解決する方法についての良いアイデアをぜひ聞きたいです。ありがとう。

解決策

多少異なりますが、最初のアプローチを使用することをお勧めします。電子メールアドレスでの変更が行われるため、その電子メールアドレスにトリガーコールバックを設定しないようにする方法は実際にはありません。ただし、以前のように複雑にする必要はありません。

ユーザーの EmailAddresses のコレクション全体に対してチェックを行う必要があるため、データベースの内容を使用して状態が有効かどうかを判断する必要があります。after_save と after_destroy を使用すると、これを簡単に行うことができます。

class EmailAdress < AR::Base
  belongs_to :user, :inverse_of :email_addresses

  scope :primary, where(:primary => true)

  after_save :ensure_single_primary_email
  after_destroy :ensure_primary_email_exists

  def ensure_single_primary_email
    user.verify_primary_email(self) if new_record? || primary_changed?
  end

  def ensure_primary_email_exists
    user.ensure_primary_email
  end
end

class User < AR::Base
  has_many :email_addresses, :inverse_of => :user, :dependent => :destroy
  attr_accessible :primary_email_address

  validates_presence_of :primary_email_address

  def primary_email_address
    if association(:email_addresses).loaded? 
      email_addresses.detect(&:primary?)
    else
      email_addresses.primary.first
    end
  end

  def primary_email_address=(email)
    email.primary = true
    email_addresses << email
  end

  def verify_primary_email(email)
    if email.primary? && email_addresses.primary.count > 1
      raise "Only one primary email can exist for a user"
    elsif !email.primary? && !email_addresses.primary.exists?
      raise "A user must have one primary email"
    end
  end

  def ensure_primary_email
    return if email_addresses.primary.exists?
    raise 'Missing primary email' if !email_addresses.exists?
    email_addresses.first.update_attribute(:primary, true)
  end
end