Entity Framework关联表属性更新逻辑错误排查

问题背景 #

我有Order、Item和Seller三个实体，在Order控制器的Post方法中尝试获取请求里的商品列表及对应购买数量，保存到数据库时却遇到两个问题：

1. 订单的商品列表保存的是商品的总库存而非请求中的购买数量
2. 尝试修改订单商品数量时，意外修改了数据库中商品表的库存数值

实体定义 #

Order实体

public class Order
{
    public long Id { get; set; }
    public DateTime Date { get; set; }
    public StatusEnum Status { get; set; }
    public long SellerId { get; set; }
    public Seller Seller { get; set; }
    public double Total { get; set; }
    public IList<Item> Items { get; set; }
}

Item实体

public class Item
{
    public long Id { get; set; }
    public string Name { get; set; }
    public string Skuid { get; set; }
    public double Price { get; set; }
    public double Amount { get; set; }
}

控制器原代码 #

[HttpPost("v1/order")]
public async Task<IActionResult> CreateOrderAsync(
    [FromServices] PottencialDataContext context,
    [FromBody] CreateOrderViewModel model)
{
    if (!ModelState.IsValid)
        return BadRequest(new ResultViewModel<string>(ModelState.GetErrors()));
    
    try
    {
        var listItem = new List<Item>();
        double total = 0;
        
        foreach (var item in model.Items)
        {
            var itemSkuid = context.Items.FirstOrDefault(x => x.Skuid == item.Skuid);
            itemSkuid.Amount -= item.Amount; 
            context.Items.Update(itemSkuid);
            
            total = TotalPriceOrder.OrderTotalPrice(listItem, item.Amount);
            listItem.Add(itemSkuid);
        }

        var seller = await context.Sellers.FirstOrDefaultAsync(x => x.Id == model.SellerId);
        if (seller == null)
            return NotFound(new ResultViewModel<Seller>("Not Found"));

        var newOrder = new Order
        {
            Date = DateTime.Now,
            Status = StatusEnum.AWAITING_PAYMENT,
            Seller = seller,
            Total = total,
            Items = listItem
        };

        await context.Orders.AddAsync(newOrder);
        await context.SaveChangesAsync();

        return Created($"v1/order/{newOrder.Id}", new ResultViewModel<Order>(newOrder));
    }
    catch (DbUpdateException e)
    {
        return StatusCode(500, new ResultViewModel<Seller>("WM0X23"));
    }
    catch
    {
        return StatusCode(500, new ResultViewModel<Seller>("WM0X24"));
    }
}

问题根源分析 #

1. 订单商品数量错误的原因 #

你直接把从数据库查询出的itemSkuid（库存商品实体）添加到订单的Items列表中，这个实体的Amount字段是数据库里的库存值——哪怕你做了itemSkuid.Amount -= item.Amount，这修改的是剩余库存数量，而非用户的购买数量，订单自然无法记录正确的购买量。

2. 修改订单商品数量导致库存变更的原因 #

EF Core对从数据库查询出来的实体默认处于跟踪状态，任何对该实体属性的修改，都会在调用SaveChangesAsync时同步到数据库。如果你后续试图修改这个实体的Amount（比如想改成购买数量），EF Core会把这个操作识别为库存变更，最终直接更新数据库里的商品表。

解决方案 #

核心思路：订单中的商品应该是独立的记录（或库存商品的快照+购买数量），不能复用被EF跟踪的库存实体。

方案1：创建Item副本记录购买数量 #

在添加到订单时，新建一个Item实例，复制库存商品的基础信息，然后设置为请求中的购买数量：

foreach (var item in model.Items)
{
    var stockItem = context.Items.FirstOrDefault(x => x.Skuid == item.Skuid);
    if (stockItem == null)
    {
        return BadRequest(new ResultViewModel<string>($"商品SKU {item.Skuid} 不存在"));
    }
    // 校验库存是否充足
    if (stockItem.Amount < item.Amount)
    {
        return BadRequest(new ResultViewModel<string>($"商品SKU {item.Skuid} 库存不足"));
    }
    // 更新库存
    stockItem.Amount -= item.Amount;
    // 创建订单商品实例，记录购买数量
    var orderItem = new Item
    {
        Name = stockItem.Name,
        Skuid = stockItem.Skuid,
        Price = stockItem.Price,
        Amount = item.Amount // 这里是购买数量，不是剩余库存
    };
    listItem.Add(orderItem);
    // 计算订单总价
    total += orderItem.Price * orderItem.Amount;
}

方案2：引入中间实体（推荐） #

更合理的设计是新增OrderItem中间实体，专门记录订单与商品的关联及购买信息，彻底区分库存商品和订单商品：

// 新增OrderItem实体
public class OrderItem
{
    public long Id { get; set; }
    public long OrderId { get; set; }
    public Order Order { get; set; }
    public long ItemId { get; set; }
    public Item Item { get; set; }
    public double PurchaseAmount { get; set; } // 购买数量
    public double PurchasePrice { get; set; } // 下单时的单价
}

// 修改Order实体
public class Order
{
    // ... 原有属性
    public IList<OrderItem> OrderItems { get; set; }
}

控制器中创建OrderItem实例：

foreach (var item in model.Items)
{
    var stockItem = context.Items.FirstOrDefault(x => x.Skuid == item.Skuid);
    // 库存校验逻辑...
    stockItem.Amount -= item.Amount;
    
    var orderItem = new OrderItem
    {
        Item = stockItem,
        PurchaseAmount = item.Amount,
        PurchasePrice = stockItem.Price
    };
    newOrder.OrderItems.Add(orderItem);
    total += orderItem.PurchasePrice * orderItem.PurchaseAmount;
}

额外注意事项 #

• 必须添加库存充足校验，避免出现负库存
• 建议用事务包裹库存更新和订单创建，确保数据一致性：

using var transaction = await context.Database.BeginTransactionAsync();
try
{
    // 库存更新、订单创建逻辑...
    await context.SaveChangesAsync();
    await transaction.CommitAsync();
}
catch
{
    await transaction.RollbackAsync();
    throw;
}

内容的提问来源于stack exchange，提问作者wesley moreira