c# - 从 Active Record 转移到领域对象模式方法

Question

场景如下:

我加入了一个团队，该团队正开始开发一个相当大的金融系统，并且据说以 DDD 为目标，具有丰富的行为领域模型。但是，我可以在项目的当前区域看到许多指向 Active Record 的气味:应用程序服务中的单个“Save”方法；一组相当贫乏的领域对象，它们有一个大的“验证”方法等。所有这些都非常像 CRUD，并且变得越来越难以管理。

我正在开发的用户故事大致翻译为“当直接借记从特定状态转变为另一种状态(未预算到预算)时，创建一个任务项目供顾问处理”。有多种规则可以确定“未列入预算”和“已列入预算”的含义。

目前 DirectDebit 类根本没有“状态”的概念，它是一个相当大的 DTO(有几个子对象——同样，所有 DTO)，有大约 1000 行验证逻辑来​​尝试确定什么它应该处于的形状。企业对直接借记可以处于的不同状态有“清晰”的概念，我相信该领域足够丰富，可以保证领域模型。

DirectDebit 应用程序服务有一个“保存”方法，它传递更新后的直接借记(从 UI)。该方法从存储库中获取现有的 DirectDebit，完全忽略它，对更新的直接借记对象运行“验证”，并将其保存到存储库中。然后该方法审核新旧直接借记对象之间的变化。

我正试图摆脱这里开始发展的类似 CRUD 的怪物，并向团队展示他们如何开始重构它。

理想情况下，我想开始重构领域对象以使用状态模式，使用良好的封装，也许使用领域事件来冒泡状态已经改变的事实。应用层将有一个名为“CreateTaskWhenBudgeted”或类似名称的领域事件处理程序来解耦逻辑。

我的问题是来自 UI 的大型“保存”直接借记方法。

是否有分解/重构这种方法的策略？没有简单的方法来判断什么是“改变的”，因为没有明确的命令正在发生。或者，我是否应该继续增加困惑而忽略我认为是问题的东西......？

这是应用程序服务的简化版本:

public void SaveDirectDebit(DirectDebit directDebit)
{
    directDebit.Validate(false);

    var beforeDirectDebit = _directDebitRepository.FetchDirectDebit(directDebit.ExpenditureId.Id);

    _directDebitRepository.SaveDirectDebit(directDebit);

    var afterDirectDebit = _directDebitRepository.FetchDirectDebit(directDebit.ExpenditureId.Id);

    var accountAuditLog = CreateDirectDebitAudit(beforeDirectDebit, afterDirectDebit);

    _auditLogRepository.CreateAudit(accountAuditLog);
}

Answer 1

当然没有一种方法可以做到(好方法与其他方法)，您有多个参数(时间、复杂性...)。

由内而外的方式

免责声明:遗留代码越庞大，这种方法似乎就越难以实现(对我来说成功的条件:你不应该是唯一关心这种重构的人)。

-> 保存到数据库是个技术活儿。你必须首先引入一些通用语言来使隐含显式，并且能够引入概念和进行设计(创建边界和分离上下文)。

另一种方式

这是我在评论中开始描述的另一种方式，对我来说，当你被遗留系统包围时，它与 Eric Evans 所说的“气泡上下文”有关，从一个干净的小有界上下文开始会更容易。这种隔离将帮助您应用比现有系统更好(不同)的设计。