C# · 12月 27, 2021

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

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