Делегирование методов экземпляра методу класса

В Ruby, предположим, у меня есть класс Foo чтобы позволить мне каталогизировать мою большую коллекцию Foos. Это фундаментальный закон природы, что все Foos являются зелеными и сферическими, поэтому я определил методы класса следующим образом:

class Foo
  def self.colour
    "green"
  end

  def self.is_spherical?
    true
  end
end

Это позволяет мне сделать

Foo.colour # "green"

но нет

my_foo = Foo.new
my_foo.colour # Error!

несмотря на то, что my_foo явно зеленый.

Очевидно, я мог бы определить метод экземпляра colour какие звонки self.class.colour, но это становится громоздким, если у меня много таких фундаментальных характеристик.

Я также могу предположительно сделать это, определив method_missing чтобы попробовать класс для каких-либо отсутствующих методов, но я не уверен, что это то, что я должен делать, или уродливый взлом, или как это сделать безопасно (особенно потому, что я на самом деле под ActiveRecord в Rails, который, как я понимаю, делает некоторые Clever Fun Stuff с method_missing).

Чтобы вы посоветовали?

8 ответов

Решение

Вы можете определить проходной объект:

module Passthrough
  def passthrough(*methods)
    methods.each do |method|
      ## make sure the argument is the right type.
      raise ArgumentError if ! method.is_a?(Symbol)
      method_str = method.to_s
      self.class_eval("def #{method_str}(*args) ; self.class.#{method_str}(*args) ; end")
    end
  end
end

class Foo
  extend Passthrough

  def self::colour ; "green" ; end
  def self::is_spherical? ; true ; end
  passthrough :colour, :is_spherical?
end

f = Foo.new
puts(f.colour)
puts(Foo.colour)

Я вообще не люблю пользоваться eval, но это должно быть довольно безопасно, здесь.

Модуль Forwardable, который поставляется с Ruby, будет делать это красиво:

#!/usr/bin/ruby1.8

require 'forwardable'

class Foo

  extend Forwardable

  def self.color
    "green"
  end

  def_delegator self, :color

  def self.is_spherical?
    true
  end

  def_delegator self, :is_spherical?

end

p Foo.color                # "green"
p Foo.is_spherical?        # true
p Foo.new.color            # "green"
p Foo.new.is_spherical?    # true

Если это просто Ruby, тогда использование Forwardable - правильный ответ

В случае, если это Rails, я бы использовал делегат, например

class Foo
  delegate :colour, to: :class

  def self.colour
    "green"
  end
end

irb(main):012:0> my_foo = Foo.new
=> #<Foo:0x007f9913110d60>
irb(main):013:0> my_foo.colour
=> "green"

Вы можете использовать модуль:

module FooProperties
  def colour ; "green" ; end
  def is_spherical? ; true ; end
end

class Foo
  extend FooProperties
  include FooProperties
end

Немного некрасиво, но лучше, чем использовать method_missing, Я постараюсь поставить другие варианты в других ответах...

С точки зрения дизайна, я бы сказал, что, хотя ответ для всех одинаков Foosцвет и сферический? являются свойствами экземпляров Foo и как таковые должны быть определены как методы экземпляра, а не методы класса.

Однако я могу видеть некоторые случаи, когда вы хотели бы, чтобы такое поведение, например, когда у вас есть Bars в вашей системе также все они синие, и вам передают класс где-то в вашем коде, и вы хотели бы знать, какого цвета будет экземпляр, прежде чем вы вызовете new на уроке.

Кроме того, вы правы, что ActiveRecord широко использует method_missing например, для динамических искателей, поэтому, если вы пошли по этому маршруту, вам необходимо убедиться, что ваш method_missing вызвал тот из суперкласса, если он определил, что имя метода не было тем, которое он мог обработать сам.

Я думаю, что лучший способ сделать это - использовать метод массива Двемти.

Я собираюсь посмотреть и заполнить детали, но вот скелет

РЕДАКТИРОВАТЬ: Yay! За работой!

class Object
  # class where singleton methods for an object are stored
  def metaclass
    class<<self;self;end
  end
  def metaclass_eval &block
    metaclass.instance_eval &block
  end
end
module Defaults
  def self.included(klass, defaults = [])
    klass.metaclass_eval do
      define_method(:add_default) do |attr_name|
        # first, define getters and setters for the instances
        # i.e <class>.new.<attr_name> and <class>.new.<attr_name>=
        attr_accessor attr_name

        # open the class's class
        metaclass_eval do
          # now define our getter and setters for the class
          # i.e. <class>.<attr_name> and <class>.<attr_name>=
          attr_accessor attr_name
        end

        # add to our list of defaults
        defaults << attr_name
      end
      define_method(:inherited) do |subclass|
        # make sure any defaults added to the child are stored with the child
        # not with the parent
        Defaults.included( subclass, defaults.dup )
        defaults.each do |attr_name|
          # copy the parent's current default values
          subclass.instance_variable_set "@#{attr_name}", self.send(attr_name)
        end
      end
    end
    klass.class_eval do
      # define an initialize method that grabs the defaults from the class to 
      # set up the initial values for those attributes
      define_method(:initialize) do
        defaults.each do |attr_name|
          instance_variable_set "@#{attr_name}", self.class.send(attr_name)
        end
      end
    end
  end
end
class Foo
  include Defaults

  add_default :color
  # you can use the setter
  # (without `self.` it would think `color` was a local variable, 
  # not an instance method)
  self.color = "green"

  add_default :is_spherical
  # or the class instance variable directly
  @is_spherical = true
end

Foo.color #=> "green"
foo1 = Foo.new

Foo.color = "blue"
Foo.color #=> "blue"
foo2 = Foo.new

foo1.color #=> "green"
foo2.color #=> "blue"

class Bar < Foo
  add_defaults :texture
  @texture = "rough"

  # be sure to call the original initialize when overwriting it
  alias :load_defaults :initialize
  def initialize
    load_defaults
    @color = += " (default value)"
  end
end

Bar.color #=> "blue"
Bar.texture #=> "rough"
Bar.new.color #=> "blue (default value)"

Bar.color = "red"
Bar.color #=> "red"
Foo.color #=> "blue"

Вы также можете сделать это:

def self.color your_args; your_expression end

define_method :color, &method(:color)

Это будет звучать как коп, но на практике редко возникает необходимость сделать это, когда вы можете так же легко вызвать Foo.color. Исключение составляют случаи, когда у вас есть много классов с определенными методами цвета. @var может быть одним из нескольких классов, и вы хотите отобразить цвет независимо.

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

В редких случаях вы хотите, чтобы метод вызывался обоими, вы можете либо сделать @var.class.color (без создания специального метода), либо создать специальный метод, например, так:

def color self.class.color end

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

Другие вопросы по тегам