has_one,: through => modelo VS método simple?

Question

Tengo algunos problemas con has_one, through => model. Lo mejor es mostrarte mi caso.

class Category 
    has_many :articles 
end 

class Article 
    has_many :comments 
    belongs_to :category 
end 

class Comment 
    belongs_to :article 
    has_one :category, :through => :articles 
end 

Everthing works fine. Puedo hacer comment.category. El problema es cuando creo un nuevo comentario y configuro su artículo, así que guardo el comentario para hacer que la asociación funcione. Ejemplo:

>> comment = Comment.new 
>> comment.article = Article.last 
>> comment.category 
    -> nil 
>> comment.article.category 
    -> the category 
>> comment.save 
>> comment.category 
    -> nil 
>> comment.reload 
>> comment.category 
    -> the category 

has_one, through => model de todos modos no se estableció, construir constructor y crear método. Entonces, quiero reemplazar mi modelo de comentarios por:

class Comment 
    belongs_to :article 
    def category 
    article.category 
    end 
end 

¿Te parece una buena idea?

Answer 1

No hay nada malo con su idea. No puedo ver muchas situaciones en las que has_one :category, :through => :articles sería la mejor opción obvia (a menos que esté ansioso de cargar con Comment.all(:include => :category)).

una pista sobre delegate:

class Comment 
    belongs_to :article 
    delegate :category, :to => :article 

Un enfoque diferente:

class Comment 
    belongs_to :article 
    has_one :category, :through => :article 

    def category_with_delegation 
    new_record? ? article.try(:category) : category_without_delegation 
    end 

    alias_method_chain :category, :delegation 

Answer 2

intenta realizar cambios en su clase de Categoría de la siguiente manera:

class Category 
    has_many :articles 
    has_many :comments, :through => :articles 
end