Question

我正在努力弄清楚如何设置我的rails评估模型，以便用户可以使用它来为其他用户留下反馈。

我在这篇文章中概述了我的问题的关键部分：

Rails - feedback on specific users, how to set up the form to identify relevant users

我收到的建议是将模型设置如下：

User.rb

has_many :given_evaluations, foreign_key: :evaluator_id, dependent: :destroy, class_name: Evaluation
  has_many :received_evaluations, foreign_key: :evaluatee_id, dependent: :destroy, class_name: Evaluation

Evaluation.rb

belongs_to :evaluator, foreign_key: :evaluator_id, class_name: User
  belongs_to :evaluatee, foreign_key: :evaluatee_id, class_name: User

评估控制器

class EvaluationsController < ApplicationController
  before_action :set_evaluation, only: [:show, :edit, :update, :destroy]
  # before_filter :get_user, only: [:show, :edit, :update, :destroy]

  # GET /evaluations
  # GET /evaluations.json
  def index
    # @evaluations = Evaluation.all
    @given_evaluations = current_user.given_evaluations
    @received_evaluations = current_user.received_evaluations
  end




  # GET /evaluations/1
  # GET /evaluations/1.json
  def show
    # @received_evaluations = @user.received_evaluations
    @evaluation = current_user.received_evaluations.find_by(id: params[:id]) || current_user.given_evaluations.find(params[:id])

    # @received_evaluation = current_user.received_evaluations.find params[:id]
  end

  # GET /evaluations/new
  def new
    @evaluation = Evaluation.new
  end

  # GET /evaluations/1/edit
  def edit
  end

  # POST /evaluations
  # POST /evaluations.json
  def create
    # @evaluation = Evaluation.new(evaluation_params)
    @evaluation = current_user.given_evaluations.build(evaluation_params)

    respond_to do |format|
      if @evaluation.save
        format.html { redirect_to @evaluation, notice: 'Evaluation was successfully created.' }
        format.json { render :show, status: :created, location: @evaluation }
      else
        format.html { render :new }
        format.json { render json: @evaluation.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /evaluations/1
  # PATCH/PUT /evaluations/1.json
  def update
    current_user.given_evaluations.find(params[:id])
    respond_to do |format|
      if @evaluation.update(evaluation_params)
        format.html { redirect_to @evaluation, notice: 'Evaluation was successfully updated.' }
        format.json { render :show, status: :ok, location: @evaluation }
      else
        format.html { render :edit }
        format.json { render json: @evaluation.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /evaluations/1
  # DELETE /evaluations/1.json
  def destroy
    current_user.given_evaluations.find(params[:id])
    @evaluation.destroy
    respond_to do |format|
      format.html { redirect_to evaluations_url, notice: 'Evaluation was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_evaluation
      @evaluation = Evaluation.find(params[:id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def evaluation_params
      params[:evaluation].permit(:overall_score, :project_score, :personal_score, :remark, :work_again?, :continue_project?, :evaluatee_id)
    end
end

评估表

<%= simple_form_for(@evaluation) do |f| %>
  <%= f.error_notification %>

  <div class="form-inputs">
    <%= f.select :evaluatee_id, User.all.map{|u| [u.formal_name, u.id]} %>

    <%= f.input :overall_score, collection: 1..10, autofocus: true, :label => "How do you rate this project experience (1 being did not meet expectations - 10 being met all expectations) ?" %>
    <%= f.input :continue_project?, as: :boolean, checked_value: true, unchecked_value: false, :label => "Do you intend to continue working on the project?" %>
    <%= f.input :remark, as: :text, :label => "Evaluate your experience", :input_html => {:rows => 10}  %>



  </div>

  <div class="form-actions">
    <%= f.button :submit %>

评估显示视图

<% @received_evaluations.each do |receval| %>
                                <div id="portfolioFiltering" class="masonry-wrapper row">
                                        <%= receval.remark %>
                                        <%#= eval.personal_score %>
                                        <small><%= receval.created_at %></small>
                                </div>    
                            <% end %>

评估显示视图的替代尝试

<% @given_evaluations.each do |receval| %>
                                <div id="portfolioFiltering" class="masonry-wrapper row">
                                        <%= receval.remark %>
                                        <%#= eval.personal_score %>
                                        <small><%= receval.created_at %></small>
                                </div>    
                            <% end %>

我现在遇到的问题是，无论我是否尝试在节目中显示给定评估或收到评估，我都会收到一条错误消息：

undefined method `each' for nil:NilClass

我无法弄清楚如何设置模型以便用户可以评估其他用户。我想在各自的展示页面上显示每个用户收到的评估。我无法弄清楚出了什么问题。我可以从控制台看到用户已收到评估结果：

=> #<Evaluation id: 8, evaluatee_id: 34, overall_score: 4, project_score: 5, personal_score: 5, remark: "jhjkhjhjkhkjhjkhjhkhjhkj", work_again?: nil, continue_project?: nil, created_at: "2016-06-12 21:52:53", updated_at: "2016-06-12 21:52:53", evaluator_id: 34>

我正在使用的用户有一个条目。但是，我找不到表明评估的方法。

Answer 1

我能看到的最直接的问题是实例变量：@given_evaluations和@received_evaluations未在您的show动作中设置，并且注释部分使用ActiveRecord#find方法，会返回一个实例，因此无法循环评估。

我认为在index操作中显示所有用户评估的更好位置，正如您已经做的那样，您可以将节目的当前逻辑移动到索引视图。要显示每个用户在show操作上收到的评估，您可以执行以下操作：

@evaluation = current_user.received_evaluations.find(params[:id])

然后在show视图中，您应该有类似的内容：

<div id="portfolioFiltering" class="masonry-wrapper row">
  <%= @evaluation.remark %>
  <%= @evaluation.personal_score %>
  <small><%= @evaluation.created_at %></small>
</div>

Rails - 关联，用户对用户的反馈 - 显示反馈

1 个答案: