如何使用LINQ从数据库填充List<Clients>集合？

首先你当前代码的核心问题是返回了空的_clientsList，而不是转换后的客户端数据，另外还有几个可以优化的地方，下面一步步解决：

1. 修正代码中的错误与冗余 #

先修复实体类的语法错误 #

你的Clients类中Id属性缺少类型声明，需要补充（类型根据数据库字段选择，比如int或Guid）：

public class Clients
{
    public int Id { get; set; }
    public string Name { get; set; }
    public string Address { get; set; }
}

修正Repository的实现 #

去掉多余的_clientsList字段，直接在EF查询中完成投影，并且返回正确的结果：

public class ClientsRepository : IClientsRepository
{
    private readonly MainDbContext _dbContext;

    // 注意：原构造函数的`base(dbContext)`无继承依据，直接移除
    public ClientsRepository(MainDbContext dbContext)
    {
        _dbContext = dbContext;
    }

    public async Task<List<Clients>> ListAllClients()
    {
        // 直接在EF查询中映射为Clients对象，无需先查匿名类型再转换
        return await _dbContext.ClientsTbl
                    .AsNoTracking()
                    .Join(
                        _dbContext.AddressTbl.AsNoTracking(),
                        client => client.Id,
                        address => address.ClientId,
                        (client, address) => new Clients
                        {
                            Id = client.Id,
                            Name = client.Name,
                            Address = address.FullAddress
                        })
                    .ToListAsync();
    }
}

2. 这类操作的最佳实践与规范 #

• 不要在Repository中维护状态：之前的_clientsList属于冗余状态，Repository的职责是封装数据访问逻辑，每次查询应直接从数据库获取最新数据，无需本地缓存（有明确缓存需求时再单独实现）。
• 利用EF投影优化性能：直接在查询中映射为目标对象，EF会生成只包含所需字段的SQL，减少数据传输和内存占用，比先查匿名类型再内存转换更高效。
• 异步方法正确使用await：确保所有异步操作都用await，避免阻塞线程，这是EF异步API的标准用法。
• 只读查询用AsNoTracking：对于不需要修改的查询，关闭EF的变更追踪，能显著提升查询性能。
• 保持Repository单一职责：每个Repository专注于一个实体（或聚合根）的数据操作，不要混入无关逻辑。
• 坚持接口与实现分离：你使用IClientsRepository接口的做法很合理，便于后续依赖注入和单元测试（比如用Mock替代真实数据库）。

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