Rails 2.3 应用程序中代码的重构

Question

我有一个相当简单的 Rails 应用程序，允许用户管理他们的客户端并为每个客户端创建一个案例。创建案例后，详细信息将发送到两个电子邮件地址（每封电子邮件中的内容不同），并使用 gem 将信息发送到 FreeagentCentral。

我已经实现了 delayed_job，它对于 Freeagent API 调用运行良好，但我认为可能有更好的，较低的开销，发送电子邮件的方法。似乎需要相当一段时间。

目前，我的 Rails 2.3 应用程序中有以下代码。

kases_controller.rb

# POST /kases
# POST /kases.xml
  def create
    @company = Company.find(params[:kase][:company_id])
    @kase = @company.kases.new(params[:kase])

    if @kase.save
        UserMailer.deliver_makeakase("[email protected]", "Highrise", @kase) if params[:sendtohighrise]
        UserMailer.deliver_makeakaseteam("[email protected]", "Highrise", @kase) if params[:notify_team_of_creation]
        @kase.delay.create_freeagent_project(current_user) if params[:send_to_freeagent]

        redirect_to(@kase)

        #flash[:notice] = 'Case was successfully created.'
        flash[:notice] = fading_flash_message("Case was successfully created.", 5)
      else
        render :new
      end        
  end

user_mailer.rb

def makeakase(email, name, kase, bccemails = [])
      recipients email
      from "[email protected]"
      subject "FW: Case creation from Survey Manager"
      bcc bccemails
      sent_on Time.now
      body :name => name, :kase => kase
  end

  def makeakaseteam(email, name, kase = [])
      recipients email
      from "[email protected]"
      subject "A new case has been created."
      sent_on Time.now
      body :name => name, :kase => kase
      content_type "text/html"
  end

我正在寻找有关减少发送电子邮件数量的建议，或者将电子邮件添加到延迟作业的方法。

我使用 PostmarkApp 来发送电子邮件，这是另一个宝石。详细信息可以在这里找到：postmark-gem

Answer 1

不确定这是否是您想要的，但我会制作一种方法来发送电子邮件和创建自由代理，然后让它由delayed_job处理。

的东西可能会更简单/更干净。

class Kase

  def send_emails(current_user, send_to_highrise, notify_team, send_to_freeagent)
    UserMailer.deliver_makeakase("[email protected]", "Highrise", self) if send_to_highrise
    UserMailer.deliver_makeakaseteam("[email protected]", "Highrise", self) if notify_team
    self.create_freeagent_project(current_user) if send_to_freeagent
  end
end

然后，在您的控制器中编写

if @kase.save
  @kase.delay.send_emails(current_user, params[:sendtohighrise], 
                         params[:notify_team_of_creation], params[:send_to_freeagent])
  redirect_to @kase
  flash[:notice] = fading_flash_message("Case was successfully created.", 5)
else
  ...

或传递 params 之类

Answer 2

我已成功使用 Starling 和 Workling 解决类似问题。我不确定这是否是一个较低的开销，因为我个人从未使用过 delayed_job 。您可能想探索一下。

Answer 3

安装delayed_job后，对于旧版本，您应该能够简单地执行以下操作。

UserMailer.send_later(:deliver_makeakase, "[email protected]", "Highrise", @kase) if params[:sendtohighrise]
UserMailer.send_later(:deliver_makeakaseteam, "[email protected]", "Highrise", @kase) if params[:notify_team_of_creation]

对于从 2.0.4 开始的版本，语法有点不同。

UserMailer.delay.deliver_makeakase("[email protected]", "Highrise", @kase) if params[:sendtohighrise]
UserMailer.delay.deliver_makeakaseteam("[email protected]", "Highrise", @kase) if params[:notify_team_of_creation]