我所理解的IRepository

    最近有一个项目采用了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情况下生成的脚本
 

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分很容易得到。
   

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);

    }