泛型仓库Get方法传Expression时类型转换错误及性能对比咨询

问题背景 #

想在泛型仓库里实现一个接收Expression的Get方法，避免为不同属性编写重复过滤逻辑，仓库代码如下：

public async Task<IQueryable<T>> Get(Expression<Func<T, bool>> expression)
{
    return databaseContext.Set<T>().Where(expression);
}

控制器调用代码：

repositoryWrapper.ImageRepository.Get(x => x.ImageId == id)
// id和x.ImageId均为int类型

调用时出现错误：Unable to cast object of type 'System.Decimal' to type 'System.Single'.

另外想确认：这种直接在仓库传Expression的方案，比先返回所有对象再做内存查询更好吗？

错误原因排查 #

虽然你声明id和x.ImageId都是int类型，但转换错误说明EF解析表达式生成SQL时遇到了类型不匹配，核心原因大概率是实体类属性与数据库列类型不一致：

• 比如实体类的ImageId定义为int，但数据库对应列是decimal类型；
• 或者EF Core的映射配置存在偏差，导致类型映射错误。

解决方案 #

1. 核对实体类Image的ImageId属性类型，与数据库对应列的类型，确保两者完全匹配（比如统一为int或decimal）；
2. 如果数据库列类型确实是decimal，可选择：
   • 将实体类属性改为decimal；
   • 在查询时显式转换类型（不推荐，可能影响索引使用）：

     repositoryWrapper.ImageRepository.Get(x => x.ImageId == (decimal)id)
     

性能对比结论 #

这种传Expression给EF的方案远优于先返回所有对象再做内存查询：

• 前者会将表达式转换为SQL语句在数据库端执行过滤，仅返回符合条件的数据，能利用数据库索引，大幅减少网络传输的数据量；
• 后者会把全表数据加载到应用内存，再用Linq to Objects做过滤，数据量大时会占用大量内存，且无法利用数据库的查询优化，性能差距会非常明显。

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