Sind Aliasing-Methoden verpönt?

Question

Ich entwickle einen Online-Shop, und der Kunde benötigt die Möglichkeit, eine Bestellung zu löschen und seine Produkte automatisch aufzufüllen (z. B. für Testaufträge). Hier ist mein erster Versuch dies bei der Umsetzung:

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 

Aber was, wenn ich brauche, später eine andere destroy_and_x Methode zu schaffen? Warum nicht zulassen, dass X als Parameter an die destroy() Methode weitergegeben wird? So, jetzt denke ich daran zu gehen:

Dies ist mehr erweiterbar, aber macht mich etwas schmutzig. Bin ich falsch, mich schmutzig zu fühlen? Gibt es einen besseren Weg, dies zu tun?

Answer 1

Ich würde sagen "Ja, das ist dreckig." Sie beabsichtigen nicht, das Verhalten der Methode 'destroy' zu ändern, sondern einige domänenspezifische Arbeiten auszuführen. Führen Sie dann destroy aus. Ihr erster Ansatz ist großartig - definieren Sie eine Methode, die das tut, was Sie wollen, und rufen Sie destroy nach Bedarf auf. Ich denke, dass das "Wrapping" oder "Affe-Patchen" einer Methode, wie du denkst, eine Technik ist, die am besten angewendet wird, wenn Standard-OO-Ansätze nicht verwendet werden können - zB wenn du Verhalten in einem ändern/verbessern musst Klasse, die außerhalb Ihres Kontrollbereichs definiert und verwendet wird.

Auch wenn Sie sind unter Berücksichtigung des Verhaltens der selbst destroy Verfahren zu modifizieren, würde ich zwingende die Methode hier vorschlagen, anstatt es zu wickeln:

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

Wie wäre es nur einen Block verwenden? Dann müssen Sie nicht Ihr Haar auseinander zu ziehen, während dies in der Klasse der Gestaltung und Sie können mehr tun, wie und wann Sie müssen:

def destroy_after &block 
    yield if block 
    destroy 
end 

Dann ist dieses es wie nennen:

order.destroy_after { order.restock_products } 

Ich kann nicht Denken Sie an einen guten Namen für diese Funktion ... aber ich hoffe, Sie bekommen die Idee.

Answer 3

Horace, ich habe deine Frage missverstanden. Ich glaube, Sie für diese suchen:

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

Jetzt können Sie Ihre Methode halten geschützt und so viele before_destroy Dinge hinzufügen, wie Sie möchten. Hoffe das funktioniert für dich ohne zu zerstören.

Viel Glück.

Answer 4

Wenn Monkey Patching Sie nachts nicht schlafen lässt, können Sie dasselbe durch Unterklassen erreichen. Wenn ich einen schnellen Hack oder einen schnellen Debug-Hack brauche, mache ich auch einen Affen-Patch.