ruby-on-rails - 延迟作业 - 销毁对象后无法为非持久记录创建作业

Question

Rails 4 和 delayed_job 4.1.2。我试图在销毁评论后延迟重新计算总体评分，但显然是因为在销毁评论对象后没有评论对象的 ID。因此，每次在尝试销毁一个对象后，它都会尝试创建一个延迟的作业，但会抛出此错误:

ArgumentError (job cannot be created for non-persisted record: 
#<Review id: 44, review: "Bad", rating: 1, reviewable_id: 2, 
reviewable_type: "Spot", user_id: 1, created_at: "2016-05-30 17:13:29", 
updated_at: "2016-05-30 17:13:29">):
  app/controllers/reviews_controller.rb:40:in `destroy'

我有以下代码:

# reviews_controller.rb
class ReviewsController < ApplicationController
  def destroy
    review.destroy
    flash[:success] = t("reviews.destroy.success")
  end
end

# review.rb
class Review < ActiveRecord::Base
  after_destroy :calculate_overall_rating

  def calculate_overall_rating
    if number_of_reviews > 0
      reviewable.update_attribute(:overall_rating, overall_rating)
    else
      reviewable.update_attribute(:overall_rating, 0)
    end
  end
  handle_asynchronously :calculate_overall_rating
end

值得注意的是，calculate_overall_rating 不需要Review 对象。

如果我删除 handle_asynchronously :calculate_overall_rating 它将起作用，并重新计算。但我正试图推迟这项工作。

Answer 1

这个错误确实是raised当您尝试延迟已删除(或尚未创建)记录的方法时，由 delayed_job 执行。这个错误的直接原因是因为 delayed_job passes self (即刚刚删除的评论对象)作为调用handle_asynchronously方法时的目标对象。我不知道为什么会这样，我刚找到一个 statement来自 gem 的一位作者，他说它的工作方式与 ActiveJob 相同。

无论如何，在我看来，您可能在错误的地方定义了重新计算方法。如果我理解正确的话，在销毁评论后，将根据给定的 reviewable(例如 spot)的所有评论重新计算平均评分。对我来说，这样的代码被定义为单个审查实例的方法似乎很奇怪。单个评论(更何况是已删除的评论)不应该对同一地点的其他评论一无所知，根本不必与它们打交道。

我猜方法应该定义为类方法，并将reviewable 作为参数。当然，这意味着您还必须制作其他计算方法 overall_rating 和 number_of_reviews，类方法。但我再次认为这是一件好事，因为这些方法的领域位于单个评论之外。类似于以下内容:

# review.rb
class Review < ActiveRecord::Base
  after_destroy :recalculate_overall_rating

  def recalculate_overall_rating
    self.class.calculate_overall_rating(reviewable)
  end

  def self.calculate_overall_rating(reviewable)
    if number_of_reviews(reviewable) > 0
      reviewable.update_attribute(:overall_rating, overall_rating(reviewable))
    else
      reviewable.update_attribute(:overall_rating, 0)
    end
  end
  handle_asynchronously :calculate_overall_rating
end

另一种选择(我想我更喜欢它)是将重新计算方法放在可审查类中，例如在 Post 类中。如果您有更多可审查的类类型，您可以制作一个包含所有这些类的模块，例如Reviewable(不过我希望这不会与 Rails 关联名称冲突)并将重新计算方法放入其中，这次作为实例 方法。为什么？因为它是可评论的实例，它希望重新计算所有评论，并且即使在评论删除后它仍然存在，因此它可以很容易地异步运行。类似于以下内容:

# reviewable.rb
module Reviewable
  def calculate_overall_rating
    if number_of_reviews > 0
      update_attribute(:overall_rating, overall_rating)
    else
      update_attribute(:overall_rating, 0)
    end
  end
  handle_asynchronously :calculate_overall_rating

  # overall_rating and number_of_reviews are also defined in this module
end

# review.rb
class Review < ActiveRecord::Base
  after_destroy :recalculate_overall_rating

  def recalculate_overall_rating
    reviewable.calculate_overall_rating
  end
end

# post.rb
class Post < ActiveRecord::Base
  include Reviewable
end