2

私はこのモデルの関係を持っています

class User < ActiveRecord::Base
  has_many :trip_memberships, dependent: :destroy
  has_many :trips, through: :trip_memberships, uniq: true
end

class Trip < ActiveRecord::Base
  has_many :trip_memberships, dependent: :destroy
  has_many :members, through: :trip_memberships, source: :user, uniq: true
end

class TripMembership < ActiveRecord::Base
  belongs_to :trip
  belongs_to :user
end

このコードのように、旅行メンバーとしてユーザーを追加すると、次のようになります。

trip = Trip.last
john = User.last
trip.members # => []  Empty right now
trip.members << john
trip.members # => [john]  Contains John
trip.members << john
trip.members # => [john]  Contains only John, but...
TripMembership.all # => [TripMembership(trip, john), TripMembership(trip, john)]
                   # There is 2 memberships, even the accessor methods only show one
                   # member because of the :uniq option

テーブルに重複を入れたくないのですtrip_membershipsが、「教えて、聞かないで」の原則に従いたいと思います。特定のレコードが存在するかどうかを確認したくありません。

この検証をTripMembershipモデルに追加しました

class TripMembership < ActiveRecord::Base
  belongs_to :trip
  belongs_to :user
  validates_presence_of :trip_id, :user_id
  validates_uniqueness_of :trip_id, scope: [:user_id]
  validates_uniqueness_of :user_id, scope: [:trip_id]
end

リレーションがたくさんあるように、保存できないレコードを追加しようとすると、 <<orメソッドがfalseを返すことを期待していましたが、リレーションではそのようには機能しないようです。concatthrough

trip.members << john # => ActiveRecord::RecordInvalid: Validation failed: Trip has already been taken, User has already been taken

誰もがこれをアーカイブするためのエレガントな方法を知っていますか？APIはこのように使うべきだと思います。

if @trip.members << new_member
   flash[:success] = "new member added"
else
   flash[:error] = "can't add this member to trip"
end
4

2 に答える 2

3

あなたはこのように試すことができます：

 if @trip.members.include?(new_member)
    flash[:error] = "Member already exists"
 else
    @trip.members << new_member
    flash[:success] = "new member added"
 end

編集

has_many :members, through: :trip_memberships, source: :user do
  def <<(member)
    if self.include?(member)
      false
    else
      super(Array(member)-self)
    end
  end
end
于 2013-02-26T14:29:54.267 に答える
1

私はちょうど考えを持っていました。

Railsの標準的な動作は、ユーザーが新しいレコードでない場合に例外を発生させることであり、よく知られたメソッドのデフォルトの動作を変更してはならないことを発見したので、カスタムとの関連付けを拡張しますbegin/rescue内でadd_memberエイリアス呼び出しを行うだけのメソッドが呼び出されました。<<

has_many :members, through: :trip_memberships, source: :user, uniq: true do
  def add(*records)
    self.<< records
  rescue ActiveRecord::RecordInvalid
    false
  end
end

Railsの方法を変更せずに期待どおりに機能する独自の方法があります。

于 2013-02-26T14:45:48.487 に答える