c# – 返回Enumerable.Empty().AsQueryable()一个坏主意?

前端之家收集整理的这篇文章主要介绍了c# – 返回Enumerable.Empty().AsQueryable()一个坏主意?前端之家小编觉得挺不错的,现在分享给大家,也给大家做个参考。
最好用一些代码解释这个:
public IQueryable<DatabaseRecord> GetQueryableLinkedRecords()
{
    if(this.currentlyHeldIds.Count() == 0)
    {
        return Enumerable.Empty<DatabaseRecord>().AsQueryable();
    }
    else
    {
        return from r in this.DBContext.DatabaseRecords
               where this.currentlyHeldIds.Contains(r.Id)
               select r;
    }
}

这个想法是,如果没有查询当前的HelloId,则没有理由再次实际查询数据库.如果currentHeldIds没有值,LINQ to sql仍将查询db.这种方法有什么问题吗?我意识到还有一些其他问题与返回IQueryable一般有关,但抛开那些论点,试图绕过这样的db调用有什么不对吗?

解决方法

我认为你应该重新思考你的功能实际上要做什么.如果您要返回IQueryable< T>,则意味着调用者将存储生成查询,并在实际执行查询时接收最新结果.记住:在这种情况下,实际上不会查询数据库,直到在查询调用.ToList(),. ToArray(),foreach等.

但是,如果您的目标是返回查询的当前状态,那么您的方法应该只返回IList< T>或类似的规定.然后,您可以返回一个空List< T>,或者在您构造的查询调用.ToList()并返回它.

一般来说,我会尽量避免返回空的可查询对象,因为它可能会误导调用者他们实际获得的内容.

猜你在找的C#相关文章