EF Core订单系统实体设计问题求解与优化建议咨询

问题背景 #

我正尝试构建一个订单系统，基础实体为代表SKU的Sku类：

public class Sku
{
    [Key]
    public Guid Id { get; set; }
    // 大量SKU相关信息
}

同时还有Customer和OrderPicker实体：

public class Customer
{
    [Key]
    public Guid Id { get; set; }
    // 客户相关信息
    public List<Order>? Orders { get; set; }
}

public class OrderPicker
{
    [Key]
    public Guid Id { get; set; }
    // 拣货员相关信息
    public List<Order>? Orders { get; set; }
}

订单与客户是一对多关系，已通过以下代码建模：

public class Order
{
      [Key]
      public Guid Id { get; set; }
      [Required]
      public Customer Customer { get; set; }
      public Guid CustomerGuid { get; set; }
      // 订单相关数据
}

并在AppContext中配置：

modelBuilder.Entity<Order>()
                .HasOne(o => o.Customer)
                .WithMany(c => c.Orders)
                .HasForeignKey(o => o.CustomerGuid)
                .HasPrincipalKey(c => c.Id);

订单与OrderPicker的关系配置同理。

在建模订单与SKU的关系时遇到问题：订单需包含多个带数量的SKU，于是引入关联实体OrderItem：

[Owned]
public class OrderItem
{
    public Guid SkuId { get; set; }
    public Sku Sku { get; set; }
    public Order Order { get; set; }
    public uint Quantity { get; set; }
}

并在Order中声明ICollection<OrderItem>，但不确定如何为此实体指定主键与外键，也不确定是否需要。

此外，用于SKU库存管理的StockItem实体：

public class StockItem
{
    public Sku Sku { get; private set; }
    [Key]
    public Guid SkuId { get; private set; }
    [Required]
    public uint StockBalance { get; private set; }
}

添加初始迁移时出现诸多报错，均与StockItem和Sku的关系有关，例如：

The 'Sku' property 'StockItem.Sku' could not be mapped because the database provider does not support this type. Consider converting the property value to a type supported by the database using a value converter

问题解答 #

1. 解决OrderItem和StockItem的问题 #

针对OrderItem的调整

• 移除[Owned]特性：Owned是EF Core值对象专用特性，OrderItem是带业务字段的关联实体（多对多中间表），不属于值对象，不能使用该特性。
• 配置复合主键与外键：订单中同一个SKU应唯一，所以OrderItem用OrderId+SkuId作为复合主键，同时关联对应实体。

调整后的OrderItem代码：

public class OrderItem
{
    // 复合主键
    public Guid OrderId { get; set; }
    public Guid SkuId { get; set; }

    // 导航属性
    public Order Order { get; set; }
    public Sku Sku { get; set; }

    public uint Quantity { get; set; }
}

在AppContext中配置关系：

modelBuilder.Entity<OrderItem>()
    .HasKey(oi => new { oi.OrderId, oi.SkuId }); // 设置复合主键

modelBuilder.Entity<OrderItem>()
    .HasOne(oi => oi.Order)
    .WithMany(o => o.OrderItems)
    .HasForeignKey(oi => oi.OrderId);

modelBuilder.Entity<OrderItem>()
    .HasOne(oi => oi.Sku)
    .WithMany() // 若Sku无需反向导航，此处留空
    .HasForeignKey(oi => oi.SkuId);

同时更新Order类的集合属性：

public class Order
{
    // 原有属性...
    public ICollection<OrderItem> OrderItems { get; set; } = new List<OrderItem>();
}

针对StockItem的调整

报错原因是导航属性Sku的访问器为private set，EF Core无法映射，且未明确一对一关系。调整方案：

• 修改Sku导航属性的访问器为protected set，兼顾封装与EF映射需求。
• 明确StockItem与Sku的一对一关系：StockItem的主键SkuId同时作为关联Sku的外键。

调整后的StockItem代码：

public class StockItem
{
    [Required]
    public Sku Sku { get; protected set; }

    [Key]
    public Guid SkuId { get; protected set; }

    [Required]
    public uint StockBalance { get; private set; }

    // EF Core所需的无参构造函数
    protected StockItem() { }

    // 业务构造函数
    public StockItem(Sku sku, uint initialStock)
    {
        Sku = sku;
        SkuId = sku.Id;
        StockBalance = initialStock;
    }

    // 库存操作方法，保证数据安全
    public void AddStock(uint amount)
    {
        StockBalance += amount;
    }

    public bool ReduceStock(uint amount)
    {
        if (StockBalance < amount) return false;
        StockBalance -= amount;
        return true;
    }
}

在AppContext中配置关系：

modelBuilder.Entity<StockItem>()
    .HasOne(si => si.Sku)
    .WithOne() // 若Sku无需反向导航，此处留空
    .HasForeignKey<StockItem>(si => si.SkuId);

2. 实体设计优化建议 #

简化外键命名

将Order中的CustomerGuid改为CustomerId，遵循EF Core默认命名规范，减少手动配置工作量：

public class Order
{
    [Key]
    public Guid Id { get; set; }
    
    [Required]
    public Customer Customer { get; set; }
    public Guid CustomerId { get; set; }
    
    public OrderPicker OrderPicker { get; set; }
    public Guid OrderPickerId { get; set; }
    
    public ICollection<OrderItem> OrderItems { get; set; } = new List<OrderItem>();
}

此时关系配置可简化：

modelBuilder.Entity<Order>()
    .HasOne(o => o.Customer)
    .WithMany(c => c.Orders); // EF自动匹配CustomerId作为外键

初始化集合导航属性

所有集合类型的导航属性声明时初始化，避免空引用异常：

public class Customer
{
    [Key]
    public Guid Id { get; set; }
    // 客户信息
    public List<Order> Orders { get; set; } = new List<Order>();
}

添加订单状态字段

订单系统建议增加枚举类型的状态字段，便于业务流程管控：

public enum OrderStatus
{
    PendingPayment,
    Paid,
    Shipped,
    Completed,
    Cancelled
}

public class Order
{
    // 原有属性...
    public OrderStatus Status { get; set; } = OrderStatus.PendingPayment;
}

增加审计字段

给订单、库存等实体添加创建/更新时间，便于数据追踪：

public class Order
{
    // 原有属性...
    public DateTime CreatedAt { get; set; } = DateTime.UtcNow;
    public DateTime? UpdatedAt { get; set; }
}

可通过EF Core的SaveChanges拦截器自动更新UpdatedAt字段。

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