24

发送电子邮件通常在对模型执行操作之后调用，但电子邮件本身是视图操作。我正在寻找您如何考虑要问自己哪些问题来确定将操作邮件方法调用放在哪里。

我见过/使用过它们：

  • 在模型方法中 - 相关但单独的关注点的不良耦合？
  • 在模型的回调中（例如 after_save）——就我目前的知识水平而言，最好的分离。
  • 在控制器操作中 - 只是感觉不对，但是在某些情况下这将是构建代码的最聪明的方式吗？

如果我想知道如何编程，我需要像程序员一样思考，因此学习如何通过特定的编程解决方案进行思考值得我独自进行几个月的编码。谢谢！

4

4 回答 4

15

迟到的答案，但我想对这个主题进行合理化：

通常，在 Web 应用程序中，您希望发送电子邮件作为对客户的直接反应。或者作为一项后台任务，以防我们谈论新闻通讯/通知邮件之类的事情。

该模型基本上是一个数据存储映射器。它的逻辑应该封装数据处理/通信与数据存储处理。因此，插入与它无关的逻辑有点棘手，并且在大多数情况下是错误的。让我们举个例子：用户注册一个帐户，应该会收到一封确认电子邮件。在这种情况下，可以说，确认电子邮件是创建新帐户的直接影响。现在，尝试在控制台中创建用户，而不是在 Web 应用程序中执行此操作。在这种情况下触发回调听起来是错误的，对吧？因此，回调选项被划伤。我们还应该在模型中编写方法吗？好吧，如果它是用户操作/输入的直接影响，那么它应该保留在该工作流程中。成功创建用户后，我会将其写入控制器。直接地。无论如何，在要在控制器中调用的模型中复制此逻辑会增加不必要的模块化，以及 Active Record 模型对 Action Mailer 的依赖性。尝试考虑在许多应用程序上共享模型，其中一些应用程序不希望使用 Action Mailer。出于上述原因，我认为邮件呼叫应该是有意义的，通常模型是不是那个地方。试着给我一些例子。

于 2012-09-24T09:35:10.590 回答
4

好吧，取决于。

我已经使用了所有这些选项以及您关于“我为什么要把它放在哪里？”的观点。很好。

如果这是我希望每次以某种方式更新模型时都发生的事情，那么我将其放入模型中。甚至可能在模型中的回调中。

有时您只是在发布报告；没有任何更新。在这种情况下，我通常有一个带有发送报告的索引操作的资源。

如果邮件程序与正在更改的模型并不真正相关，我可以看到将其放入回调中。我不经常这样做。我更有可能仍将其封装在模型中。我做过，只是不经常。

于 2009-02-03T21:42:51.573 回答
2

我知道已经有一段时间了，但最佳实践永远不会消失，对吧？:)

电子邮件按照定义是异步通信（确认电子邮件除外，但即使是这封电子邮件，在必须确认之前留出延迟应该是一种最佳实践）。

因此，在我看来，最合乎逻辑的发送方式是：

  • 在后台操作中（使用Sidekiq或delay_job）
  • 在一个回调方法中：“嘿，这个动作成功完成了，也许我们现在可以告诉世界了？”

Rails 中的问题是它没有太多的回调（例如在 JS 中）：我个人觉得有这样的代码很脏：

after_save :callback

def callback
  if test_that_is_true_once_in_the_objects_life
    Mailer.send_email()
  end
end

所以，如果你真的想像程序员一样思考，那么想法是在你的应用程序中设置一些自定义回调系统。

例如。

def run_with_callback(action, callback_name)
  if send(action)
    delay.send(callback_name)
  end
end

甚至在您的应用程序中创建一个事件系统将是一个不错的解决方案。

但最终这些解决方案在时间上是相当昂贵的，所以人们最终在行动之后将其内联编写

def activate
  [...]
  user.save
  Mailer.send_mail
  respond_to 
  [...]
end

这是同步编程中最接近回调的方式，并且结果使 Mailers 到处调用（inModel和 in Controller）。

于 2014-10-16T09:27:03.680 回答
0

控制器对于邮件程序来说是一个好地方有几个原因：

  • 与模型无关的电子邮件。
  • 如果您的电子邮件依赖于几个彼此不了解的模型。
  • 将模型提取到 API 并不意味着重新实现邮件程序。
  • 邮件内容由您不想传递给模型的请求变量确定。
  • 如果您的业务模型需要大量不同的电子邮件，则模型回调可以叠加。
  • 如果电子邮件不依赖于模型计算的结果。
于 2018-02-20T18:50:51.557 回答