C# - 对象是否应该在更改状态等内容时负责创建历史对象？

Question

这更像是一个架构/最佳实践问题，所以请随意添加您的两分钱。我知道我在标题中说明了状态，但这适用于对象的任何基本属性。我认为下面的帐户示例比状态更有助于证明我的问题。

这是一个示例帐户对象:

public class Account
{
   private IList<Transaction> _transactions;

   public AddTransaction(trans as Transaction)
   {
      _transaction.add(trans)
   }
}

现在假设我想开始保留每次使用此对象添加交易的历史记录。

public class AccountHistory
{
   private DateTime _historyDate;
   private String _details;

   public AccountHistory(string details)
   {
      _historyDate = DateTime.Now;
      _details = details;
   }
}

在这个级别，我通常会做的是向帐户对象添加一个历史事件集合，并添加一行代码以在 AddTransaction() 方法内创建一个历史事件，如下所示

public AddTransaction(trans as Transaction)
{
   _transaction.add(trans);
   **_historyEvents.add(new AccountHistory("Transaction Added: " + trans.ToString());**
}

现在下一部分是问题开始出现的地方。假设我想做一个批量发布，并且我想保留在这个批量发布中更改了哪些帐户的记录，以便像报告之类的东西，或者如果我以后需要撤消它。所以我会创建一个这样的对象。

public class HistoryGroup()
{
   private IList<AccountHistory> _events;
}

从这里我看到了几个不同的选项来处理这个问题，因为上面的示例代码无法处理它。

1) 在服务类型对象中创建一个函数，该函数循环访问调用 AddTransaction() 方法的帐户列表，并创建绑定(bind)到 HistoryGroup 的历史记录

 public void AddTransactions(IList<Account> accounts, Transaction trans)
    {
       HistoryGroup history = new HistoryGroup(); 
       for (int x=0;x <=accounts.Count - 1; x++)
       {
         accounts(x).AddTransaction(trans);
         history.AddEvent(new AccountHistory("Added Transaction: " + trans.ToString();
       }
    }

2) 将某种类型的 HistoryManager 对象与要添加的事务一起传递到 AddTransaction 方法中。然后该函数可以使用历史管理器来创建记录。

好的，这篇文章够长了。如果我还不够清楚，请告诉我。感谢您的输入。

Answer 1

您的方法可能工作得很好，但让我提出一个替代方案。

为什么不向 Account 类添加 TransactionAdded 事件。

然后您可以从(我在这里猜测的)HistoryGroup 对象订阅事件，这样每次触发事件时都会添加一个新的 AccountHistory 对象。

更新

如评论中所述，实现目标的另一种方法是让 HistoryGroup 实现一个接口(interface)(ITransactionLogger 或类似的东西)，然后修改 Account 以便可以注入(inject) ITransactionLogger 依赖项。

从复杂性和调试的角度来看，走这些 route 的任何一条都会使事情更容易管理，但不允许像事件这样的多个记录器。

这将使您的代码更加灵活，同时允许对 TransactionAdded 事件感兴趣的其他消费者进行订阅。