Question

我想知道解决以下问题的正确方法是什么。

我有两种模式：

Publisher and Issue. Publisher has_many issues.

我想管理发布商列表和问题列表中的问题。

例如：

用户可以点击每个发布者附近的“问题”链接。然后他进入问题列表，但只过滤适当的发布者。他可以点击“创建新问题”并转到表单添加新问题。在此表单上，我无需向他显示选择发布者的选择列表
在问题列表中，用户可以点击“创建新问题”并转到表单，但这次他应该从select中选择发布者，这与创建的问题有关。

简单来说，我只需要针对问题和发布商问题采取行动。

首先我尝试制作一个：

  resources :issues

  resources :publishers do
    resources :issues
  end

和问题控制器：

before_filter :find_issue

def find_issue
 @publisher = Publisher.find(params[:publisher_id]) if params[:publisher_id]
 @issues = @publisher ? @publisher.issues : Issue
end

但是我必须在我的视图和控制器中制作很多条件。

例如，如果问题是从发布商创建的，则成功时我想重定向到publisher_issues_path而不是issue_path，反之亦然。与“返回列表”等类似的所有链接都存在同样的问题。所以代码在我看来不是很透明。

现在我想知道使用命名空间。

namespace :publishers,  do
  resources :issues
end

并制作

# app/controllers/publishers/issues_controller.rb
module Publishers
  class IssuesController < ApplicationController
    # actions that expect a :publisher_id param
  end
end

# app/controllers/issues_controller.rb
class IssuesController < ApplicationController
  # your typical actions without any publisher handling
end

并为两个控制器操作创建单独的视图。

有没有更好或更清洁的方法来解决这类问题？我想让我的代码尽可能干。非常感谢您的回复。

Answer 1

路线：

resources :issues
resources :publishes do
  resources :issues
end

控制器：

class IssuesController < ApplicationController
  before_filter :build_issue, only: [:new, :create]
  before_filter :load_issue,  only: [:show, :edit, :update, :destroy]

  def index
    @issues = issues.page(params[:page])
  end

  ... all other actions will have access to @issue

private
  def issues
    if params[:publisher_id].present?
      Publisher.find(params[:publisher_id]).issues
    else
      Issue
    end  
  rescue ActiveRecord::RecordNotFound
    redirect_to issues_path(alert: 'Publisher not found')
  end

  def build_issue
    @issue = issues.new(issue_params)
  end

  def load_issue
    @issue = issues.find(params[:id])
  rescue ActiveRecord::RecordNotFound
    redirect_to issues_path(alert: 'Issue not found')
  end

  def issue_params
    # whitelisted attributes goes here
  end
end

为避免使用条件，请使用操作而不是完整的命名路径，即：

redirect_to action: :index
link_to 'Issues', {action: :index}

如何管理嵌套资源和非嵌套资源

1 个答案: