Rails after_initialize solo sui "nuovi"

Question

Ho i seguenti 2 modelli

class Sport < ActiveRecord::Base 
    has_many :charts, order: "sortWeight ASC" 
    has_one :product, :as => :productable 
    accepts_nested_attributes_for :product, :allow_destroy => true 
end 

class Product < ActiveRecord::Base 
    belongs_to :category 
    belongs_to :productable, :polymorphic => true 
end 

Uno sport non può esistere senza il prodotto, così nel mio sports_controller.rb ho avuto:

def new 
    @sport = Sport.new 
    @sport.product = Product.new 
... 
end 

ho cercato di spostare la creazione del prodotto per il modello sportivo, utilizzando after_initialize:

after_initialize :create_product 

def create_product 
self.product = Product.new 
end 

ho imparato in fretta t il cappello after_initialize viene chiamato ogni volta che viene creato un modello (ad esempio, da una chiamata find). Quindi non era questo il comportamento che stavo cercando.

Qual è il modo in cui dovrei modellare il requisito che tutti gli sport hanno uno product?

Grazie

Answer 1

Inserire la logica nel controller potrebbe essere la migliore risposta come hai affermato, ma potresti fare in modo che after_initialize funzioni come segue:

after_initialize :add_product 

def add_product 
    self.product ||= Product.new 
end 

In questo modo, imposta il prodotto solo se non esiste alcun prodotto. Potrebbe non valere l'overhead e/o essere meno chiaro di avere la logica nel controller.

Edit: La risposta di Per Ryan, prestazioni-saggio il seguente sarebbe probabilmente meglio:

after_initialize :add_product 

def add_product 
    self.product ||= Product.new if self.new_record? 
end 

Answer 2

Invece di usare after_initialize, come su after_create?

after_create :create_product 

def create_product 
    self.product = Product.new 
    save 
end 

Sembra che risolverebbe il problema?

Answer 3

Sembra che tu sia molto vicino. Dovresti essere in grado di eliminare del tutto la chiamata after_initialize, ma prima credo che se il tuo modello Sport ha una relazione "has_one" con: prodotto come hai indicato, allora il tuo modello di prodotto dovrebbe anche "appartenere allo sport". Aggiungi questo al vostro modello di prodotto

belongs_to: :sport 

Il passo successivo, si dovrebbe ora essere in grado di creare un'istanza di un modello di sport in questo modo

@sport = @product.sport.create(...) 

Questo si basa fuori le informazioni da Association Basics da Ruby on Rails Guide, che potresti avere una lettura se non sono esattamente corretto

Answer 4

Se lo fate self.product ||= Product.new sarà ancora cercare un prodotto ogni volta che fate un find perché ha bisogno di controlla se è nullo o no. Di conseguenza non farà alcun carico impaziente. Per fare ciò solo quando viene creato un nuovo record, è sufficiente verificare se si tratta di un nuovo record prima di impostare il prodotto.

after_initialize :add_product 

def add_product 
    self.product ||= Product.new if self.new_record? 
end 

ho fatto qualche analisi comparativa di base e il controllo if self.new_record? non sembra influire sulle prestazioni in alcun modo evidente.

Answer 5

Si dovrebbe solo eseguire l'override inizializzare metodo come

class Sport < ActiveRecord::Base 

    # ... 

    def initialize(attributes = {}) 
    super 
    self.build_product 
    self.attributes = attributes 
    end 

    # ... 

end 

metodo Initialize viene mai chiamato quando record viene caricato dal database. Si noti che nel codice sopra gli attributi vengono assegnati dopo che il prodotto è stato creato. In tale attributo l'assegnazione degli attributi può influire sull'istanza del prodotto creata.

Answer 6

Sicuramente after_initialize :add_product, if: :new_record? è il modo più pulito qui.

Mantenere il condizionale dalla funzione add_product