Clases definidas dinámicamente que comparten datos de forma incorrecta: error o error de codificación.

Question

He intentado configurar un sistema mediante el cual puedo generar una serie de clases de Ruby similares, distinguidas por un parámetro entero, que guardo en una variable de clase de la clase relevante, algo parecido a las plantillas de C++.

Sin embargo, la referencia (y, por lo tanto, la creación) de una nueva versión de la clase con plantilla sobrescribe los parámetros guardados en las versiones anteriores, y no sé por qué.

He aquí un ejemplo mínimo

class Object 
    def self.const_missing(name) 
    if name =~ /^Templ(\d+)$/ 
     return make_templ $1.to_i 
    else 
     raise NameError.new("uninitialised constant #{name}") 
    end 
    end 

private 
    def make_templ(base) 
    # Make sure we don't define twice 
    if Object.const_defined? "Templ#{base}" 
     return Object.const_get "Templ#{base}" 
    else 
     # Define a stub class 
     Object.class_eval "class Templ#{base}; end" 

     # Open the class and define the actual things we need. 
     Object.const_get("Templ#{base}").class_exec(base) do |in_base|   
     @@base = in_base 

     def initialize 
      puts "Inited with base == #{@@base}" 
     end 
     end 

     Object.const_get("Templ#{base}") 
    end 
    end 
end 

irb(main):002:0> Templ1.new 
Inited with base == 1 
=> #<Templ1:0x26c11c8> 
irb(main):003:0> Templ2.new 
Inited with base == 2 
=> #<Templ2:0x20a8370> 
irb(main):004:0> Templ1.new 
Inited with base == 2 
=> #<Templ1:0x261d908> 

¿He encontrado un fallo en mi Ruby (Ruby 1.9.2p290 (2011-07-09) [i386-mingw32]), o tiene simplemente codificado algo mal?

Answer 1

porque primero sintácticamente referencia @@base en el contexto de la clase Object, que es una variable de clase de objeto y todas las subclases de TemplX objeto hacen referencia a la clase var de la superclase. Puede cambiar su código para usar Module#class_variable_set y class_variable_get para evitar el enlace en la superclase.

Algunos otros problemas con su código: Noto que no hizo make_templ un par de método de clase de self.const_missing, aunque se envió correctamente porque Object es un antecesor de Class. Lo mejor es evitar todas las formas de eval (cadena) cuando existen otros métodos. No deberías subir NameError si no manejas el const_missing, sino más bien enviarlo a super como alguien más puede estar en la cadena y quieres hacer algo para resolver la constante.

class Object 
    def self.const_missing(name) 
    if name =~ /^Templ(\d+)$/ 
     return make_templ $1.to_i 
    end 
    super 
    end 

private 
    def self.make_templ(base) 
    klass_name = "Templ#{base}" 
    unless const_defined? klass_name 
     klass = Class.new(Object) do 
     class_variable_set :@@base, base 
     def initialize 
      puts "Inited with base == #{self.class.class_variable_get(:@@base)}" 
     end 
     end 
     const_set klass_name, klass  
    end 

    const_get klass_name 
    end 
end 

Las variables de clase tienen propiedades de mezcla de información interesantes ya menudo indeseables a través de la herencia. Has golpeado a uno de los gotchas. No sé qué otras propiedades necesita alrededor de @@base, pero parece que obtendrá un mejor aislamiento y resultados menos sorprendentes utilizando una variable de instancia de clase en su lugar. Para más explicaciones: Fowler, RailsTips

Answer 2

El comentario de @Casper ayuda a señalar por qué su código no funciona. Para una solución, considere usar variables de instancia de clase en lugar de variables de clase. Esto debería ayudar a evitar tener que eval y esquivar las trampas comunes de la utilización de variables de clase:

---

EDIT: añade refactorización de @dbenhur, el cambio de variable de clase a instancia de clase variable.

class Object 
    def self.const_missing(name) 
    name =~ /^Templ(\d+)$/ ? make_templ($1.to_i) : super 
    end 

private 
    def self.make_templ(base) 
    klass_name = "Templ#{base}" 
    if const_defined? klass_name 
     const_get klass_name 
    else 
     klass = Class.new(Object) do 
     class << self 
      attr_accessor :base 
     end 
     self.base = base 
     def initialize 
      puts "Inited with base == #{self.class.base}" 
     end 
     end 
     const_set klass_name, klass  
    end 
    end 
end 

puts Templ1.new.class.base 
# => Inited with base == 1 
# => 1 
puts Templ2.new.class.base 
# => Inited with base == 2 
# => 2 
puts Templ1.new.class.base 
# => Inited with base == 1 
# => 1