如何一步一步用DDD设计一个电商网站（五）—— 停下脚步，重新出发

本系列所有文章

阅读目录

前言
单元测试
纠正错误，重新出发
结语

一、前言

　　实际编码已经写了2篇了，在这过程中非常感谢有听到观点不同的声音，借着这个契机，今天这篇就把大家提出的建议一个个的过一遍，重新整理，重新出发，为了让接下去的DDD之路走的更好。

二、单元测试

　　蟋蟀兄在我的第三篇文章下面指出：

　　这点其实是我偷懒了，单元测试其实不单单在DDD中是一个很重要的一环，在我们崇尚敏捷，快速迭代的大背景下，有良好的单元测试模块可以保证快速迭代下的项目质量。有甚至可以使用测试先行的TDD模式。

　　单元测试的好处我就不多说了，那么现在开始在项目中增加单元测试。单元测试有多种命名方式，我个人的方式是给每一个对象单独建立一个测试类，然后里面每个单元测试方法的命名规则为"方法名_条件_预期的结果"这样子。那么根据我们之前的Cart和CartItem的建模，编写的单元测试如下：

    [TestClass]
    public class CartTest
    {
        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void Constructor_CartIdDefault_ThrowArgumentException()
        {
            var cart = new Cart(default(Guid), Guid.NewGuid(), DateTime.Now);
            Assert.AreNotEqual(null, cart);
        }

        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void Constructor_UserIdDefault_ThrowArgumentException()
        {
            var cart = new Cart(Guid.NewGuid(), default(Guid), DateTime.Now);
            Assert.AreNotEqual(null, cart);
        }

        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void Constructor_LastChangeTimeDefault_ThrowArgumentException()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), default(DateTime));
            Assert.AreNotEqual(null, cart);
        }

        [TestMethod]
        public void AddCartItem_NotExisted_TotalItemCountIsIncreased()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), DateTime.Now);
            cart.AddCartItem(new CartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100));
            Assert.AreEqual(1, cart.TotalItemCount());

            cart.AddCartItem(new CartItem(new Guid("22222222-2222-2222-2222-222222222222"), 1, 100));
            Assert.AreEqual(2, cart.TotalItemCount());
        }

        [TestMethod]
        public void AddCartItem_Existed_TotalItemCountIsNotIncreasedTotalItemNumIsIncreased()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), DateTime.Now);
            cart.AddCartItem(new CartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100));
            Assert.AreEqual(1, cart.TotalItemCount());
            Assert.AreEqual(1, cart.TotalItemNum());

            cart.AddCartItem(new CartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100));
            Assert.AreEqual(1, cart.TotalItemCount());
            Assert.AreEqual(2, cart.TotalItemNum());
        }
    }

    [TestClass]
    public class CartItemTest
    {
        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void ModifyQuantity_LessZero_ThrowArgumentException()
        {
            var cartItem = new CartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            cartItem.ModifyQuantity(-1);
        }

        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void ModifyQuantity_EqualsZero_ThrowArgumentException()
        {
            var cartItem = new CartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            cartItem.ModifyQuantity(0);
        }

        [TestMethod]
        public void ModifyQuantity_MoreZero_Success()
        {
            var cartItem = new CartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            cartItem.ModifyQuantity(10);
            Assert.AreEqual(10, cartItem.Quantity);
        }

        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void ModifyPrice_LessZero_ThrowArgumentException()
        {
            var cartItem = new CartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            cartItem.ModifyPrice(-1);
        }

        [TestMethod]
        public void ModifyQuantity_EqualsZero_Success()
        {
            var cartItem = new CartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            cartItem.ModifyQuantity(0);
            Assert.AreEqual(0, cartItem.Price);
        }

        [TestMethod]
        public void ModifyQuantity_MoreZero_Success()
        {
            var cartItem = new CartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            cartItem.ModifyQuantity(10);
            Assert.AreEqual(10, cartItem.Price);
        }
    }

三、纠正错误，重新出发

　　在写CartItemTest的时候发现了一个问题。领域对象的设计中有一个要点，就是实体必须需要通过其所属的聚合根才能访问，这样才能体现出聚合的的整体性，并且减少外界对聚合内部过多的了解。而目前对于CartItem的运用却有些背道而驰的意思，由外部对象进行实例化，必然增加了外部调用方对整个购物项构造过程的了解。有一位园友tubo有提到这点。

　　我思考了下，觉得这位园友的建议是对的。他建议的改法恰恰能够满足这个要求，隐藏了构造CartItem实体的细节。
　　好了那先把CartItem的构造函数访问类型设置为internal吧，这样也只能在CartItem所在的Domain项目中进行实例化了，然后再修改Cart.AddCartItem方法的参数。变为如下：

        public void AddCartItem(Guid productId, int quantity, decimal price)
        {
            var cartItem = new CartItem(productId, quantity, price);
            var existedCartItem = this._cartItems.FirstOrDefault(ent => ent.ProductId == cartItem.ProductId);
            if (existedCartItem == null)
            {
                this._cartItems.Add(cartItem);
            }
            else
            {
                existedCartItem.ModifyPrice(cartItem.Price); //有可能价格更新了，每次都更新一下。
                existedCartItem.ModifyQuantity(existedCartItem.Quantity + cartItem.Quantity);
            }
        }

单元测试也做出相应的更改：

    [TestClass]
    public class CartTest
    {
        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void Constructor_CartIdDefault_ThrowArgumentException()
        {
            var cart = new Cart(default(Guid), Guid.NewGuid(), DateTime.Now);
            Assert.AreNotEqual(null, cart);
        }

        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void Constructor_UserIdDefault_ThrowArgumentException()
        {
            var cart = new Cart(Guid.NewGuid(), default(Guid), DateTime.Now);
            Assert.AreNotEqual(null, cart);
        }

        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void Constructor_LastChangeTimeDefault_ThrowArgumentException()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), default(DateTime));
            Assert.AreNotEqual(null, cart);
        }

        [TestMethod]
        public void AddCartItem_NotExisted_TotalItemCountIsIncreased()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), DateTime.Now);
            cart.AddCartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            Assert.AreEqual(1, cart.TotalItemCount());

            cart.AddCartItem(new Guid("22222222-2222-2222-2222-222222222222"), 1, 100);
            Assert.AreEqual(2, cart.TotalItemCount());
        }

        [TestMethod]
        public void AddCartItem_Existed_TotalItemCountIsNotIncreasedTotalItemNumIsIncreased()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), DateTime.Now);
            cart.AddCartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            Assert.AreEqual(1, cart.TotalItemCount());
            Assert.AreEqual(1, cart.TotalItemNum());

            cart.AddCartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            Assert.AreEqual(1, cart.TotalItemCount());
            Assert.AreEqual(2, cart.TotalItemNum());
        }
    }

    [TestClass]
    public class CartItemTest
    {
        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void ModifyQuantity_LessZero_ThrowArgumentException()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), DateTime.Now);
            cart.AddCartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            var cartItem = cart.GetCartItem(new Guid("11111111-1111-1111-1111-111111111111"));
            Assert.AreNotEqual(null, cartItem);
            Assert.AreEqual(1, cartItem.Quantity);
            cartItem.ModifyQuantity(-1);
        }

        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void ModifyQuantity_EqualsZero_ThrowArgumentException()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), DateTime.Now);
            cart.AddCartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            var cartItem = cart.GetCartItem(new Guid("11111111-1111-1111-1111-111111111111"));
            Assert.AreNotEqual(null, cartItem);
            Assert.AreEqual(1, cartItem.Quantity);
            cartItem.ModifyQuantity(0);
        }

        [TestMethod]
        public void ModifyQuantity_MoreZero_Success()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), DateTime.Now);
            cart.AddCartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            var cartItem = cart.GetCartItem(new Guid("11111111-1111-1111-1111-111111111111"));
            Assert.AreNotEqual(null, cartItem);
            Assert.AreEqual(1, cartItem.Quantity);
            cartItem.ModifyQuantity(10);
            Assert.AreEqual(10, cartItem.Quantity);
        }

        [TestMethod]
        [ExpectedException(typeof(ArgumentException))]
        public void ModifyPrice_LessZero_ThrowArgumentException()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), DateTime.Now);
            cart.AddCartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            var cartItem = cart.GetCartItem(new Guid("11111111-1111-1111-1111-111111111111"));
            Assert.AreNotEqual(null, cartItem);
            Assert.AreEqual(100, cartItem.Price);
            cartItem.ModifyPrice(-1);
        }

        [TestMethod]
        public void ModifyPrice_EqualsZero_Success()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), DateTime.Now);
            cart.AddCartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            var cartItem = cart.GetCartItem(new Guid("11111111-1111-1111-1111-111111111111"));
            Assert.AreNotEqual(null, cartItem);
            Assert.AreEqual(100, cartItem.Price);
            cartItem.ModifyPrice(0);
            Assert.AreEqual(0, cartItem.Price);
        }

        [TestMethod]
        public void ModifyPrice_MoreZero_Success()
        {
            var cart = new Cart(Guid.NewGuid(), Guid.NewGuid(), DateTime.Now);
            cart.AddCartItem(new Guid("11111111-1111-1111-1111-111111111111"), 1, 100);
            var cartItem = cart.GetCartItem(new Guid("11111111-1111-1111-1111-111111111111"));
            Assert.AreNotEqual(null, cartItem);
            Assert.AreEqual(100, cartItem.Price);
            cartItem.ModifyPrice(10);
            Assert.AreEqual(10, cartItem.Price);
        }
    }

　　这样一来，被玻璃鱼儿和netfocus2位园友所指出的奇怪的“UserBuyProductDomainService”也自然消失了。应用层代码变成：

        public Result Buy(Guid userId, Guid productId, int quantity)
        {
            var product = DomainRegistry.ProductService().GetProduct(productId);
            if (product == null)
            {
                return Result.Fail("对不起，未能获取产品信息请重试~");
            }

            var cart = _getUserCartDomainService.GetUserCart(userId);
            cart.AddCartItem(productId, quantity, product.SalePrice);
            DomainRegistry.CartRepository().Save(cart);
            return Result.Success();
        }

四、结语

　　DDD的道路是坎坷的，我希望通过在园子里发布的文章能够结交到志同道合的DDD之友，欢迎大家不吝啬自己的见解，多多留言，也让想学习或者正在学习DDD的园友少走一些弯路。

本文的源码地址：https://github.com/ZacharyFan/DDDDemo/tree/Demo5。

作者：Zachary
出处：https://zacharyfan.com/archives/141.html

▶关于作者：张帆（Zachary，个人微信号：Zachary-ZF）。坚持用心打磨每一篇高质量原创。欢迎扫描右侧的二维码~。

定期发表原创内容：架构设计丨分布式系统丨产品丨运营丨一些思考。

如果你是初级程序员，想提升但不知道如何下手。又或者做程序员多年，陷入了一些瓶颈想拓宽一下视野。欢迎关注我的公众号「跨界架构师」，回复「技术」，送你一份我长期收集和整理的思维导图。

如果你是运营，面对不断变化的市场束手无策。又或者想了解主流的运营策略，以丰富自己的“仓库”。欢迎关注我的公众号「跨界架构师」，回复「运营」，送你一份我长期收集和整理的思维导图。

posted @ 2016-11-21 06:22 Zachary_Fan 阅读(7554) 评论(5) 编辑收藏举报

会员力量，点亮园子希望

刷新页面返回顶部

Zachary_Fan