我如何重构 Rail 的 ApplicationController，并将常量设置为 ActiveRecord 结果？

Question

我正在开发一个 Rails 2.1 项目，该项目具有类似于以下内容的 ApplicationController：

class ApplicationController < ActionController:Base
  THIS_SITE = Site.find_by_url('www.example.net')
  ADMIN_ROLES = StaffRole.find(:all, :conditions => ["site_id = ? AND name IN (?)", ApplicationController::THIS_SITE.id, 'Administrator'])
end

我无法运行测试，因为在我的测试环境中，当尝试定义 ADMIN_ROLES 时，我会收到一条错误消息，指出 THIS_SITE 为 nil。我通过执行以下操作解决了该问题：

class ApplicationController < ActionController:Base
  def self.this_site
    @@this_site ||= Site.find_by_url('www.example.net')
  end
  def self.admin_roles
    @admin_roles ||= StaffRole.find(:all, :conditions => ["site_id = ? AND name IN (?)", ApplicationController::this_site.id, 'Administrator'])
end

ApplicationController::THIS_SITE 和 ApplicationController::ADMIN_ROLES 在整个应用程序的视图、控制器和模型中使用。我觉得这一切都是错误和肮脏的。我认为这是错误的/肮脏的吗？如果不是，如何重构？

谢谢。

Answer 1

测试模式下的问题是，即使在填充测试数据库之前，ApplicationController 类也会被加载。虽然使用常量本身没有问题；定义类方法将加载从数据库时刻移至未来，并让您运行测试。

但是，我确实认为您的代码中的“@@this_site”可能应该是“@this_site”。

从系统架构师的角度来看，this_site 和 admin_roles 看起来都像每个应用程序的设置，应该从控制器部分移出。定义应用程序范围设置的好地方是在 config/initializers 下面或在您的环境中。rb

BTW：Rails 2.1 是不是有点过时了？