1

我一直在查看这段代码来创建一个基本的购物车，但缺点是它使用静态方法，因此购物车项目（当添加到购物车时）在会话之间共享。有人可以指出如何修改 ShoppingCart 方法以消除此限制吗？

完整的代码参考在这里

但我确定这是有问题的代码

// Readonly properties can only be set in initialization or in a constructor
public static readonly ShoppingCart Instance;

// The static constructor is called as soon as the class is loaded into memory
static ShoppingCart() {
    // If the cart is not in the session, create one and put it there
    // Otherwise, get it from the session
    if (HttpContext.Current.Session["ASPNETShoppingCart"] == null) {
        Instance = new ShoppingCart();
        Instance.Items = new List<CartItem>();
        HttpContext.Current.Session["ASPNETShoppingCart"] = Instance;
    } else {
        Instance = (ShoppingCart)HttpContext.Current.Session["ASPNETShoppingCart"];
    }
}

// A protected constructor ensures that an object can't be created from outside
protected ShoppingCart() { }

public void AddItem(int productId) {
    // Create a new item to add to the cart
    CartItem newItem = new CartItem(productId);

    // If this item already exists in our list of items, increase the quantity
    // Otherwise, add the new item to the list
    if (Items.Contains(newItem)) {
        foreach (CartItem item in Items) {
            if (item.Equals(newItem)) {
                item.Quantity++;
                return;
            }
        }
    } else {
        newItem.Quantity = 1;
        Items.Add(newItem);
    }
}
4

2 回答 2

15

我使用过几个商业购物车，每个购物车都将购物车存储在数据库中，甚至在结帐之前，并且只在 session.id 中存储了一个会话 ID。该 SessionID 绑定到临时购物车中的字段。

我强烈建议遵循相同的模式。假设您的网站非常受欢迎。在内存中存储太多数据（无论是在会话中还是在应用程序中）你会遇到问题。

于 2012-04-20T16:52:20.490 回答
3

如果您使用的是静态变量，那么任何线程（无论是哪个用户）都可以访问该数据。这意味着您基本上在所有用户之间共享一张购物卡，我怀疑您不想要。

相反，您可以有一个受保护的构造函数来防止手动实例化，然后有一个静态方法来读取 Session 对象并获取当前实例。至于填写Items列表的静态方法，您应该在构造函数中执行此操作。

public static ShoppingCart GetInstance()
{
    ShoppingCart cart = (ShoppingCart)Session["ASPNETShoppingCart"];

    if (cart == null)
    {
        Session["ASPNETShoppingCart"] = cart = new ShoppingCart();
    }

    return cart;
}

protected ShoppingCart()
{
    Items = new List<CartItem>();
}
于 2012-04-20T17:14:13.957 回答