2009-07-23 6 views
9

Je rencontre des problèmes avec mon application Ruby on Rails. J'ai deux modèles - «patient» et «adresse», un patient a une adresse, et une adresse appartient à un patient.Erreur AssociationTypeMismatch sur l'application Ruby on Rails

Patient.rb

class Patient < ActiveRecord::Base 
    has_many :charge_slips 
    has_one :address 

    validates_presence_of :last_name 
    validates_presence_of :first_name 
    validates_presence_of :middle_name 

end 

Address.rb

class Address < ActiveRecord::Base 
    belongs_to :patient 
    validates_associated :patient 
end 

patient controller.rb

class PatientController < ApplicationController 
    def index 
    @title = "Outpatient Services - Patient" 
    @today = Date.today.to_formatted_s(:long) 
    @patients = Patient.find(:all) 
    end 

    def new 
    @patient = Patient.new 
    @address = Address.new 
    end 

    def create 
    @patient = Patient.new(params[:patient]) 
    @patient.created_on = Date.today.to_formatted_s(:long) 

    if @patient.save 
     @address = Address.new(params[:address]) 
     @address.patient_id = @patient.id 
     if @address.save 
     redirect_to :action => 'index' 
     else 
     redirect_to :action => 'new' 
     end 
     redirect_to :action => 'index' 
    else 
     redirect_to :action => 'new' 
    end 
    end 
end 

new.html.rb

<%= content_tag('h3', 'Create New Patient') %> 
<hr> 
<% form_for @patient, :url => { :action => "create" } do |patient_form| -%> 
    <%= error_messages_for :patient %> 
    <%= patient_form.label :last_name, 'Last Name:' %> <%= patient_form.text_field :last_name, :size => 30 %><br> 
    <%= patient_form.label :first_name, 'First Name:' %> <%= patient_form.text_field :first_name, :size => 30 %><br> 
    <%= patient_form.label :middle_name, 'Middle Name:' %> <%= patient_form.text_field :middle_name, :size => 30 %><br> 

    <fieldset> 
     <legend>Patient's Permanent Address</legend> 
     <%= error_messages_for :address %> 
     <% patient_form.fields_for @address do |address_fields| -%> 
      <%= address_fields.label :street_name, 'Street Name:' %> <%= address_fields.text_field :street_name %><br> 
      <%= address_fields.label :barangay, 'Barangay:' %> <%= address_fields.text_field :barangay %><br> 
      <%= address_fields.label :city_municipality, 'City/Municipality:' %> <%= address_fields.text_field :city_municipality %><br> 
      <%= address_fields.label :country, 'Country:' %> <%= address_fields.text_field :country %><br> 
      <%= address_fields.label :zip_cide, 'Zip Code:' %> <%= address_fields.text_field :zip_code %><br> 
     <% end -%> 
    </fieldset> 

    <%= submit_tag "Add Patient" %> 
<% end -%> 

Chaque fois que j'ajoute un nouveau patient, une erreur est générée. Voici une partie de l'erreur:

ActiveRecord::AssociationTypeMismatch in PatientController#create 

Address(#31360520) expected, got HashWithIndifferentAccess(#23815500) 

RAILS_ROOT: C:/www/Outpatient Application Trace | Framework Trace | Full Trace 

C:/ruby/lib/ruby/gems/1.8/gems/activerecord-2.3.3/lib/active_record/associations/association_proxy.rb:263:in `raise_on_type_mismatch' 
C:/ruby/lib/ruby/gems/1.8/gems/activerecord-2.3.3/lib/active_record/associations/has_one_association.rb:52:in `replace' 
C:/ruby/lib/ruby/gems/1.8/gems/activerecord-2.3.3/lib/active_record/associations.rb:1246:in `address=' 
C:/ruby/lib/ruby/gems/1.8/gems/activerecord-2.3.3/lib/active_record/base.rb:2740:in `send' 
C:/ruby/lib/ruby/gems/1.8/gems/activerecord-2.3.3/lib/active_record/base.rb:2740:in `attributes=' 
C:/ruby/lib/ruby/gems/1.8/gems/activerecord-2.3.3/lib/active_record/base.rb:2736:in `each' 
C:/ruby/lib/ruby/gems/1.8/gems/activerecord-2.3.3/lib/active_record/base.rb:2736:in `attributes=' 
C:/ruby/lib/ruby/gems/1.8/gems/activerecord-2.3.3/lib/active_record/base.rb:2434:in `initialize' 
C:/www/Outpatient/app/controllers/patient_controller.rb:14:in `new' 
C:/www/Outpatient/app/controllers/patient_controller.rb:14:in `create' 

Je suis nouveau à RoR et aimerais apprendre la langue par la pratique. Je veux savoir ce qui ne va pas avec le code. Merci!

Répondre

28

d'abord votre modèle Patient a besoin d'un accepts_nested_attributes_for

class Patient < ActiveRecord::Base 
    has_many :charge_slips 
    has_one :address 

    validates_presence_of :last_name 
    validates_presence_of :first_name 
    validates_presence_of :middle_name 

    accepts_nested_attributes_for :address 

end 

Votre contrôleur peut être simplifié beaucoup. Il n'est pas nécessaire de sauvegarder l'adresse séparément puisque @patient.save s'en chargera. Vous n'avez pas besoin de définir l'attribut created_on manuellement car il sera défini automatiquement :) En outre, lorsque @patient.save échoue, vous voulez probablement render :action => 'new' et non redirect_to :action => 'new'. Cela affichera à nouveau le formulaire avec toutes les erreurs de validation (redirect_to non.)

Notez également que j'ai renommé votre classe de contrôleur à PatientsController au lieu de PatientController. Cela sera plus conforme aux conventions RESTful de Rails et vous aidera également à simplifier votre vision. Si vous faites cela, vous aurez besoin d'un map.resources :patients dans votre fichier routes.db, et vous devrez également renommer vos fichiers.

class PatientsController < ApplicationController 
    def index 
    @title = "Outpatient Services - Patient" 
    @today = Date.today.to_formatted_s(:long) 
    @patients = Patient.find(:all) 
    end 

    def new 
    @patient = Patient.new 
    @patient.build_address 
    end 

    def create 
    @patient = Patient.new(params[:patient]) 

    if @patient.save 
     redirect_to :action => 'index' 
    else 
     render :action => 'new' 
    end 
    end 
end 

Votre vue a une petite erreur. Il doit être fields_for :address et non fields_for @address. Aussi depuis que votre contrôleur est maintenant RESTful, vous pouvez supprimer la pièce :url => { :action => "create" }.

<%= content_tag('h3', 'Create New Patient') %> 
<hr> 
<% form_for @patient do |patient_form| -%> 
    <%= error_messages_for :patient %> 
    <%= patient_form.label :last_name, 'Last Name:' %> <%= patient_form.text_field :last_name, :size => 30 %><br> 
    <%= patient_form.label :first_name, 'First Name:' %> <%= patient_form.text_field :first_name, :size => 30 %><br> 
    <%= patient_form.label :middle_name, 'Middle Name:' %> <%= patient_form.text_field :middle_name, :size => 30 %><br> 

    <fieldset> 
     <legend>Patient's Permanent Address</legend> 
     <%= error_messages_for :address %> 
     <% patient_form.fields_for :address do |address_fields| -%> 
       <%= address_fields.label :street_name, 'Street Name:' %> <%= address_fields.text_field :street_name %><br> 
       <%= address_fields.label :barangay, 'Barangay:' %> <%= address_fields.text_field :barangay %><br> 
       <%= address_fields.label :city_municipality, 'City/Municipality:' %> <%= address_fields.text_field :city_municipality %><br> 
       <%= address_fields.label :country, 'Country:' %> <%= address_fields.text_field :country %><br> 
       <%= address_fields.label :zip_cide, 'Zip Code:' %> <%= address_fields.text_field :zip_code %><br> 
     <% end -%> 
    </fieldset> 

    <%= submit_tag "Add Patient" %> 
<% end -%> 

Hope this helps :)