这更像是一个架构/最佳实践问题,所以请随意添加您的两分钱。我知道我在标题中说明了状态,但这适用于对象的任何基本属性。我认为下面的帐户示例将有助于比状态更好地展示我的问题。
这是一个示例 Account 对象:
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) 在 Service 类型对象中创建一个函数,该函数循环调用 AddTransaction() 方法的帐户列表,并创建与 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 方法中。然后该函数可以使用历史管理器来创建记录。
好吧,这篇文章够长了。如果我不够清楚,请告诉我。感谢您的输入。
最佳答案
你的方法可能工作得很好,但让我提出一个替代方案。
为什么不向 Account 类添加 TransactionAdded 事件。
然后,您可以从(我在这里猜测)HistoryGroup 对象订阅事件,以便每次触发事件时都添加一个新的 AccountHistory 对象。
更新
正如评论中提到的,实现目标的另一种方法是让 HistoryGroup 实现一个接口(interface)(ITransactionLogger 或类似的东西),然后修改 Account 以便可以注入(inject) ITransactionLogger 依赖项。
从复杂性和调试的角度来看,采用这些 route 的任何一条都可以使事情更容易管理,但不允许像事件这样的多个记录器。
这将使您的代码更加灵活,同时允许其他对 TransactionAdded 事件感兴趣的消费者订阅。
关于C# - 当一个对象改变状态时,是否应该负责创建一个历史对象?,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/3239157/