méthode non pour ActiveRecord::Relation

Utilisateur modèle

class User < ActiveRecord::Base
  has_many :medicalhistory 
end

Mdedicalhistory modèle

class Medicalhistory < ActiveRecord::Base
  belongs_to :user #foreign key -> user_id
  accepts_nested_attributes_for :user
end

Erreur

undefined method `lastname' for #<ActiveRecord::Relation:0xb6ad89d0>


#this works
@medicalhistory = Medicalhistory.find(current_user.id) 
print   "\n" + @medicalhistory.lastname

#this doesn't!
@medicalhistory = Medicalhistory.where("user_id = ?", current_user.id)
print   "\n" + @medicalhistory.lastname #error on this line
  • ce message d'erreur?
  • Ne @medicalhistory.first.lastname travail?
  • 🙁 oui, c'est....insight?
  • vous ne savez pas si c'est les "meilleures pratiques", mais @medicalhistory = Medicalhistory.où("user_id = ?", current_user.id)[0] devrait fonctionner
InformationsquelleAutor Omnipresent | 2011-05-14
  1. 40

    Bien, vous obtenez en retour un objet de ActiveRecord::Relation, pas votre modèle d'instance, et donc l'erreur, puisqu'il n'existe pas de méthode appelée lastname dans ActiveRecord::Relation.

    Faire @medicalhistory.first.lastname fonctionne parce que @medicalhistory.first est de retour de la première instance du modèle qui a été trouvé par le where.

    Aussi, vous pouvez imprimer @medicalhistory.class pour le travail et "erroring" code et de voir comment ils sont différents.

    • Merci, j'ai raté first méthode lorsque l'on regarde l'API.
    InformationsquelleAutor Zabba
  2. 5

    Une autre chose à noter, :medicalhistory devrait être au pluriel car c'est un has_many relation

    De sorte que votre code: 

    class User < ActiveRecord::Base
  has_many :medicalhistory 
end

    Devrait être écrit: 

    class User < ActiveRecord::Base
  has_many :medicalhistories 
end

    Des Rails docs (trouvé ici)

    Le nom de l'autre modèle est la marque du pluriel lorsque l'on déclare une has_many
    association.

    C'est parce que les rails déduit automatiquement le nom de la classe à partir de l'association nom.

    Si un utilisateur n' had_one medicalhistory ce serait singulier que vous aviez écrit:

    class User < ActiveRecord::Base
  has_one :medicalhistory 
end

    Je sais que vous avez déjà accepté une réponse, mais pensé que ce serait aider à réduire les erreurs et de confusion.

    InformationsquelleAutor Thomas