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

发布时间：2020-12-15 03:53:22 所属栏目：百科来源：网络整理

导读：最好用一些代码解释这个： public IQueryableDatabaseRecord GetQueryableLinkedRecords(){ if(this.currentlyHeldIds.Count() == 0) { return Enumerable.EmptyDatabaseRecord().AsQueryable(); } else { return from r in this.DBContext.DatabaseRecords

最好用一些代码解释这个：

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()并返回它.

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

（编辑：李大同）

【声明】本站内容均来自网络，其相关言论仅代表作者个人观点，不代表本站立场。若无意侵犯到您的权利，请及时与联系站长删除相关内容!