从 Rails 中的其他控制器调用方法

Question

我试图从另一个控制器中调用一个方法并得到一个无方法错误。

所以我有两个控制器 Jobs 和 Admin，我正在尝试调用：

<% @jobs.each do |job| %>

我将此命令放在管理员控制器的视图中，在我拥有的管理员的实际控制器文件中：

  # GET /jobs
  # GET /jobs.xml
  # GET /admin
  def index
    @jobs = Job.all

    respond_to do |format|
      format.html # index.html.erb
      format.xml  { render :xml => @jobs }
    end

我认为这会拉出我需要的所有东西，但我仍然收到 nomethoderror，那么我应该如何将作业控制器包含到管理控制器中？

Answer 1

从您发布的代码来看，您似乎没有使用查询从数据库中获取您想要的工作，例如

@jobs = Job.all

Job.all 是一个从数据库中获取所有可用记录的活动记录查询。听起来您可能对所有 MVC 部分如何协同工作感到困惑。

首先，您不应该在控制器之间共享方法，实际上您的控制器将只有很少的代码。

Rails 将路由一个请求，并查找控制器中定义的动作，如果找不到，它将转到视图文件夹并在那里获取该动作名称的模板，例如对@987654323 的请求@ 将在控制器中查找索引操作，然后如果不存在，则查找要呈现的 views/jobs/index.erb 模板。因此，如果您提供静态数据并且不需要查找数据，您甚至不需要在控制器中执行操作。但是，通常需要该操作，因为它在控制器中进行数据库和模型查找，然后将这些实例变量传递给视图。将所有数据库查询和模型操作置于视图之外。

def index
  @jobs = Job.all
end

在视图中

@jobs.each do |job| ...

respond_to 块是完全没有必要的，除非您想返回与 html 格式不同的模板，例如 web 服务的 xml 或 json。

Answer 2

您尚未在管理控制器中定义 jobs 操作。您正在管理控制器的索引操作中加载 @jobs。如果您想要这项工作，请将Admin 控制器的index 操作重命名为作业并添加该路由。添加路由的一种方法是在config/routes.rb 中添加一行。

match '/jobs' => 'admin#jobs'

阅读this。