エイリアス処理の方法はなんですか？

Question

私はオンラインストアを開発しており、顧客は注文を削除して商品を自動的に補充する必要があります（テストオーダーなど）。

class Order < ActiveRecord::Base 
    def destroy_and_restock 
    restock_products 
    destroy 
    end 

    protected 

    def restock_products 
    line_items.each do |li|  
     li.product.quantity_on_hand += li.quantity 
     li.product.save 
    end 
    end 
end 

しかし、別のdestroy_and_xメソッドを後で作成する必要がある場合はどうすればよいですか。 Xをパラメータとしてdestroy()メソッドに渡すことを許可しないでください。だから今、私はこれで行こうと思っています：

alias :old_destroy :destroy 
def destroy(options = {}) 
    if options['restock'] == true 
    restock_products 
    end 
    old_destroy 
end 

protected 

def restock_products 
    line_items.each do |li|  
    li.product.quantity_on_hand += li.quantity 
    li.product.save 
    end 

これは、より拡張性ですが、私はやや汚い感じさせます。私は汚い感じに間違っていますか？これを行うより良い方法はありますか？

Answer 1

「はい、これは汚れています」と言います。あなたの意図は、 'destroy'メソッドの動作を変更するのではなく、ドメイン固有の作業を行うことです。destroyを実行してください。あなたの最初のアプローチは素晴らしいです - あなたが望むことをするメソッドを定義し、必要に応じてdestroyを呼び出します。あなたが考えているように、メソッドを 'ラッピング'や 'モンキーパッチング'する方法は、標準的なオブジェクト指向のアプローチを使用できないときに最適な手法です - たとえば、あなたのコントロールの範囲外で定義され使用されるクラス。あなたはがdestroy方法自体の動作を変更を検討している場合であっても

、私はむしろそれを包むよりも、ここでメソッドをオーバーライドをお勧めしたい：

def destroy(options = {}) 
    restock_products if options['restock'] 
    super() # I think parens are necessary here, to avoid passing options up the chain 
end 

Answer 2

どのようにだけブロックを使用してはどうですか？そして、あなたはクラスでこれを設計しながら、離れてあなたの髪を引っ張って持っていけない、あなたは、よりを行うことができますし、必要なときには：

def destroy_after &block 
    yield if block 
    destroy 
end 

次に、このようにそれを呼び出す：

order.destroy_after { order.restock_products } 

を私がすることはできませんこの機能の良い名前を考えてみてください...しかし、私はあなたがそのアイデアを得ることを願っています。

Answer 3

Horace、私はあなたの質問に誤解しました。私はあなたがこのために探していると思う：

http://api.rubyonrails.org/classes/ActiveRecord/Callbacks.html

今、あなたはあなたの方法で保護され続けると、あなたが好きなだけbefore_destroyのものを追加することができます。破壊を無効にすることなく、これがあなたのために働くことを望みます。

幸運のベスト。

Answer 4

夜間に猿のパッチを当てても眠れない場合は、サブクラス化することで同じことができます。私が素早くハックする、または素早くデバッグするハックを必要とする時、私はサルのパッチも同様にします。