EF Core 6不可修改实体时通过外键正确加载关联数据的方法

首先为提出入门级问题致歉。
我目前有一套无法修改的既有EF Core 6模型，其中包含如下Wiring与Location实体类：

// cannot change this entity
public class Wiring
{
    public int Id {get; set;}
    public string Name {get; set;}
    public int? LocationAId {get; set;}
    public int? LocationBId {get; set;}
}

// cannot change this entity
public class Location
{
    public int Id {get; set;}
    public string Name {get; set;}
}

在业务逻辑层中，我定义了一个DTO类，用于存储Wiring名称及其连接的两个Location的名称。Wiring实体一定包含Name属性值，但关联Location的Name属性可能为null或空值。

public class WiringDto
{
    public string Name {get; set;}
    public string? LocationAId {get; set;}
    public string? LocationBId {get; set;}
}

核心疑问：加载Wiring数据时，加载关联Location数据的正确实现方式是什么？
我目前编写了一套可正常运行的实现，但代码会产生如下警告：

Captured variable is disposed in the outer scope（捕获的变量已在外层作用域中被释放）
产生该警告的关联数据加载代码如下：

var wiringDtos = await context.Wirings
.AsNoTracking()
.Select(w => new WiringDto
{
    w.Name,
    context.Locations.FirstOrDefault(l => l.Id == w.LocationAId) != null ? context.Locations.FirstOrDefault(l => l.Id == w.LocationAId).Name : null,
    context.Locations.FirstOrDefault(l => l.Id == w.LocationBId) != null ? context.Locations.FirstOrDefault(l => l.Id == w.LocationBId).Name : null,
})
.ToListAsync();

你的代码产生警告、存在性能隐患的核心原因有三个：

1. 你在Select投影表达式中直接引用了外层的context实例，编译器检测到DbContext是可释放资源，且被表达式树捕获，因此抛出「捕获的变量已在外层作用域中被释放」的警告，极端场景下如果查询执行时DbContext已经被释放，会直接抛出运行时异常。
2. 代码存在严重的N+1查询问题：对每一条Wiring记录，你会为两个关联Location各执行2次重复查询（1次判断是否存在、1次取Name值），当Wiring数据量较大时，数据库请求次数会线性增长，性能极差。
3. 代码存在冗余逻辑：完全不需要先判断关联Location是否存在再取Name，用null条件运算符?.即可直接安全取值。

因为你无法修改实体类，不需要给实体加导航属性，直接使用两次GroupJoin做左关联即可，EF Core会自动把代码翻译成单次执行的LEFT JOIN SQL，无额外查询、不会捕获context触发警告，性能远高于原写法：

var wiringDtos = await context.Wirings
    .AsNoTracking()
    // 左关联Location表对应LocationA
    .GroupJoin(
        context.Locations,
        w => w.LocationAId,
        loc => loc.Id,
        (w, locList) => new { Wiring = w, LocationA = locList.FirstOrDefault() }
    )
    // 左关联Location表对应LocationB
    .GroupJoin(
        context.Locations,
        temp => temp.Wiring.LocationBId,
        loc => loc.Id,
        (temp, locList) => new WiringDto
        {
            Name = temp.Wiring.Name,
            // 直接用null条件运算符取值，不存在则返回null
            LocationAId = temp.LocationA?.Name,
            LocationBId = locList.FirstOrDefault()?.Name
        }
    )
    .ToListAsync();

注：根据你的业务描述，DTO中LocationAId、LocationBId实际存储的是Location的Name值，建议后续把属性名调整为LocationAName、LocationBName，避免语义混淆。

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