problema que soporta dos sintaxis forma en rubí

Question

Tengo una situación donde tengo que llamar a algo como esto:

class Office 

    attr_accessor :workers, :id 

    def initialize 
    @workers = [] 
    end 

    def workers worker 
    type = worker.type 
    resp = Worker.post("/office/#{@id}/workers.json", :worker => {:type => type}) 
    worker = Worker.new() 
    resp.to_hash.each_pair do |k,v| 
     worker.send("#{k}=",v) if worker.respond_to?(k) 
    end 
    self.workers << worker 
    end 

end 

clase Trabajador

class Worker 
    attr_accessor :office_id, :type, :id 

    def initialize(options={}) 
    @office_id = options[:office].nil? ? nil : options[:office].id 
    @type = options[:type].nil? ? nil : options[:type].camelize 
    if !@office_id.nil? 
     resp = self.class.post("/office/#{@office_id}/workers.json", :worker => {:type => @type}) 
     @id = resp.id 
     office = options[:office] 
     office.workers = self 
    end 
    end 

    def <<(worker) 
    if worker 
     type = worker.type 
     resp = Worker.post("/office/#{office_id}/workers.json", :worker => {:type => type}) 
     debugger 
     @id = resp.id 
     resp.to_hash.each_pair do |k,v| 
     self.send("#{k}=",v) if self.respond_to?(k) 
     end 
     debugger 
     return self 
    end 
    end 

puedo hacer algo como esto muy bien

office = Office.new() 
new_worker = Worker.new() 
office.workers new_worker 

Pero tengo que hacer lo mismo que he hecho anteriormente como el siguiente. Antes de eso, necesito cambiar el método de inicialización de Office para iniciar el método def <<(worker) de la instancia de trabajador.

class Office 
    ... 
    def initialize 
    @workers = Worker.new 
    @workers.office_id = self.id 
    end 


office = Office.new() 
new_worker = Worker.new() 
office.workers << new_worker 

¿Ahora el problema es que la implementación posterior crea 2 instancias del trabajador?

Answer 1

No estoy 100% seguro de por qué no está obteniendo un error aquí, pero como Office # workers last line es self.workers < < trabajador, está agregando el nuevo trabajador creado en Office # workers (hecho en la tercera línea del método), y luego devolver el objeto de los trabajadores, que luego obtiene # < < llamado nuevamente con el nuevo trabajador creado fuera del método

Answer 2

No estoy del todo seguro, pero supongo que usted ' d gustaría tener esto:

class Office 

    attr_accessor :workers, :id 

    def initialize 
    @workers = [] 
    end 

    alias_method :workers, :return_worker_array 

    def workers worker 
    unless worker 
     return_worker_array 
    else 
     type = worker.type 
     resp = Worker.post("/office/#{@id}/workers.json", :worker => {:type => type}) 
     worker = Worker.new() 
     resp.to_hash.each_pair do |k,v| 
     worker.send("#{k}=",v) if worker.respond_to?(k) 
     return_worker_array << worker 
    end 
end 

final

De esta manera usted puede deshacerse de Worker#<< por completo y también se debe quitar la línea

office.workers = self 

en Worker#initialize desde office.workers se supone que es una matriz. Es una mala idea cambiar el tipo de un atributo (el tipado de pato estaría bien) de ida y vuelta porque es probable que pierda la noción del estado actual y tarde o temprano se encontrará con errores.

Para seguir "Separación de preocupaciones", recomendaría hacer toda la administración de workers únicamente en Office, de lo contrario, se vuelve demasiado confuso demasiado rápido y será mucho más difícil de mantener a largo plazo.