Настройка attr_reader для ленивой реализации атрибутов - PullRequest
5 голосов
/ 14 июля 2010

(Большое редактирование, я получил часть пути туда ...) Я взломал и придумал это как способ указать вещи, которые необходимо сделать перед чтением атрибутов:

class Class
  def attr_reader(*params)
    if block_given?
      params.each do |sym|
        define_method(sym) do
          yield
          self.instance_variable_get("@#{sym}")
        end
      end
    else
      params.each do |sym|
        attr sym
      end
    end
  end
end

class Test
  attr_reader :normal
  attr_reader(:jp,:nope) { changethings if @nope.nil? }

  def initialize
    @normal = "Normal"
    @jp = "JP"
    @done = false
  end

  def changethings
    p "doing"
    @jp = "Haha!"
    @nope = "poop"
  end

end

j = Test.new

p j.normal
p j.jp

Но changethings не признается как метод - у кого-нибудь есть идеи?

Ответы [ 3 ]

4 голосов
/ 14 июля 2010

Вам необходимо оценить блок в контексте экземпляра.yield по умолчанию оценивает его в своем собственном контексте.

class Class
  def attr_reader(*params, &blk)
    if block_given?
      params.each do |sym|
        define_method(sym) do
          self.instance_eval(&blk)
          self.instance_variable_get("@#{sym}")
        end
      end
    else
      params.each do |sym|
        attr sym
      end
    end
  end
end
1 голос
/ 14 июля 2010

ИМХО изменение контекста блока довольно нелогично, с точки зрения того, кто будет использовать такие attr_reader на стероидах .

Возможно, вам следует рассмотреть простой подход "указать имя метода с использованием необязательных аргументов":

def lazy_attr_reader(*args, params)
  args.each do |e|
    define_method(e) do
      send(params[:init]) if params[:init] && !instance_variable_get("@#{e}")
      instance_variable_get("@#{e}")
    end
  end
end

class Foo
  lazy_attr_reader :foo, :bar, :init => :load

  def load
    @foo = 'foo'
    @bar = 'bar'
  end
end

f = Foo.new
puts f.bar
#=> bar
1 голос
/ 14 июля 2010

Вот еще один альтернативный подход, который вы можете посмотреть.Это не так элегантно, как то, что вы пытаетесь сделать, используя define_method, но, возможно, стоит посмотреть.

Добавьте новый метод lazy_attr_reader к Class

class Class
  def lazy_attr_reader(*vars)
    options = vars.last.is_a?(::Hash) ? vars.pop : {}
    # get the name of the method that will populate the attribute from options
    # default to 'get_things'
    init_method = options[:via] || 'get_things'
    vars.each do |var|
      class_eval("def #{var}; #{init_method} if !defined? @#{var}; @#{var}; end")
    end
  end
end

используйте это так:

class Test
  lazy_attr_reader :name, :via => "name_loader"

  def name_loader
    @name = "Bob"
  end
end

В действии:

irb(main):145:0> t = Test.new
=> #<Test:0x2d6291c>
irb(main):146:0> t.name
=> "Bob"
...