Question

I’m preparing data for a datatable in Linq2Sql

This code highlights as a ‘Possible multiple enumeration of IEnumerable’ (in Resharper)

// filtered is an IEnumerable or an IQueryable

var total = filtered.Count();

var displayed = filtered
                .Skip(param.iDisplayStart)
                .Take(param.iDisplayLength).ToList(); 

And I am 100% sure Resharper is right.

How do I rewrite this to avoid the warning

To clarify, I get that I can put a ToList on the end of filtered to only do one query to the Database eg.

 var filteredAndRun = filtered.ToList();

 var total = filteredAndRun.Count();

 var displayed = filteredAndRun
                .Skip(param.iDisplayStart)
                .Take(param.iDisplayLength).ToList(); 

but this brings back a ton more data than I want to transport over the network.

I’m expecting that I can’t have my cake and eat it too. 🙁

Answer 1

It sounds like you’re more concerned with multiple enumeration of IQueryable<T> rather than IEnumerable<T>.

However, in your case, it doesn’t matter.

The Count call should translate to a simple and very fast SQL count query. It’s only the second query that actually brings back any records.

If it is an IEnumerable<T> then the data is in memory and it’ll be super fast in any case.

I’d keep your code exactly the same as it is and only worry about performance tuning when you discover you have a significant performance issue. 🙂