哪个 MVC 层应该处理声称玩具的孩子？

Question

我的应用中有两个模型：child 和 has_many :toys 和 toy belongs_to :child。我进行了为此工作所需的数据库迁移（将child_id 添加到toys 表中）。

起初，孩子独立存在，玩具独立存在（没有关联）。在幼儿园的每一天开始时，没有孩子拥有任何玩具。要玩玩具，孩子必须先认领它，然后成为它的主人。所以，现在我需要以某种方式实现一个child.claim(toy) 方法，在这里我被卡住了。具体来说：

1. 应该进入child 控制器或模型吗？或者也许应该以某种方式将两者分开？
2. 如果它应该进入控制器，它应该对应于 CRUD 操作之一还是它自己的东西，如 def claim(toy)？

编辑 1： 孩子是用户，通过浏览器登录。 （今天的孩子可以做一些了不起的事情）

Answer 1

我会构建一个单独的类来处理有关儿童和玩具的所有逻辑。称其为上下文，称其为关注，但要做到这一点。

class ToyChild # or ToyInteraction or ChildContext::Toys...

  attr_reader :toy, :child

  def initialize(toy, chid)
    @toy   = toy
    @child = child
  end

  def associate
    toy.child = child
    # could be more difficult: you should check if the child has not enough toys, if the toy is not already assigned to another child etc...
    #I'd avoid saving here since you may want to perform other operations
  end

  def foo
    #code here
  end

end

在控制器中：

assoc = ToyChild.new(toy, child).associate
assoc.save

这种编码风格：

• 更容易测试

• 职责分工明确

• 保持干燥（控制器中没有特定代码）

Answer 2

孩子是玩具的外键

当然还有其他方法可以做到这一点，但根据您最初的问题，最简单的解决方案是让您的行为与儿童与玩具表中的玩具相关联这一事实保持一致。

最简单的解决方案

抛开关于在完美的 OOP/MVC 设计中应该做什么的争论，做出改变的明智的地方是在控制器中，因为当孩子认领玩具时，该认领由 #update 处理（或者也许甚至是玩具控制器上的 #claim) 操作。

孩子是用户，通过浏览器登录。

在您的情况下，会话已经知道孩子是谁，因此将外键添加到 Toy 模型是微不足道的。控制器是接收关联模型参数的实体，因此它是告诉玩具模型为给定玩具更新哪些属性的正确位置。

当然还有更复杂的解决方案，但根据您原始帖子中提供的信息，它们的价值值得怀疑。与往常一样，您的里程可能会有所不同。

Answer 3

实际上，如果 child 是用户，则不需要声明方法。您可以在控制器中使用claim_toy 方法。在每个玩具的玩具索引视图中，您可以提供如下链接。

<%= link_to "claim", claim_toy_path(:toy_id => toy.id) %>

您的控制器方法将如下所示。

def claim_toy
  toy = Toy.find(params[:toy_id])
  current_child.toys << toy
end

足够简单。顺便说一句，这不是一个安宁的解决方案。