Rails：除了 user_id 之外，大多数 Quiz 属性都传递给 params。为什么？

Question

出于测试/学习目的，我有一个具有以下属性的测验模型（来自 schema.rb 的 sn-p）：

create_table "quizzes", force: true do |t|
    t.integer  "user_id"
    t.string   "answer1"
    t.string   "answer2"
    t.string   "answer3"
    t.string   "answer4"
    t.string   "answer5"
    t.string   "answer6"
    t.string   "answer7"
    t.string   "answer8"
    t.datetime "created_at"
    t.datetime "updated_at"
  end

  add_index "quizzes", ["user_id"], name: "index_quizzes_on_user_id"

测验被发送到“新”视图，如下所示：

def new
    @quiz = Quiz.new(user_id: current_user.id)
end

当用户完成测验时，QuizzesController#create 动作被触发：

def create
    @results = Quiz.create(post_params) #from private method
    render :show
end

（无论是呈现“显示”视图还是默认的“创建”视图，它都有效 - 在我测试和学习时，我现在将这两个视图都用作占位符）

post_params 方法如下所示：

def post_params
    params.require(:quiz).permit(:user_id, :answer1, :answer2, :answer3, :answer4, :answer5, :answer6, :answer7, :answer8) #add other attributes here
end

（我也尝试过params.require(:quiz).permit!，但得到了相同的结果）。

问题是，在视图中，它看起来像这样：

<div class="container">
      <!-- Example row of columns -->
      <div class="row">
        <h1>Results</h1>
        <p><%= @results.inspect %></p>
    </div>
</div>

...@results.inspect 行显示所有 answer1、answer2 等属性，但将 user_id 属性显示为 nil - 尽管在 QuizzessController#new 操作中使用 user_id 设置了测验。我做错了什么，导致 user_id 无法正确传递？

Answer 1

如果您总是要将user_id 设置为current_user.id，我建议您在create 操作中添加该属性。

这样您就不必担心用户输入。否则，用户可能会滥用它来更改 id，因此测验会被添加到另一个用户。

首先，从允许的post_params

中删除user_id

def post_params
  params.require(:quiz).permit(:answer1, :answer2, :answer3, :answer4, :answer5, :answer6, :answer7, :answer8)
end

然后只需将您的创建操作更新为：

def create
  # as mentioned by tagCindy, this is the way to set up the association
  @quiz = current_user.build_quiz(post_params)

  # you should also re-render the new form if there are any errors
  if @quiz.save
    render :show
  else
    render :new
  end
end

Answer 2

首先，查看ActiveRecord associations。您的测验模型和您的用户模型之间应该有关联（多对一或一对一）。如果关联设置正确，并且 current_user 数据保存到会话中，则无需将 user_id 与 POST 一起传递。

class QuizzesController < ApplicationController
# grabs the current_user from the session
before_filter :get_current_user

# ...

def create
    # METHOD 1: initialize and persist, check for errors
    @quiz = current_user.create_quiz(quiz_params)
    if @quiz.errors
        render :new
    else
        redirect_to @quiz # or where ever

   # METHOD 2: initialize, check for successful save
   @quiz = current_user.build_quiz(quiz_params)
   if @quiz.save
        render :new
    else
        redirect_to @quiz # or where ever
end

# ...

private

def get_current_user
    @current_user = User.find(session[:user_id])
end

def quiz_params
    params.require(:quiz).permit(:answer1, :answer2, :answer3, :answer4, :answer5, :answer6, :answer7, :answer8)
end

end

如果您不希望将 user_id 存储在会话中，我会设置您的路由，以便将 user_id 传递给路由中的控制器。应不惜一切代价避免将 user_id 作为表单上的隐藏值传递，因为它不安全并且很容易被操纵和/或欺骗。

附带说明，您应该有一个与 Quiz 具有多对一关系的 Answer 模型。

Answer 3

那是因为post_params 旨在处理提供给控制器的参数。即 POST/GET

user_id（我假设）不在您的表单中，因此不会通过表单提供给控制器。

我会做的是：

def create
  @results = Quiz.create(post_params.merge(user_id: current_user.id) #from private method
  render :show
end

如果您正确设置了关系，那就更好了（如果它是 1>1，则单数化测验）：

def create
  @results = current_user.create_quiz(post_params) #this will fill in the user_id field for you
  render :show
end

在您的表单中，请勿：

= f.hidden_field :user_id, current_user.id

这样做意味着任何用户都可以为任何其他用户创建测验。