更新关联时如何销毁其他 has_one 关系？

Question

我有一个 User，它 has_one Widget。

class User
  has_one :widget, :dependent => :destroy
end

class Widget
  belongs_to :user
end

当我为 User 创建一个新的 Widget 时，我想销毁与 User 关联的旧控件。

这是我的情况：

创建用户：

user = User.new
user.save
user # => #<User id: 1>

创建用户的小部件：

widget = Widget.new
widget.user = user
widget.save

重新加载并检查小部件：

user.reload
user.widget # => #<Widget id: 1, user_id: 1>

构建一个小部件，注意现有的一个小部件在保存另一个小部件之前被销毁：

user.build_widget # => #<Widget id: nil, user_id: 1>
user.reload
user.widget # => nil

重新创建用户的小部件：

user.create_widget # => #<Widget id: 2, user_id: 1>

创建另一个小部件：

widget = Widget.new :user => user
widget.save

现在，两者都存在：

Widget.find(2) # => #<Widget id: 2, user_id: 1>
Widget.find(3) # => #<Widget id: 3, user_id: 1>

用户的是第一个：

user.reload
user.widget # => #<Widget id: 2, user_id: 1>

有吗有什么方法可以做到这一点：

def create
  @widget = current_user.build_widget(params[:widget])

  respond_to do |format|
    if @widget.save
      format.html { redirect_to widget_path, notice: 'Widget was successfully created.' }
      format.json { render json: @widget, status: :created, location: @widget }
    else
      format.html { render action: 'new' }
      format.json { render json: @widget.errors, status: :unprocessable_entity }
    end
  end
end

在保存之前不删除旧的小部件，或者这样：

def create
  @widget = Widget.new(params[:widget])
  @widget.user = current_user

  respond_to do |format|
    if @widget.save
      format.html { redirect_to widget_path, notice: 'Widget was successfully created.' }
      format.json { render json: @widget, status: :created, location: @widget }
   else
      format.html { render action: 'new' }
      format.json { render json: @widget.errors, status: :unprocessable_entity }
    end
  end

结束

不保留两份副本？

我不想搞砸我的控制器有类似的交易

Widget.transaction do
  old_widget.destroy
  new_widget.save
end

，但到目前为止，这似乎是唯一的方法。

Answer 1

看起来您有两个使用路径，用户可以在其中创建小部件。从用户端和小部件端。如果您通过一段代码将它们集中起来，并进行一些唯一性验证以确保不会出现任何失误，那就更好了。

在 user.create_widget 中使用 find_or_create_by 怎么样，以便您可以在需要更新现有小部件或创建新小部件时对其进行编辑。

Answer 2

您应该更新用户现有的widget记录，而不是创建新记录并销毁旧记录。
你可以这样做：

class User
  has_one :widget, :dependent => :destroy

  def assign_widget(attr_hash)
    widget ? widget.update(attr_hash) : widget.create(attr_hash)
    widget.reload
  end
end