1

我有一位同事喜欢将控制器传递给服务对象。例如，控制器方法可能如下所示：

class FooController < ApplicationController

  ...

  def show
    Foo.new(self).call
  end

  ...

end

服务对象如下所示：

class Foo

  attr_reader :controller, :resource_id

  delegate :render, :params, :head, to: :controller

  def initialize(controller, resource_id)
    @controller = controller
    @resource_id = resource_id
  end

  def call
    resource = SomeActiveRecordModel.find(resource_id)
    if resource
      render json: resource.to_json
    else
      head :not_found
    end
  end

end

不知何故，我觉得这适得其反，并且是货物崇拜软件工程的一个例子。

我更愿意将服务对象与控制器完全分开。依赖项将传递给服务对象的构造函数，参数将作为方法参数传递给服务对象。任何结果都只是从该方法返回。

可悲的是，每当我在代码审查中提到它时，我的同事们并没有对此感到兴奋，而我又觉得这相对令人沮丧。

各自方法的优缺点是什么？我怎样才能更好地为我的案子辩护？我在这里错过了什么吗？

4

1 回答 1

2

我怀疑答案是“视情况而定”。

在您给出的确切示例中，我没有看到特别的优势，并且会产生一定程度的混淆。另外，总的来说，我同意您将服务对象与控制器分开。

但是，有时我发现自己将控制器传递给服务对象。例如，当我在动态构建视图时有很多复杂的工作要做。

于 2016-04-15T14:42:43.840 回答