Rails 4 中的自定义 rake 任务问题

Question

我正在尝试创建一个 Rake 任务，该任务将在一个月内未更新列表但似乎无法正常工作时发送电子邮件。

我认为问题可能是我正在尝试提取 listing_agent 电子邮件，但其中有 listing_agent_id。

update_listing_mailer.rb：

class UpdateListingMailer < ActionMailer::Base
  default from: "Help <help@realestate.com>"

  def listing_update_agent(agent)
    @agent = agent
    @listing = listing
    mail to: "#{agent.email}", subject: "Your listing hasn't been updated in a month!"
  end
end

listing_update_agent.html.erb：

Hiya, <br><br>

The listings below have not been updated in a month! Is it still on the market? If so, please update. If not, please remove from the market.<br><br>

<strong>Address:</strong> <%= @listing.address %>
<strong>Last Updated:</strong> <%= @listing.updated_at %>
<br><br>

Thanks,<br>
Management

listing_needs_update.rake：

namespace :listings do
  desc "Sends an email to the listing agent when that listing hasn't been updated in a month"
  task listing_update_agent: :enviroment do
    Listing.all.each do |listing|
      if listing.updated_at == Date.today - 30
        UpdateListingMailer.listing_update_agent(listing.listing_agent_id).deliver_now
      end
    end
  end
end

错误：

追踪：

Answer 1

我会在这里回答以使其更清楚。

你的任务被命名为listing_update_agent，这意味着你应该调用它：

rake listings:listing_update_agent

而不是：

rake listings:listing_needs_update

我还可以考虑对您的代码进行一些改进：

1) 任务做事，因此将您的任务称为 notify_outdated_listings

2) 获取需要通知的列表并仅处理这些列表会更有效，而不是像在任务中那样询问表中的每条记录。 您可以通过在列表模型中创建一个范围来实现这一点，例如：

class Listing < ActiveRecord::Base
  scope :not_updated_in_last_month, -> { where('updated_at >= ?', 1.month.ago) } 
end

然后在你的任务中：

namespace :listings do
  desc "Sends an email to the listing agent when that listing hasn't been updated in a month"
  task listing_update_agent: :enviroment do
    Listing.not_updated_in_last_month.each do |listing|
      UpdateListingMailer.listing_update_agent(listing.listing_agent_id).deliver_now
    end
  end
end

您能说出这种变化可能带来的性能差异吗？