ruby-on-rails - validates - ruby callbacks

Rails 3: ¿Debería guardar explícitamente un objeto en una devolución de llamada after_create?

Question

Código relevante: http://pastebin.com/EnLJUJ8G

class Task < ActiveRecord::Base
  after_create :check_room_schedule

  ...

  scope :for_date, lambda { |date| where(day: date) }
  scope :for_room, lambda { |room| where(room: room) }

  scope :room_stats, lambda { |room| where(room: room) }
  scope :gear_stats, lambda { |gear| where(gear: gear) } 

  def check_room_schedule
    @tasks = Task.for_date(self.day).for_room(self.room).list_in_asc_order
    @self_position = @tasks.index(self)

    if @tasks.length <= 2
      if @self_position == 0 
        self.notes = "There is another meeting in 
    this room beginning at # {@tasks[1].begin.strftime("%I:%M%P")}."
        self.save
      end
    end
  end

  private

    def self.list_in_asc_order
      order(''begin asc'')
    end
end

Estoy haciendo una pequeña aplicación de tareas. Cada tarea está asignada a una habitación. Una vez que agregue una tarea, quiero utilizar una devolución de llamada para verificar si hay tareas en la misma sala antes y después de la tarea que acabo de agregar (aunque mi código solo maneja un caso marginal en este momento).

Así que decidí usar after_create (ya que el usuario lo comprobará manualmente si lo editan, por lo tanto no after_save) para poder usar dos ámbitos y un método de clase para consultar las tareas del día, en la sala, y ordenarlas por hora. Luego encuentro el objeto en la matriz y empiezo a usar sentencias if.

Tengo que guardar explícitamente el objeto. Funciona. Pero se siente raro que estoy haciendo eso. No tengo demasiada experiencia (primera aplicación), así que no estoy seguro de si esto está mal visto o si es convencional. He buscado un montón y he revisado un libro de referencia, pero no he visto nada tan específico.

Gracias.

Answer 1

Esto parece una tarea para before_create para mí. Si tiene que guardar en su devolución de llamada after_* , es probable que desee utilizar una devolución de llamada before_* lugar.

En before_create no before_create que llamar a save , ya que el guardado ocurre después de que el código de devolución de llamada se ejecute por ti.

Y en lugar de guardar y luego consultar para ver si obtienes 2 o más objetos, deberías consultar un objeto que chocará antes de guardar.

En código psuedo, lo que tienes ahora:

after creation
  now that I''m saved, find all tasks in my room and at my time
  did I find more than one?
    Am I the first one?
      yes: add note about another task, then save again
      no: everything is fine, no need to re-save any edits

Lo que deberías tener:

before creation
  is there at least 1 task in this room at the same time?
    yes: add note about another task
    no: everything is fine, allow saving without modification

Algo más como esto:

before_create :check_room_schedule
def check_room_schedule
  conflicting_task = Task.for_date(self.day)
                         .for_room(self.room)
                         .where(begin: self.begin) # unsure what logic you need here...
                         .first
  if conflicting_task
    self.notes =
      "There is another meeting in this room beginning at #{conflicting_task.begin.strftime("%I:%M%P")}."
  end
end