méthode d'auto module appelé Undefined method_added

Question

Je suis en train de mettre en œuvre une hooker pour les méthodes de classe, quelque chose comme before_action et after_action. Le problème est si je définir une méthode l'intérieur du module, avec define_method ou normalement avec def do_before; puts 'do_before called'; end à l'intérieur method_added obtient non défini. Alors, comment puis-je appeler un module method intérieur method_added?

module Hooker 

    [:before, :after].each do |element| 
    define_method("#{element}_action") do |name| 
     puts "#{element}_action called with parameter #{name}" 
    end 
    define_method("do_#{element}") do 
     puts "do_#{element} called" 
    end 
    end 
    def method_added(name) 
    return if @filtering # Don't add to original_ methods 
    @filtering = true 

    alias_method :"original_#{name}", name 
    define_method name do |*args| 
     do_before # undefined method `do_before' for #<Bar:0x007ff2f183c318> 
     self.send("original_#{name}", *args) 
     do_after # undefined method `do_after' for #<Bar:0x007ff2f183c318> 
    end 

    @filtering = false 
    end 
end 

class Bar 
    extend Hooker 

    before_action 'foo2' 
    after_action 'bar2' 

    def my_func 
    puts 'MyFunc called' 
    end 
end 

Bar.new.my_func 

Answer 1

C'est parce que vous utilisez - extend

ajoute les méthodes et les constantes du module spécifié métaclasse

de la cible, mais vous devez également - include

il se mélange dans les méthodes de module spécifié comme méthodes d'instance dans la classe cible

# some code goes here 

class Bar 
    extend Hooker 
    include Hooker 

    before_action 'foo2' 
    after_action 'bar2' 

    def my_func 
    puts 'MyFunc called' 
    end 
end 

Bar.new.my_func 
=> before_action called with parameter foo2 
=> after_action called with parameter bar2 
=> do_before called 
=> MyFunc called 
=> do_after called 

Beaucoup plus clairement de séparer ce à différents modules.

Answer 2

est ici une solution de rechange, dans laquelle vous n'avez pas besoin d'appeler explicitement dans la classe à la fois extend et include:

module Hooker 
    def self.included(klass) 
    klass.extend ClassMethods 
    end 

    module ClassMethods 
    [:before, :after].each do |element| 
     define_method("#{element}_action") do |name| 
     puts "#{element}_action called with parameter #{name}" 
     end 
     define_method("do_#{element}") do 
     puts "do_#{element} called" 
     end 
    end 
    end 

    def method_added(name) 
    return if @filtering # Don't add to original_ methods 
    @filtering = true 

    alias_method :"original_#{name}", name 
    define_method name do |*args| 
     do_before 
     self.send("original_#{name}", *args) 
     do_after 
    end 

    @filtering = false 
    end 
end 

class Bar 
    include Hooker 

    before_action 'foo2' 
    after_action 'bar2' 

    def my_func 
    puts 'MyFunc called' 
    end 
end 

Bar.new.my_func 

L'astuce ici est que le module Hooker est maintenant extend ing si le include .