转自：
最近有一个项目采用了EntityFramework，对于基本的增删改查操作，我们采用了传统的仓储模式(IRepository)，但对于项目中的仓储接口的定义及实现上我认为存在部分缺陷。这个创建模式是以前同事编写好的，然后我们在新项目中利用它。 分页查询接口。
接口定义：
IEnumerable<T> QueryByPage(Func<T,
bool> FunWhere, Func<T,
string> FunOrder,
int PageSize,
int PageIndex,
out int recordsCount);
接口实现:
public IEnumerable<T> QueryByPage(Func<T,
bool> FunWhere, Func<T,
string> FunOrder,
int PageSize,
int PageIndex,
out int recordsCount) { recordsCount = context.Set<T>().Where(FunWhere).OrderByDescending(FunOrder).Count();
return context.Set<T>().Where(FunWhere).OrderByDescending(FunOrder).Select(t => t).Skip((PageIndex -
1) * PageSize).Take(PageSize); }
缺陷一：对于仓储接口，集合返回IEnumerable。 如果返回的是IEnumerable,那么实际上，系统会将表中的所有数据加载到内存中，然后再进行条件过滤，排序，再分页。如果表记录稍微多一点的话，性能可想而知。此种情况下应该推荐返回IQueryable,它才是真正适合和数据库打交道的对象。在客户端应用程序没有访问实际对象值之前，比如ToList()操作，它只是一个编译过程，根据用户传入的参数构建查询计划最终生成用于查询所用的SQLScript脚本。这种方式才是真正意义上的按需所取。 下面我们来定义一个新接口：
IQueryable<T> Query(Expression<Func<T,
bool>> filter);
注意这里是一个开放性特别大的查询接口，如果说不要轻易为客户端开放IQueryable这也没也问题，而且也不推荐将仓储接口直接开放给客户端应用程序，应该在仓储接口上为每个特定的应用系统提供全新的接口，比如可以这样：
public interface Iaspnet_UsersRepository { List<aspnet_UsersModel> QueryByPage(Expression<Func<T,
bool>> filter, Expression<Func<T,
string>> FunOrder,
int PageSize,
int PageIndex,
out int recordsCount); }
这里为了演示方便，就直接调用仓储接口做测试。 IEnumerable调用代码：
var list = service.QueryByPage(p => p.UserId != Guid.Empty,p=>p.UserName,
1,
1,
out recordsCount).ToList();
IEnumerable情况下生成的脚本：
SELECT [ Extent1 ].
[ UserId ] AS [ UserId ],
[ Extent1 ].
[ UserName ] AS [ UserName ],
[ Extent1 ].
[ LoweredUserName ] AS [ LoweredUserName ],
[ Extent1 ].
[ MobileAlias ] AS [ MobileAlias ],
[ Extent1 ].
[ IsAnonymous ] AS [ IsAnonymous ],
[ Extent1 ].
[ LastActivityDate ] AS [ LastActivityDate ] FROM [ dbo ].
[ aspnet_Users ] AS [ Extent1 ] IQueryable调用
var list
= service.Query(p
=>p.UserId
!=Guid.Empty).OrderBy(p
=>p.UserName).Take(
1).Skip(
1).ToList();
IQueryable情况下生成的脚本
View Code exec sp_executesql N
' SELECT [Limit1].[UserId] AS [UserId], [Limit1].[UserName] AS [UserName], [Limit1].[LoweredUserName] AS [LoweredUserName], [Limit1].[MobileAlias] AS [MobileAlias], [Limit1].[IsAnonymous] AS [IsAnonymous], [Limit1].[LastActivityDate] AS [LastActivityDate] FROM ( SELECT [Limit1].[UserId] AS [UserId], [Limit1].[UserName] AS [UserName], [Limit1].[LoweredUserName] AS [LoweredUserName], [Limit1].[MobileAlias] AS [MobileAlias], [Limit1].[IsAnonymous] AS [IsAnonymous], [Limit1].[LastActivityDate] AS [LastActivityDate], row_number() OVER (ORDER BY [Limit1].[UserName] ASC) AS [row_number] FROM ( SELECT TOP (1) [Project1].[UserId] AS [UserId], [Project1].[UserName] AS [UserName], [Project1].[LoweredUserName] AS [LoweredUserName], [Project1].[MobileAlias] AS [MobileAlias], [Project1].[IsAnonymous] AS [IsAnonymous], [Project1].[LastActivityDate] AS [LastActivityDate] FROM ( SELECT [Extent1].[UserId] AS [UserId], [Extent1].[UserName] AS [UserName], [Extent1].[LoweredUserName] AS [LoweredUserName], [Extent1].[MobileAlias] AS [MobileAlias], [Extent1].[IsAnonymous] AS [IsAnonymous], [Extent1].[LastActivityDate] AS [LastActivityDate] FROM [dbo].[aspnet_Users] AS [Extent1] WHERE [Extent1].[UserId] <> @p__linq__0 ) AS [Project1] ORDER BY [Project1].[UserName] ASC ) AS [Limit1] ) AS [Limit1] WHERE [Limit1].[row_number] > 0 ORDER BY [Limit1].[UserName] ASC ',N
' @p__linq__0 uniqueidentifier ',
@p__linq__0 = ' 00000000-0000-0000-0000-000000000000 ' 小结：
只有返回IQueryable，才会实现按需所取。尽管这个方法在最上层的服务层没有进行封装，理论上客户端应用程序无法访问到此方法，但既然提供了，而且对外是Public，那么在一定程度上就会对程序员造成误导。
缺陷二：不太实用的接口定义 我们来看一下这个接口定义
IEnumerable<T> QueryByPage(List<KeyValuePair<
string,
object>> queryWhere, List<KeyValuePair<
string,
int>> funOrder,
int PageSize,
int PageIndex,
out int recordsCount)
再看下它的实现，可以看出这个方法实现了按需所取，但这是多么的不协调啊(上面那个分页方法就不是按需所取)。就像有人做事一样，50%的工作可以打100分，其余的50%只能打60分，而且这40分很容易得到。
View Code string objName =
string.Empty; Type type =
typeof(T);
var properties = Context.GetType().GetProperties();
foreach (
var propertyInfo
in properties) {
if (propertyInfo.PropertyType.GetGenericArguments()[
0].Name == type.Name) { objName = propertyInfo.Name;
break; } }
string queryObj = EntitySQLGenerator.GenerateQuery(queryWhere, funOrder, Context.DefaultContainerName, objName); IQueryable<T> result = Context.CreateQuery<T>(queryObj); recordsCount = result.Count();
return result.Select(t => t).Skip((PageIndex -
1) * PageSize).Take(PageSize).AsEnumerable<T>();
查询条件难以理解 List<KeyValuePair<string, object>>，用户根本不知道需要传递什么参数，提供的接口参数一定要让调用者容易理解。 我理解的仓储接口
如果还有哪些不够的，可以根据情况再决定增加哪些功能。创建接口不推荐直接开放给客户端，应该在此基础上重新为每个应用定义接口。
public interface IMyRepository<T>
where T :
class,
new() { T Create(); T Update(T entity); T Insert(T entity);
void Delete(T entity); List<T> FindAll(); IQueryable<T> Query(Expression<Func<T,
bool>> filter); }