在 has_many 中创建重复记录：通过关联

Question

我有三个模型：Booking、ExtraNight 和 BookedExtraNight。 Running Rails 2.3.11

Booking:

has_many :extra_nights, :through => :booked_extra_nights
has_many :booked_extra_nights, :dependent => :destroy

ExtraNight:

has_many :booked_extra_nights, :dependent => :destroy
has_many :bookings, :through => :booked_extra_nights

BookedExtraNight:

belongs_to  :extra_night
belongs_to  :booking

现在，由于应用程序的工作方式，在创建 booking_extra_night 时预订就存在。 Booking_extra_night 是通过“预订更新”操作创建的。 在视图中，booked_extra_night 是使用以下方式构建的：

<% unless @booking.booked_extra_nights.exists? %>
   <% @booking.booked_extra_nights.build %>
<% end %>

我使用nested_form_for @booking 和f.fields_for :booked_extra_nights 来创建booked_extra_nights。

现在一切正常，直到创建了 Booking_extra_night 的两个副本后按下提交（更新操作）。

查看日志：

Processing BookingsController#update (for 127.0.0.1 at 2011-02-21 07:44:22) [PUT]
Parameters: {"action"=>"update", "_method"=>"put",   "authenticity_token"=>"b/M+VjMxA8RFqbubhBeF494B/zhxi/2Eb3EtoCoRLx0=",  "id"=>"5b2jwg7qw5na3vz4nt", "booking"=>{"booked_extra_nights_attributes"=>{"0"=> {"number_of_days"=>"2", "from_date(1i)"=>"2011", "from_date(2i)"=>"9", "from_date(3i)"=>"1",  "_destroy"=>"", "extra_night_id"=>"7"}}}, "controller"=>"bookings"}
Booking Load (1.3ms)   SELECT * FROM "bookings" WHERE ("bookings"."random_url_key" =  '5b2jwg7qw5na3vz4nt') LIMIT 1
Variant Load (0.6ms)   SELECT * FROM "variants" WHERE ("variants"."id" = 27) 
SQL (0.1ms)   BEGIN
SQL (0.7ms)   INSERT INTO "booked_extra_nights" ("number_of_days", "created_at",   "updated_at", "booking_id", "from_date", "extra_night_id") VALUES(2, '2011-02-21  06:44:22.525154', '2011-02-21 06:44:22.525154', 69, '2011-09-01', 7) RETURNING "id"
SQL (0.8ms)   COMMIT
SQL (0.6ms)   BEGIN
SQL (0.6ms)   INSERT INTO "booked_extra_nights" ("number_of_days", "created_at",  "updated_at", "booking_id", "from_date", "extra_night_id") VALUES(2, '2011-02-21 06:44:22.544452', '2011-02-21 06:44:22.544452', 69, '2011-09-01', 7) RETURNING "id"
SQL (25.8ms)   COMMIT
SQL (0.1ms)   BEGIN
Booking Update (0.6ms)   UPDATE "bookings" SET "updated_at" = '2011-02-21 06:44:22.575409', "aasm_state" = 'step3' WHERE "id" = 69
SQL (0.5ms)   COMMIT
Redirected to http://localhost:3000/bookings/5b2jwg7qw5na3vz4nt/step3

正如您所看到的，创建了两条相同的记录，现在如果我要构建 4 个 Booking_extra_nights 并按提交，我最终会得到 8 条记录。

我还发现，如果我在创建预订的同时创建一个 booking_extra_night 记录，那么我可以添加任意数量的记录，而无需重复。 据我所知，这种情况发生在所有 2.3.x 版本的 Rails 上，所以这显然是我做错了。任何帮助将不胜感激，因为它正在我的头脑中。

谢谢！

Answer 1

我也遇到过这个问题，我的解决方案是在我的控制器中，不使用构建，而是使用 new 并提供父参数。

在我的代码中提取.. 而不是使用

@new_brand = @company.brands.build/new

我使用

@new_brand = Brand.new(:company => @company)

第一个将此对象作为空对象添加到@company，然后当您提交它时在您的内部创建另一个新对象创造行动。

第二个只是为了表单的目的在内存中创建一个新对象，但是当将值提交给创建操作时，没有与公司绑定的对象

Answer 2

谢谢，兔子！遗憾的是，你让我走上了正轨，但更新操作一团糟。
我已经重新考虑了它，现在一切正常了，我认为双重保存与我首先更新预订，然后重定向，然后通过 AASM 再次更新预订有关。

下面是旧的更新操作（我告诉过你这是一团糟）：

def update
  @booking = Booking.find_by_random_url_key(params[:id])
  @variant = @booking.variant
  if params[:back_button]
    if @booking.aasm_state == "step2"
      redirect_to booking_step1_url(@booking)
    elsif @booking.aasm_state == "step3"
      redirect_to booking_step2_url(@booking)
    elsif @booking.aasm_state == "step4"
      redirect_to booking_step3_url(@booking)
    elsif @booking.aasm_state == "step5"
      redirect_to booking_step4_url(@booking)
    end
    @booking.previous!
  else
    if @booking.update_attributes(params[:booking]) && @booking.aasm_state == "step1"
      redirect_to booking_step2_url(@booking)
      @booking.next!
    elsif @booking.update_attributes(params[:booking]) && @booking.aasm_state == "step2"
      @booking.next!
      redirect_to booking_step3_url(@booking)
    elsif @booking.update_attributes(params[:booking]) && @booking.aasm_state == "step3"
      redirect_to booking_step4_url(@booking)
      @booking.next!
    elsif @booking.update_attributes(params[:booking]) && @booking.aasm_state == "step4"
      redirect_to booking_url(@booking)
      @booking.next!
    end
  end
end

这是新的、重构的更新操作。

  def update
    @booking = Booking.find_by_random_url_key(params[:id])
    @variant = @booking.variant
    if params[:back_button]
      @booking.previous!
      redirect_to :controller => "bookings", :action => "#{@booking.aasm_state}", :id => @booking
    else
      @booking.update_attributes(params[:booking])
      @booking.next!
      redirect_to :controller => "bookings", :action => "#{@booking.aasm_state}", :id => @booking
    end
  end

功能相同，但少了很多行。而且它不会造成重复的额外住宿。

谢谢，兔子。你的回答让我思考，因为我曾多次尝试更改视图和模型中的内容但没有成功。我只是假设控制器可以工作，因为它可以用于其他所有事情。