asp.net MVC应用程序中的POCO格式

Question

我正在创建一个简单的aspnetmvc购物车应用程序,并定义了类似于以下类的类:

public class Cart
{
    public Guid Id { get; set; }
    public string Comment { get; set; }
    public DateTime CreatedOn { get; set; }
    public DateTime UpdatedOn { get; set; }
    public DateTime? DeletedOn { get; set; }
    public List<CartItem> CartItems { get; set; }
}

public class CartItem
{
    public Guid Id { get; set; }
    public Guid CartId { get; set; }
    public string Sku { get; set; }
    public double ItemAmount { get; set; }
    public double Amount { get; set; }
    public int Quantity { get; set; }
}

有一个非常简单的存储库,看起来像这样:

public interface ICartRepository
{
    Cart CreateCart();
    Cart GetCart(Guid Id);
    Cart UpdateCart(Cart cart);
    void DeleteCart(Cart cart);
}

在创建类之后,我开始意识到我更适合从Cart类中分离List属性,然后在我的视图模型中重新组合它们.

public class vmCart
{
    public Cart cart { get; set; }
    public List<CartItem> CartItems { get; set; }
    public string CreatedOn
    {
        get
        {
            return cart.CreatedOn.ToString();
        }
    }
    public string CartTotal
    {
        get
        {
            var total = (double)0;
            foreach (var lineItem in CartItems)
            {
                total += lineItem.Amount;
            }
            return total.ToString("c");
        }
    }
}

这意味着我必须在我的模型中为CartItems的CRUD添加其他方法,但仍然允许我将对象作为组合实体呈现给视图(通过viewmodel).

两种格式都没有明显的优势,但我希望对设计有任何反馈.

最好的祝福,

哈尔

Answer 1

我个人会将CartItems保留在购物车中.原因如下:

1. 有明显的"有"关系表明Cart中有CartItems.

2. 购物车是您的域模型中的明确聚合.如果没有装载购物车物品,您不太可能在装载购物车实体.虽然这会使您的CRUD操作在您的存储库中变得更复杂,但是任何存储库使用者都需要执行UpdateCart()而不是执行自己的迭代并执行UpdateCartItem().

3. 将其分解为另一个对象会增加没有特色的复杂性

4. 无论是哪种方式,都可以轻松地将项目放入View中.

如果您的域模型或您处理购物车项目的方式发生任何变化,这些假设可能会发生变化,从而您采取的方法也会发生变化.但这就是我现在看到的.