3

我正在寻找有关如何解决以下设计问题的建议（使用基于 stackoverflow 的虚构示例）。我试图避免一个贫乏的领域模型，并为这种类型的案例寻求一般的“最佳实践”建议。

设想：

假设正在为 stackoverflow 开发一个新功能，只要他/她的问题收到 10 个赞成票，就会向问题的所有者发送电子邮件通知。

域对象模型是这样的：

public class Question
{
    string Question { get; set; }
    IList<Votes> Upvotes { get; set; }
    User Owner { get; set; }

    public void AddUpvote(Vote upvote)
    {
        Upvotes.Add(upvote);
    }
}

潜在的实现：

  1. 更改AddUpvote()以获取IEmailerService参数并执行AddUpvote()方法内的逻辑。

    public void AddUpvote(Vote upvote, IEmailerService emailer)
{
    Upvotes.Add(upvote);
    if ( Upvotes.Count == 10 )
    {
        emailer.Send(Owner.EmailAddr);
    }
}

  2. 在内部检测此状态AddUpvote()并AddUpvote()从 IoC 容器解析 IEmailService（而不是将 IEmailerService 作为参数传递）。

  3. 在调用 的外部服务对象中检测此状态question.AddUpvote()。

    public void UpvoteClickHandler(Question question)
{
    question.AddUpvote(new Upvote());
    if ( question.Upvotes.Count == 10 )
    {
        _emailer.Send(question.Owner.EmailAddr);
    }
}

  4. 你更好的解决方案在这里！

4

4 回答 4

5

您真的不想将这两者混合在一起，因为它们有不同的关注点。让 Question 类关心问题，让消息服务关心当投票达到 10、20、100 或...时该怎么做

以下示例仅用于演示目的，但您会明白这一点。有明确的关注点分离，因此如果发送消息的要求发生变化，Question 类不必更改。请记住，根据 SOLID 原则，一个类应该只有一个改变的理由。

public class Question
{
    public string Description { get; set; }
    public Int32 Votes { get; set; }
    public User Owner { get; set; }

    public event EventHandler<QuestionEventArgs> OnUpvote;

    private void RaiseUpvoteEvent(QuestionEventArgs e)
    {
        var handler = OnUpvote;
        if (handler != null) handler(this, e);
    }

    public void Upvote()
    {
        Votes += 1;

        RaiseUpvoteEvent(new QuestionEventArgs(this));
    }
}

public class MessageService
{
    private Question _question;

    public MessageService(Question q)
    {
        _question = q;

        q.OnUpvote += (OnUpvote);
    }

    private void OnUpvote(object sender, QuestionEventArgs e)
    {
        if(e.Question.Votes > 10)
            SendMessage(e.Question.Owner);
    }
}

public class QuestionEventArgs: EventArgs
{
    public Question Question { get; set; }

    public QuestionEventArgs(Question q)
    {
        Question = q;
    }
}

所以你有它。有很多其他方法可以实现这一点，但事件模型是一个很好的方法，它实现了您在实现中想要的关注点分离，以便更早地进行维护。

于 2009-05-29T23:52:47.633 回答
2

选项 1) 和 2) 都作为发送电子邮件的错误位置而跳出来。Question 实例不应该知道这两件事：

  1. 它不应该知道策略，即何时发送电子邮件。
  2. 它不应该知道策略的通知机制，即电子邮件服务。

我知道这是一个品味问题，但是您将问题与政策以及发送电子邮件的机制紧密联系在一起。将这个 Question 类移动到另一个项目（例如 ServerFault，它是 StackOverflow 的姊妹站点）真的很难

我对这个问题很感兴趣，因为我正在为我正在构建的帮助台创建一个通知系统。这是我在我的系统中所做的：

创建一个 NotificationManager（基本上，将通知的关注点完全移到一个单独的类中）。

public Class NotificationManager
{
    public void NotificationManager(NotificationPolicy policy, IEmailService emailer)
    {
    }
}

然后我按照这个思路做了一些事情（UpvoteClickHandler 依赖于 NotificationManager 实例）：

public void UpvoteClickHandler(Question question)
{
    question.AddUpvote(new Upvote());
    _notificationManager.Notify(Trigger.UpvoteAdded, question);
}

UpvoteClickHandler 所做的就是告诉 NotificationManager 已向问题添加了赞成票，并让 NotificationManager 确定是否以及如何发送电子邮件。

于 2009-05-29T23:39:37.753 回答
1

大家好，

在我看来，“每当他/她的问题收到 10 个赞成票时，向他/她的问题的所有者发送电子邮件通知”是域逻辑，因此它应该进入域对象，以避免贫血域。

必须进入基础设施层的是发送电子邮件（即与 smtp 服务器通信）的操作。

所以我认为选项1并非完全错误。请记住，您始终可以通过传递 IEmailerService 的模拟实现来测试您的对象。

此致，

斯特凡诺

于 2009-07-28T11:03:20.917 回答
1

答案取决于您对应用程序和对象设计的基本方法。并且（在此处编辑）您认为系统最重要的特征。看起来你有数据、问题和业务规则，赞成票。根本不质疑对象。因此，您应该将数据视为数据并允许数据工具对其进行处理，而不是将行为混入其中。传统的对象设计将所有行为和数据都包含在对象中，因此发送电子邮件将属于对象。（选项 1 和 2）我想这是黑匣子或自包含对象方法。正如我所了解的那样，现代实践将对象作为简单的数据持有者。这意味着要四处移动，持久化，转换并对其进行处理。也许生活在 C 语言中只是结构。

于 2009-05-29T23:48:40.403 回答