模型之间关系的NoMethodError

时间:2015-03-05 17:07:47

标签: ruby-on-rails ruby ruby-on-rails-3 relationship

我正在建立两个模型之间的关系。患者模型和工作清单模型。我的病人可以属于工作清单,工作清单可以有很多病人。在我的患者表格中,我希望能够从下拉菜单中选择工作清单。我在表单中创建了一个方法,但它抛出了一个错误。这是我的代码:

来自_form.html.erb的

<div class="field">
  <%= f.label :worklist_id %>
  <%= f.collection_select :worklist_id, @worklists, :id , :name %>
</div>

来自patient.rb:

class Patient < ActiveRecord::Base
  belongs_to :worklist
end
来自worklist.rb

class Worklist < ActiveRecord::Base
  has_many :patients
end
来自schema.rb的

  create_table "patients", force: :cascade do |t|
    t.string   "lastname"
    t.string   "middlename"
    t.string   "firstname"
    t.date     "birthdate"
    t.string   "sex"
    t.string   "ethnicity"
    t.string   "uniqueid"
    t.string   "accountnumber"
    t.string   "medicaidid"
    t.string   "medicareid"
    t.string   "ssn"
    t.datetime "created_at",    null: false
    t.datetime "updated_at",    null: false
    t.integer  "worklist_id"
  end

这是我得到的错误:

NoMethodError in Patients#edit
undefined method `worklist_select'

这是PatientController:

class PatientsController < ApplicationController
  before_action :set_patient, only: [:show, :edit, :update, :destroy]

  # GET /patients
  # GET /patients.json
  def index
    @patients = Patient.all
  end

  # GET /patients/1
  # GET /patients/1.json
  def show
  end

  # GET /patients/new
  def new
    @patient = Patient.new
  end

  # POST /patients
  # POST /patients.json
  def create
    @patient = Patient.new(patient_params)
    @worklists = Worklist.all

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

  # GET /patients/1/edit
  def edit
  end

  # PATCH/PUT /patients/1
  # PATCH/PUT /patients/1.json
  def update
    respond_to do |format|
      if @patient.update(patient_params)
        format.html { redirect_to @patient, notice: 'Patient was successfully updated.' }
        format.json { render :show, status: :ok, location: @patient }
      else
        format.html { render :edit }
        format.json { render json: @patient.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /patients/1
  # DELETE /patients/1.json
  def destroy
    @patient.destroy
    respond_to do |format|
      format.html { redirect_to patients_url, notice: 'Patient was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

    def import
      Patient.import(params[:file])
      redirect_to patients_path, notice: "Patients Added Successfully"
    end

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

    # Never trust parameters from the scary internet, only allow the white list through.
    def patient_params
      params.require(:patient).permit(:lastname, :middlename, :firstname, :birthdate, :sex, :ethnicity, :uniqueid, :accountnumber, :medicaidid, :medicareid, :ssn, :worklist_id)
    end
end

2 个答案:

答案 0 :(得分:0)

视图中的

Worklist.all是个问题。直接与db交谈的观点永远不会好。这是Controller(或服务)的工作。

在您的控制器中,您需要以下内容:@worklists = Worklist.all,然后在视图中使用@worklists

这只是一个开始。您可以发布您的控制器,以便我们可以进一步提供帮助吗?

更新:现在我们可以更好地了解这种情况。让我们这样做:

因此,在您的PatientsController中,您需要在create方法中使用它:

@worklists = Worklist.all

然后在_form.html.erb(来自患者文件夹而非Worklist文件夹)中,您需要:

<%= f.label :worklist_id %>
<%= f.collection_select :worklist_id, @worklists, :id , :name %>

答案 1 :(得分:0)

我想如果你改变这个:

<%= worklist_select( :patient, :worklist_id, Worklist.all, :id, :name, {}, { :multiple => false } ) %>

更像这样的事情:

<%= f.select( :worklist_id, Worklist.all, :id, :name, {}, { :multiple => false } ) %>

你可能会得到你想要的东西。