Repository Pattern中Where方法内存占用过高加载冗余数据解决方法

内存占用过高的核心原因是当前仓储的GetAll()方法返回类型为IEnumerable<T>，而非IQueryable<T>：

• 基于IEnumerable<T>的Where、OrderBy等操作会在内存中执行，流程是先把全表10万条数据全部加载到应用内存、完成全量对象实例化，再执行条件筛选，最终仅返回少量符合条件的数据，其余对象全部被GC回收，产生大量无效内存开销。
• 基于IQueryable<T>的查询会将Lambda条件转换为表达式树，由ORM框架（EF/EF Core等）翻译成对应SQL语句，直接在数据库侧完成过滤，仅将符合条件的结果返回给应用，从根源上避免全量数据加载。

方案1：修改GetAll返回类型为IQueryable（兼容现有查询写法） #

同步修改仓储接口和实现类的GetAll()方法，返回IQueryable<T>，严禁在方法内部调用ToList()、AsEnumerable()等会提前触发数据库查询的方法：

// 仓储接口定义
public interface IImageRepository
{
    IQueryable<MyImage> GetAll();
}

// EF Core实现示例
public class ImageRepository : IImageRepository
{
    private readonly AppDbContext _dbContext;
    public ImageRepository(AppDbContext dbContext)
    {
        _dbContext = dbContext;
    }

    public IQueryable<MyImage> GetAll()
    {
        // 可在此处追加全局过滤条件，比如软删除过滤：.Where(x => !x.IsDeleted)
        return _dbContext.Set<MyImage>().AsQueryable();
    }
}

改造完成后，你现有两种查询写法都会自动翻译成SQL在数据库侧执行，不会加载全量数据：

// 单条查询：生成SQL为 SELECT * FROM Images WHERE Id = 1，仅返回1条匹配记录
var singleImage = imageRepostery.GetAll().Where(x=> x.Id == 1).ToList();

// 范围查询：生成SQL为 SELECT * FROM Images WHERE Id > 500 OR Id < 1000，仅返回符合条件的记录
List<MyImage> images = imageRepostery.GetAll().Where(x=> x.Id > 500 || x.Id < 1000 ).ToList();

方案2：封装表达式参数查询方法（更严谨，避免IQueryable滥用） #

如果不希望直接对外暴露IQueryable，可以在仓储层新增接收表达式树参数的查询方法，同样能实现数据库侧过滤，调用方式更可控：

// 仓储接口新增方法
public interface IImageRepository
{
    List<MyImage> Query(Expression<Func<MyImage, bool>> predicate);
    // 其他原有方法...
}

// 方法实现
public List<MyImage> Query(Expression<Func<MyImage, bool>> predicate)
{
    return _dbContext.Set<MyImage>().Where(predicate).ToList();
}

// 调用示例
List<MyImage> images = imageRepostery.Query(x=> x.Id > 500 || x.Id < 1000);

• 不要在GetAll()返回的IQueryable结果上调用AsEnumerable()、ToList()后再做筛选，这类操作会提前触发全量数据查询，回到内存过滤的老问题。
• 写查询条件时尽量使用数据库字段支持的操作，不要在Lambda中写C#特有、数据库无法识别的函数（比如自定义的C#计算方法），否则会抛出查询翻译异常。
• 单条主键查询建议单独封装GetById(int id)方法，内部直接用FirstOrDefault(x => x.Id == id)实现，语义更清晰。

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