java - 如何处理 DDD 聚合内的 getOrCreateIfAbsent？

Question

我的需求中有一个情况，我想看看是否有人对此有强烈的意见。

我正在开发的项目要求，给定一个客户，一旦客户将产品添加到购物车，如果购物车不存在，则必须创建购物车。目前，聚合是客户，它包含包含产品的购物车。因为该系统是支持真正的电子商务项目的辅助系统，所以一旦收到“AddProductCommand”，客户和购物车的创建就被认为是理所当然的。在这种情况下，如果客户和购物车尚不存在，则必须创建它们。

如果客户不存在，我当前的实现会在应用程序服务中创建客户。创建后，我使用 customer.addProduct(addProductRequest) 传递 AddProductRequest。 AddProductRequest 包含购物车 id 和产品 id。问题就来了。一旦进入客户聚合，如果购物车不存在，我必须创建它。所以基本上，我在 customer 中没有 addCart 以及我首先调用然后调用 addProduct 的购物车实体。如果客户聚合内部不存在购物车，我需要创建购物车，然后将产品添加到其中。为此，我不创建任何工厂，因为我不想有一个使测试复杂化的静态方法，而且我也不能只在聚合内创建一个新工厂。我只是在 protected 方法中使用 new 运算符创建购物车实体，在测试以验证我在那里做什么时会重写该方法。

所以我的问题是，这是一种有效的方法吗？在我看来，要将产品添加到购物车，应该首先将购物车添加到客户中，如果不存在则失败。为此，我需要向应用程序服务添加逻辑，首先我会询问客户是否有具有该 id 的购物车，否则创建它，然后在添加产品之前将其添加到客户。我可以添加一个给出此请求的域服务，但随后我需要注入(inject)工厂来创建购物车，当我在几个地方读到域服务不应注入(inject)任何工厂时，应该是应用服务的工作。

我可以这样做，但项目将来会变得更加复杂，因为还会有另一层，可以在其中添加产品的优惠券列表，如果不存在，还必须创建客户、购物车和产品当使用 AddVoucherCommand 时。在这种情况下，如果我不想在模型内创建实体，我需要在应用程序/域服务中检查每个聚合或实体内部是否具有必要的实体，我认为这不是很 DDD友好，或者继续做我现在正在做的事情。也就是说，每个聚合/实体负责在调用 addXXXX 方法之前创建必要的实体。

一些简化的代码来解释我现在正在做什么以及将来必须做什么:

public class CustomerService {
    public void addVoucher(AddVoucherRequest addVoucherRequest) {
        Customer customer = customerRepo.load(customerId);
        customer.addVoucher(addVoucherRequest);
        customerRepo.save(customer);
    }
}

public class Customer() {
    public void addVoucher(AddVoucherRequest addVoucherRequest) {
        Cart cart = getOrCreateIfAbsent(addVoucherRequest.getCartId());
        cart.addVoucher(addProductRequest);
    }
    private Cart getOrCreateIfAbsent(long cartId){
        Optional<Cart> cart = carts.stream().filter(cart -> cart.getId() == cartId).findFirst();
        return cart.orElseGet(() -> {
             Cart newCart = createCart(cartId);
             carts.add(newCart);
             return newCart;
        }
     }
     protected Cart createCart(long cartId) {
         return new Cart(cartId);
     }
}
public class Cart() {
     public void addVoucher(AddVoucherRequest addVoucherRequest) {
         Product product = getOrCreateIfAbsent(addVoucherRequest.getProductId());
         product.addVoucher(addVoucherRequest);
     }
     private void getOrCreateIfAbsent(long productId) {
         Optional<Product> product = products.stream().filter(product -> product.getId() == productId).findFirst();
        return product.orElseGet(() -> {
             Product newProduct = createProduct(productId);
             products.add(newProduct );
             return newProduct ;
        }
     }
     protected Product createProduct(long productId) {
         return new Product(productId);
     }
 }

有什么建议吗？

谢谢!

Answer 1

选择等待客户订购某些内容以在新系统中创建他/她，而不是预先从旧系统导入所有客户是一个纯粹的技术决策，在我看来，它与您的域无关。

因此，它不应该以任何方式影响您设计域对象的方式 - 据我了解，getOrCreateIfAbsent() 应该是某种应用程序服务中的行为，而不是某种应用程序服务中的行为您的实体。