1

我正在尝试为我的 rails 应用程序进行日志记录，并且对 rails 中使用的哲学存在一些困境。我的应用程序具有以下Link模型has_many Hit：

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

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

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

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

4

2 回答 2

3

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

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

控制器有一定的编排职责。在一个人体内创造一个物体并不是令人发指的罪行。毕竟，我们在create行动中一直在这样做。

于 2010-07-06T12:39:19.060 回答
2

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

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

class LinksController < ApplicationController
  def hit
    link = Link.find(params[:id])
    link.hit!(request.dup)
  end
end
于 2010-07-06T14:14:54.413 回答