ShoppingCart模型计算属性空引用问题：TotalWeight实现是否正确？

你的TotalWeight计算属性实现确实存在问题，空引用错误就是因为它没有处理Product属性为null的情况。

先看你的模型代码：

[Key] public int Id { get; set; }
public int ProductId { get; set; }
public virtual Products Product { get; set; }
[Display(Name = "Quantity")] public int Quantity { get; set; }
[NotMapped] public int TotalWeight { get { return this.Quantity * this.Product.Weight; } }

问题根源 #

在EF中，如果没有**预先加载（Include）**关联的Product，或者延迟加载没有被正确触发（比如上下文已经被释放），this.Product就会是null。这时候直接访问this.Product.Weight，必然会抛出空引用异常。

修复方案 #

1. 给计算属性添加空值保护（最直接的修复）

修改TotalWeight的实现，先判断Product是否为null，再执行计算：

[NotMapped] 
public int TotalWeight 
{ 
    get 
    { 
        // 如果Product为空，返回0（或者你业务需要的其他默认值）
        return this.Product != null ? this.Quantity * this.Product.Weight : 0; 
    } 
}

如果使用C# 6及以上版本，还可以用空条件运算符简化写法：

[NotMapped] 
public int TotalWeight => (this.Product?.Weight ?? 0) * this.Quantity;

这里要注意运算符优先级，一定要把(this.Product?.Weight ?? 0)用括号括起来，避免出现先计算0 * Quantity的逻辑错误。

2. 确保查询时加载关联的Product

在查询ShoppingCart数据的时候，使用Include方法预先加载Product，从源头避免延迟加载带来的空引用问题：

// EF Core查询示例
var shoppingCarts = _context.ShoppingCarts
    .Include(sc => sc.Product) // 预先加载关联的Product实体
    .ToList();

只要数据库中ProductId对应有效的Product记录，这样查询出来的ShoppingCart对象，Product属性就不会是null。

额外建议 #

如果你的业务逻辑要求ShoppingCart必须关联有效的Product，可以考虑在数据库层面给ProductId添加外键约束，确保它不能引用不存在的Product记录，从根源减少空引用的发生概率。

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