Rails Mailer 最佳实践 - 控制器或 after_create 回调？ [关闭]

Question

在常见的 Ruby on Rails 3/4 应用程序中，有两个选项可以在创建模型后传递邮件：

1) 直接从控制器的创建操作发送邮件

#listings_controller.rb

def create
    @listing.create(params[:listing])
    if @listing.save
        ListingMailer.new_listing_notice(@listing).deliver
        format.html {redirect_to @listing}
        ...
    end
end

或2）通过模型回调发送邮件

#listing.rb


class Listing
    after_create :notify
    ...
    def notify
        ListingMailer.new_listing_notice(self).deliver
    end
end

目前是否就哪种方式更好达成共识？从控制器发送可以提供更多控制权，但是如果始终发送邮件，是否有任何理由不使用回调？这主要是风格问题，还是有其他重要问题？

Answer 1

通常，维护模型中具有after_* 挂钩的代码要困难得多。当然，在某些情况下，使用回调是非常合理的（例如，在某些应用程序中应该一直进行计算校验和），但这些情况是规则的例外。

在您的电子邮件示例中，以下是回调方法的一些缺点：

邮件界面和列表模型因不同原因发生变化 (Single Responsibility Principle)。

例如，您希望使用特殊队列发送电子邮件。与队列通信的接口不应该以任何方式影响列表模型的构建方式。

您的应用程序不需要一直发送电子邮件。

电子邮件只是与外界互动的一种方式。并非所有的业务逻辑都需要与外部世界联系起来。正如apneadiving 提到的那样，其中一个例子是导入。另一个例子是控制台界面（您想在使用 rails 控制台时收到电子邮件吗？）

测试很困难。

这更多是 1 和 2 的结果，但随着时间的推移，测试 after 钩子变得越来越困难。在测试列表模型时需要模拟邮件程序，这使得测试变得不清楚，并且在发生变化时更难以维护。