Rails 中的关注点分离困境

Question

我正在尝试为我的 Rails 应用程序进行日志记录，但对 Rails 中使用的哲学存在一些困境。我的应用程序有 Link 模型，其中 has_many Hits：

class Link < AR::Base
  has_many :hits
end

class Hit < AR::Base
  belongs_to :link
end

现在每次点击链接时，我都会调用 hit! 在链接上记录请求的方法（为了保持控制器瘦身，我使模型变胖）：

class LinksController < ApplicationController
  def hit
    link = Link.find(params[:id])
    link.hit!(request)
  end
end

class Link < AR::Base
  def hit!(request)
    params = extract_data_from_request(request)
    hits.create(params)
  end
end

现在这就是我感到困惑的地方。我想记录 request 对象附带的数据（如远程 IP、引用者、用户代理等），因此我需要将请求对象传递给模型，但我认为这不符合到“关注点分离”并模糊了 MVC 设计模式中的责任线（当然，如果我错了，请纠正我）。另外，如果我在控制器本身中创建一个 Hit 对象，那么我将制作瘦模型和胖控制器：

class LinksController < ApplicationController
  def hit
    hit_params = extract_data_from_request(request)
    Hit.create(hit_params.merge(:link_id => params[:id])
  end
end

尽管后一种情况使测试变得更容易（我不需要在模型中模拟请求）规格）-这似乎不太正确。

对此有任何建议 - 非常感谢。

PS extract_data_from_request(req) 方法放置在需要的适当位置。它返回 Hit 对象所需属性的哈希值。

Answer 1

就我个人而言，我会警惕过度思考这些事情。

命中的概念与网站或 Web 应用程序密切相关，（HTTP）请求的概念也是如此。胖控制器反模式更多地涉及包含 ActiveRecord 查找语句和业务逻辑的冗长控制器操作（通常以 if/elsif/else 块）可以轻松提取到模型中。

控制器具有一定的编排职责。在一个人体内创造一个物体并不是一种令人发指的罪行。毕竟，我们一直在 create 操作中执行此操作。

Answer 2

是的，我同意约翰的观点。请求的概念通常是“控制器事物”，但在这种情况下，您的模型是对请求进行建模，因此在这种情况下它绝对属于模型领域。实际上，一旦请求对象跨越了从控制器到模型的边界，它就只是另一个对象，没有特殊的属性：它不再关心获取和响应 html 请求的过程，它只是一个你可以做任何你想做的事情的对象。

不过，需要警惕的一件事是，在 ruby​​ 中，参数是通过引用传递的。这意味着您在模型中操作的请求对象与控制器中处理的请求对象是同一对象。我可能过于偏执（或者完全错误），但您可能希望将其副本传递给模型而不是实际请求本身。 IE

class LinksController < ApplicationController
  def hit
    link = Link.find(params[:id])
    link.hit!(request.dup)
  end
end