时间戳

首页 > 解决方案 > C# List.Add 覆盖以前的对象

c# - C# List.Add 覆盖以前的对象

问题描述

我正在尝试创建一个简单的对象列表，但是在每个 foreach 循环中，以前的记录都被新的循环覆盖，以前的记录被新记录覆盖。因此，如果 realData 中有 6 个条目，则列表将具有最后一条记录的 6 倍。

我是否以某种方式重新创建列表而不是添加到列表中？还有另一种我忽略的创建列表的方法吗？

我的代码是

    public async Task<IActionResult> OrderOverview()
    {
        var itemList = new List<OrderItemVM>();
        var realData = await _context.OrderItem.ToListAsync();
        var orderItemVM = new OrderItemVM();

        foreach (var item in realData)
        {
            orderItemVM.Id = item.Id;                
            orderItemVM.OrderId = item.OrderId;
            orderItemVM.OrderName = _context.Order.Find(item.OrderId).OrderName;                
            orderItemVM.ItemName = item.ItemName;

            itemList.Add(orderItemVM);
        }

        return View(itemList);
    }

标签： c#asp.net-core

解决方案

您正在修改先前添加的对象，而不是添加新对象。你应该做这个。

foreach (var item in realData)
{
    OrderItemVM orderItemVM = new OrderItemVM ();
    orderItemVM.Id = item.Id;                
    orderItemVM.OrderId = item.OrderId;
    orderItemVM.OrderName = _context.Order.Find(item.OrderId).OrderName;                
    orderItemVM.ItemName = item.ItemName;

    itemList.Add(orderItemVM);
}

因此，基本上在每次迭代中，您都会创建一个新的空对象，然后分配该值并将其添加到 List 中。

推荐阅读