整洁架构下领域聚合根对象属性的正确更新方式

你的整体分层架构和大部分领域建模思路符合DDD与整洁架构的规范，但存在一处核心的聚合边界违反问题，以及几处细节实现bug，具体拆解如下：

符合规范的部分 #

• 分层依赖方向正确：领域层内聚DTO接口、领域模型、工厂、自定义异常等核心组件，基础设施层实现领域层定义的DTO接口，应用层依赖领域层的抽象接口（仓储、工厂）完成流程编排，完全遵循整洁架构「依赖指向内层」的要求，没有出现领域层依赖外层组件的问题。
• 领域对象封装思路正确：所有领域对象通过带参构造完成初始化校验，属性采用private set避免外部随意修改，通过显式的行为方法修改对象状态，新建聚合根时使用工厂模式封装复杂创建逻辑，这些都是DDD的标准实践。
• 用例1的更新逻辑符合要求：通过仓储加载聚合根后，调用聚合根上暴露的UpdateBillingAddress/UpdateShippingAddress这类显式方法修改属性，没有直接绕过领域对象修改数据，符合聚合封装的要求。

存在的问题 #

1. 核心问题：破坏了聚合根的访问边界 #

你在用例2中直接调用buyerAggregate.Account.ActivateAccount()的写法违反了DDD聚合的核心规则：

聚合根是外部访问聚合内部所有实体的唯一入口，外部对象不允许直接调用聚合内部非根实体的方法。

Account作为Buyer聚合内部的实体，所有对它的操作都应该由Buyer聚合根统一转发和管控。直接绕开聚合根调用内部实体方法，会导致聚合层面的业务规则无法统一收口：后续如果要加「激活账户必须校验买家已完成实名认证」「账户状态变更后发送对应领域事件」「记录账户状态变更日志」这类跨实体的规则，你没法保证所有调用方都不会绕过校验逻辑，很容易出现脏数据。

2. 细节实现问题 #

• Account类的RemoveRole方法逻辑写反了：当前判断是if (role == null)才执行删除，实际应该是找到对应角色（role != null）时才删除，否则会抛出空引用异常。
• 领域行为方法缺少业务校验：不管是Buyer的地址更新方法，还是Account的状态变更方法，目前都只做了简单的赋值，没有前置校验（比如已注销的账户不能再激活、地址不能为无效值），把本该内聚在领域层的校验逻辑丢给了应用层，容易出现规则漏判。
• 注意不要滥用工厂：工厂只负责全新聚合根的创建，从数据库加载已有聚合的重建逻辑应该由仓储层负责，不要用工厂处理已存在聚合的重建，避免新建和重建的逻辑耦合。

修正方案 #

1. 把Account的状态变更入口收回到Buyer聚合根上，示例代码：

   // 在Buyer类中新增方法
   public void ActivateAccount()
   {
       // 可以在这里加聚合层面的前置校验，比如判断买家基础信息是否完整
       if (Name == null) throw new InvalidOperationException("买家姓名未完善，无法激活账户");
       Account.Activate();
       // 统一在这里添加领域事件，不需要每个调用方自己记着加
       AddDomainEvent(new BuyerAccountActivatedEvent(Id));
   }
   

   同时把Account类的状态修改方法访问级别调整为internal，或者改成仅Buyer类可访问，禁止应用层直接调用。
2. 给所有领域行为方法补上对应的业务校验，把和领域状态相关的规则全部内聚在领域层，不要让应用层承担领域逻辑判断。
3. 修复RemoveRole方法的判断逻辑bug。

内容的提问来源于stack exchange，提问作者Yuvaraj Velmurugan