Динамически созданные модули в ruby

Question

Я экспериментирую с шаблоном, на который я хотел бы получить обратную связь:

module Concerns
  def AuthenticatedS3Concern(options)
    AuthenticatedS3ConcernHelper.go(options)
  end

  module_function :AuthenticatedS3Concern

  module AuthenticatedS3ConcernHelper
    def self.go(options = {:attribute => :photo})
      @@auth_attr = options[:attribute] # the photo clip reference 
      @@auth_attr_url = "#{@@auth_attr}_authenticated_url" # set this to do a one time download

      Module.new do
        def self.included(base)
          base.send :include, AuthenticatedS3ConcernHelper::InstanceMethods
        end    

        class_eval %(
          def #{@@auth_attr}_authenticated_url(time_limit = 7.days)
            authenticated_url_for('#{@@auth_attr}', time_limit)
          end
        )
      end
    end

    module InstanceMethods    
      def authenticated_url_for(attached_file, time_limit) 
        AWS::S3::S3Object.url_for(self.send(attached_file).path('original'), self.send(attached_file).bucket_name, :expires_in => time_limit)
      end
    end    
  end
end

Который можно использовать так:

require 'concerns/authenticated_s3_concern'
require 'concerns/remote_file_concern'

class Attachment
  include Concerns.AuthenticatedS3Concern(:attribute => :attachment) 
end

Мне интересно, если этохороший подход или плохой подход или что.Есть ли лучший способ выполнить этот вид переменных модулей?

Спасибо

Answer 1

Кроме того, что ваши разработчики по обслуживанию мозгов страдают, я не вижу никакого преимущества в этом.

Насколько я понимаю, весь этот код создает метод экземпляра во включающем классе с именем attribute_name_authenticated_url, который является просто оболочкой для authenticated_url_for.

Вы могли бы легко сделать то же самое, используя method_missing или определив и вызвав метод класса, который создает ваш метод экземпляра. ИМО, этот подход намного проще и удобочитаемее:

module Concerns
  module AuthenticatedS3
    def authenticated_url_for(attached_file, time_limit = 7.days) 
      AWS::S3::S3Object.url_for(self.send(attached_file).path('original'), self.send(attached_file).bucket_name, :expires_in => time_limit)
    end    
  end
end

class Attachment
  include Concerns::AuthenticatedS3
end

@attachment = Attachment.new
@attachment.authenticated_url_for(:attribute_name)

Метапрограммирование лучше всего подходит, когда оно не мешает тому, что вы пытаетесь сделать.

Answer 2

Не знаю, зачем вам вообще нужны модули.

Если все, что вам нужно, это динамически добавить метод с динамическим именем, вы можете начать с:

def make_me_a_method meth
  define_method(meth){|param=7|
    puts param
  }
end

class C
  make_me_a_method :foo
end

C.new.foo(3)
#=> 3