Rails：在两个模型之间共享一个控制器

Question

我有两个模型：

学生 教室

它们都有一个完全相同的操作：它显示每日活动报告。那就是：

/students/1
/classrooms/1

获取相关模型的活动并将其显示在页面上。

为了解决这个问题，我创建了一个ReportsController，它提取了构建报告的所有常见逻辑。

如果我这样离开路线：

/students/1/report
/classrooms/1/report

然后我可以让ReportsController#show 操作为:student_id 或:classroom_id 查找params 以确定它正在处理的模型类型（用于查询数据库并呈现正确的视图）。

但我希望 URL 更简洁，因此我还更改了我的 routes.rb 文件以将这些模型的 show 操作传递给 reports#show 控制器操作：

resources :students, :classrooms do
  member do
    get :show, to: 'reports#show'
  end
end

这可行，但我不能再依赖params 来确定要使用哪个模型以及要渲染哪个视图。

问题：我应该为模型解析 request.fullpath 吗？或者有没有更好的方法让共享控制器了解它正在使用的模型？

Answer 1

我会将通用逻辑放在Event 模型中：

#Event Model
class Event < ...
      def self.your_event_method
         #self here will be either student.events or classroom.events 
         #depending on which controller called it
      end
end

class StudentsController < ...
  ...

  def show
    student = Student.find(params[:id])
    student.events.your_event_method
  end

end

class ClassroomsController < ...
  ...

  def show
    classroom = Classroom(params[:id])
    classroom.events.your_event_method
  end

end

Answer 2

将两个 show 方法路由到同一个控制器方法以进行代码重用有点像用翻斗车敲钉子。

即使您可以通过查看请求 url 找到资源，您也会开始将 ResortsController 拆分为一堆 if 和 switch，甚至在您开始之前。

一种解决方案是在模块中添加通用操作：

module Reporting
  extend ActiveSupport::Concern

  def show
    # the Student or Classroom should be available as @resource
    render 'reports/show'
  end

  included do
    before_action :find_resource, only: [:show]
  end

  private

  def find_resource
    model = self.try(:resource_class) || guess_resource_class
    @resource = model.find(params[:id])
  end

  # This guesses the name of the resource based on the controller name.
  def guess_resource_class
    self.class.name[0..-11].singularize.constantize
  end
end

---

class StudentController < ApplicationController
  include Reporting
end

# Example where resource name cannot be deduced from controller
class PupilController < ApplicationController
  include Reporting
  private
  def resource_class
    Student
  end
end

self.class.name[0..-11].singularize.constantize 基本上是 Rails 使用约定优于配置在用户控制器中自动加载User 的方式，即使没有任何代码。

但是，DRY 控制器最重要的关键是让控制器保持纤细。大多数功能可以移动到模型层或委托给服务对象。