首页 > 代码库 > 如何一步一步用DDD设计一个电商网站(五)—— 停下脚步,重新出发

如何一步一步用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_Fan
出处:http://www.cnblogs.com/Zachary-Fan/p/6069169.html

如何一步一步用DDD设计一个电商网站(五)—— 停下脚步,重新出发