Rails - ActiveRecord 在嵌套路由中有多个条件

Question

我正在建立一个用户可以创建项目的网站。每个项目包含几篇论文。每篇论文都可以通过回答几个问题来进行审核。

我有四个模型（用户、项目、论文、问题）。

在questions_controller 中，我创建了一个动作replies，目的是选择具有特定project_id 和paper_id 的所有回复。

在对象@replies中，我想选择所有在URL中显示project_id和paper_id的论文和项目。

http://0.0.0.0:3000/projects/3/papers/2/questions/1/replies

我想选择所有带有project_id == 3 和paper_id == 2 的回复

为此，我编写了以下代码：

questions_controller.rb

  def replies
    @project = Project.find(params[:project_id])
    @paper = Paper.find(params[:paper_id])
    @replies = Question.where("project_id = ? AND paper_id = ?", params[:project_id], params[:paper_id])
  end

并通过@replies in：

views/questions/replies.html.erb

<% @replies.each do |reply| %>
  <%= reply %>
<% end %>

但是，它不起作用。事实上，我在@replies 中的查询并没有在我的views/questions/replies.html.erb 中传递任何内容。 我不确定我做错了什么。

routes.rb

Rails.application.routes.draw do
  resources :projects do
    resources :papers do
      resources :questions do
        collection do
          get 'replies'
        end
      end
    end
  end
devise_for :users
root to: 'pages#home'

project.rb

class Project < ApplicationRecord
  belongs_to :user
  has_many :papers, dependent: :destroy
  has_many :questions
end

paper.rb

class Paper < ApplicationRecord
  belongs_to :project
  has_many :questions
  has_one_attached :paper_pdf
end

question.rb

class Question < ApplicationRecord
  belongs_to :user
  belongs_to :paper
  belongs_to :project
end

user.rb

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  has_many :projects
  has_many :questions
end

Answer 1

您的应用程序设计中存在不同的问题。

第一个是问题的关系：

class Question < ApplicationRecord
  belongs_to :user
  belongs_to :paper
  belongs_to :project
end

我了解您在这里要做什么：一个问题可以适用于不同的模型：用户、论文、项目。虽然有一个问题：这个问题只能引用这些模型中的一个（一个问题同时适用于不同的对象会很奇怪）。然后，当您创建与用户相关的新问题时，论文 (paper_id) 和项目 (project_id) 的外键将为空。

这不是一个好的设计。这样做的方法是使 Question 模型成为多态模型。

如果 Question 应该只适用于论文，正如你上面提到的，那么为什么 Question 是 Paper 以外的其他模型的子模型？

第二个问题：您尝试使用两个外键获取问题：

@replies = Question.where("project_id = ? AND paper_id = ?", params[:project_id], params[:paper_id])

正如我在前面的问题 1 中所描述的，您的 Question 模型可能有 3 个外键（指向它所属的 3 个模型），但一个问题一次不能引用多个模型。

因此，当您尝试同时使用这 2 个外键提出问题时，您的查询可能得不到答案。

第三个问题：使用浅嵌套，你的路由太深了：

http://0.0.0.0:3000/projects/3/papers/2/questions/1/replies

这里不需要三层嵌套。您似乎是初学者，因此可能感觉不直观，但请查看下面的建议

建议：

如果我正确阅读了您的问题布局，则问题仅适用于论文。 那么如果你想得到与这篇论文相关的所有问题，最好在这个模型中创建一个新方法get replies，而不是问题模型。

让它成为一个成员，而不是一个集合：

路线：

...
resources :papers do
  member do
     get 'replies'
  end
end
...

那么当你的路线以papers/5/replies结束时

您可以在纸张控制器中触发以下代码：

def replies
  @paper = Paper.find(params[:id])
  @replies = @paper.questions
end

您注意到我的操作代码实际上并不需要路由中的任何前面的 id。我只选择与数字 5 相关的params[:id]。纸模型的 id。

如果您有深层嵌套路由，您会意识到路由中有一些未使用的数据。路线中真正重要的数据是最后一个参数。也许是前面的参数。

如果您需要确保论文属于当前用户项目，只需将当前用户与论文 id 进行比较，例如：

unless current_user.id == Paper.find(params[:id]).project.user.id
redirect_to root_path

然后就是这样