Railsのベストプラクティス - このコードはどこにありますか？

Question

私は必要なことをすることができる場所がいくつかありますが、どこがベストプラクティスに沿っているのかわかりません。

オーダーコントローラがあり、オーダーが正常に作成された後、サブスクリプション（オーダーが成功した場合のみ）と参照（オーダーが関連付けられている場合のみ）を作成します。

ここで明白な選択肢は、Orderモデルでafter_createを使用することですが...どのようにしてセッションデータを取得できますか？ （照会ID、フレンドID、バウチャーIDは、Order dbに格納する必要がないため、セッション内にのみ存在します）。

作成アクションで購読および紹介オブジェクトを作成するだけです（私が月にどのように持っていますか）、より良い方法がありますか？私は誰を期待していないので、私は本当にこれを適切に行うしたいと思います - （支払いが成功した場合@ order.purchaseのみtrueを返す）

def create 
if @order.save 
    if @order.purchase 
     Subscription.create(:order_id => @order.id, :product_id => @order.product_id) 
     if @order.voucher 
     Referral.create(:user_id => session[:friend_id], :order_id => @order.id, 
         :voucher_amount => @voucher_value) 
     end 
     render :action => "success" 
    else 
     render :action => "failure" 
    end 
    else 
    render :action => 'new' 
    end 
end 

任意の助けをいただければ幸いです ：

は、ここに私のアクションを作成しますたぶん簡単な質問であるかと尋ねています。コールバックの

Answer 1

私は最近同様の質問をしました。見てください、私はコールバックの単純な仮想属性もあなたのためにそれを行うと思います。

Fetch current user in after_create filter

Answer 2

使用は、あなたの人生は簡単だろう、あなたは次モデルのafter_saveコールバックですべてのものを行う

after_save使用する必要があります。セッション変数がモデルで利用できない場合、あなたはまた、すべての

class Order < ActiveRecord::Base 
    def post_save require_attr 

    #create subscriptions 

    # create referral 

    end 
end 

よう

のようなのparamsを必要と受け入れるすべてのロジックに対処するためのpost_saveメソッドを持つことができます。コールバックhere

編集用レールのAPIドキュメントを参照してください