这太干了吗，我是不是太过分了？

Question

我一直在干燥一些代码，其中一个重构如下：
我有 3 个控制器（ConstructionCompanies、RealEstateCompanies、People），所有控制器都具有以下模式：

class ConstructionCompaniesController < ApplicationController
before_filter :correct_user, :only => [:edit, :update]

private
   def correct_user
      @company = ConstructionCompany.find(params[:id]) 
      if(current_user.owner != @company.user)
        redirect_to(root_path)
      end
    end   

class RealEstateCompaniesController < ApplicationController
  before_filter :correct_user, :only => [:edit, :update]
...

private
   def correct_user
      @company = RealEstateCompany.find(params[:id]) 
      if(current_user.owner != @company.user)
        redirect_to(root_path)
      end
    end   

如您所见，每个控制器中都会重复正确的用户。
因此，我在包含所有这些内容的助手中所做的事情是我创建了一个方法：

def correct_user_for_seller_of_controller(controller)
    #"User".classify will return the class User etc.
    @seller = controller.controller_name.classify.constantize.find(params[:id])     
    redirect_to(root_path) unless (current_user == @seller.user)
  end                            

了解我拥有的每个控制器的内部：

class ConstructionCompaniesController < ApplicationController

  before_filter :only => [:edit, :update] do |controller| correct_user_for_seller_of_controller(controller) end           

class RealEstateCompaniesController < ApplicationController

 before_filter :only => [:edit, :update] do |controller| correct_user_for_seller_of_controller(controller) end           

我喜欢现在是 DRY 的事实，但问题是它对我来说似乎有点复杂，很难去理解。我是不是太过分了？

Answer 1

将 right_user 方法添加到 ApplicationController 类中。

class ApplicationController
  def correct_user_for_seller_of_controller
    #"User".classify will return the class User etc.
    @seller = controller_name.classify.constantize.find(params[:id])     
    redirect_to(root_path) unless (current_user == @seller.user)
  end  
end

在您的控制器中使用新方法作为过滤器方法：

class RealEstateCompaniesController < ApplicationController

 before_filter :correct_user_for_seller_of_controller, :only => [:edit, :update]

end

Answer 2

把它清理干净绝对是件好事。我认为你可能让事情变得比必要的稍微复杂一些，尽管如此，对于所有的 Proc 之类的东西。

如果过滤器在控制器的实例上运行，则无需将控制器传递给其自身。只需在方法中调用 controller_name 即可。

Answer 3

我个人会进一步干燥它，并将过滤器移至超类（或 AppplicationController）。

该方法本身也绝对可以简化。例如，进行一些内省。