モデルの保存

Question

私は新しいメソッドを定義したcountry.rbのメソッドを持っています。

class Country < ActiveRecord::Base 
    has_many :states, :dependent => :destroy 

    after_save :count_total_states 

    def count_total_states 
    self.duration = State.count(:conditions => { :country_id => self.id }) 
    Country.save!(:duration => self.duration) 
    end 
end 

私はself.durationの結果を得ました。私はそれを実行したときしかし、それは私が毎回新しい状態が作成されている国に属している、それは状態の数をカウントしたい

undefined method 'save!' for #<Class:0x111170d10>

を言いました。何をすべきかアドバイスしてください。ありがとう。

Answer 1

代わりにupdate_columnを使用してください。コールバックと検証をスキップします。ここにはupdate_columnのドキュメントがあります。

また、states.countには、国別に州を検索するためのクエリが用意されています。 has_many :statesはこれを可能にします。

class Country < ActiveRecord::Base 
    has_many :states, :dependent => :destroy 

    after_save :count_total_states 

    def count_total_states 
    update_column(:duration, states.count) 
    end 
end 

Answer 2

それはこのようにする必要があります

def count_total_states 
    duration = State.count(:conditions => { :country_id => self.id }) 
    self.save!(:duration => duration) 
end 

Answer 3

こちらのミスのカップル。

1. 救います！インスタンスメソッドなので、Country.save！仕事はしません（国はクラスです）。
2. 保存しています。無限ループを引き起こすafter_saveコールバックによってトリガーされているメソッドで呼び出されます。

よりよい解決策は次のようになります。もちろん

class Country < ActiveRecord::Base 
    has_many :states, :dependent => :destroy 

    after_create :count_total_states 

    def count_total_states 
    self.duration = State.count(:conditions => { :country_id => self.id }) 
    self.save   
    end 
end 

をこのシナリオでbefore_saveコールバックを使用して別の利点は、あなたがそうでなければ起こったであろう追加のSQL（UPDATE ...）を保存しているありますafter_saveを使ってあなたのシナリオでは

EDIT

I want it to count number of states belong to the country everytime a new state is created. 

、上記のステートメントを与え、あなた以下のコメントはafter_createメソッドを使用する必要があります。