ASP.NET MVC 5中Session存储列表值异常：添加商品仅显示首个

我来帮你排查这个问题～你的代码里有个核心逻辑错误：每次添加商品时，你都重新创建了一个只包含当前新商品的List，而且只有当Session["Cart"]为空的时候才会把这个列表存入Session。当你添加第二个商品时，Session["Cart"]已经有值了，你完全没把新商品加到已有的购物车列表里，所以视图永远只会显示第一次添加的那个商品。

修正后的代码逻辑 #

正确的做法应该是：先从Session里取出已有的购物车列表（如果没有就新建空列表），把新商品添加到这个列表中，再把更新后的列表存回Session。具体代码如下：

// 从Session中取出已有的购物车列表，没有则创建空列表
List<ShoppingCartItem> shoppingCartItems = Session["Cart"] as List<ShoppingCartItem> ?? new List<ShoppingCartItem>();

// 创建新的购物车商品项
ShoppingCartItem newItem = new ShoppingCartItem() 
{
    Product = product.Name, 
    Attributes = atts, 
    Options = opts, 
    Price = producttotalprice, 
    Quantity = 1
};

// 将新商品添加到购物车列表
shoppingCartItems.Add(newItem);

// 更新Session中的购物车数据
Session["Cart"] = shoppingCartItems;

// 返回视图，传入更新后的购物车列表
return View(shoppingCartItems);

关键细节说明 #

• 使用Session["Cart"] as List<ShoppingCartItem> ?? new List<ShoppingCartItem>()：这个语法是先尝试把Session中的值转换成List<ShoppingCartItem>，如果转换失败（比如Session为空或者类型不对），就创建一个新的空列表，避免空引用异常。
• 每次添加商品都要把更新后的列表重新存入Session：因为Session存储的是对象的引用，虽然直接修改列表内容（比如Add操作）本质上会修改Session里的列表，但显式重新赋值可以确保万无一失，尤其是在Session需要序列化的场景下。
• 视图中要确保正确遍历购物车列表：比如在Razor视图里用foreach循环遍历Model（也就是传入的购物车列表），才能完整显示所有商品。

内容的提问来源于stack exchange，提问作者Nasir Khan