Extensiones para IEnumerable genérica

Question

Tengo dos extensiones de IEnumerable:

public static class IEnumerableGenericExtensions 
{ 
    public static IEnumerable<IEnumerable<T>> InSetsOf<T>(this IEnumerable<T> source, int max) 
    { 
     List<T> toReturn = new List<T>(max); 
     foreach (var item in source) 
     { 
      toReturn.Add(item); 
      if (toReturn.Count == max) 
      { 
       yield return toReturn; 
       toReturn = new List<T>(max); 
      } 
     } 
     if (toReturn.Any()) 
     { 
      yield return toReturn; 
     } 
    } 

    public static int IndexOf<T>(this IEnumerable<T> source, Predicate<T> searchPredicate) 
    { 
     int i = 0; 
     foreach (var item in source) 
      if (searchPredicate(item)) 
       return i; 
      else 
       i++; 

     return -1; 
    } 
} 

Entonces escribo este código:

 Pages = history.InSetsOf<Message>(500); 
     var index = Pages.IndexOf(x => x == Pages.ElementAt(0)); 

donde Historia clase pública: IEnumerable embargo, como resultado I' No obtuve '0' como esperaba, sino '-1'. No puedo entender, ¿por qué?

Answer 1

Cuando se escribe Pages.IndexOf(x => x == Pages.ElementAt(0));, en realidad se ejecuta InSetsOfmuchas veces, debido a deferred execution (también conocido como perezoso). Para ampliar:

• Pages = history.InSetsOf<Message>(500) - esta línea no se ejecuta InSetsOf en absoluto.
• Pages.IndexOf - Itera más de Pages, por lo que comienza a ejecutar InSetsOf una vez.
• x == Pages.ElementAt(0) - esto ejecuta InSetsOf otra vez, una vez por cada elemento en la colección de Pages (o al menos hasta searchPredicate devuelve verdadero, lo que no sucede aquí).

Cada vez que se ejecuta InSetsOf se crea una nueva lista (en concreto, una nueva primera lista, porque utiliza ElementAt(0)). Estos son dos objetos diferentes, por lo que la comparación de == falla.

Una solución muy sencilla sería la de devolver una lista, por lo Pages no es una consulta diferida, sino una colección concreta:

Pages = history.InSetsOf<Message>(500).ToList(); 

Otra opción es utilizar SequenceEqual, aunque me gustaría recomendar el almacenamiento en caché la primera elemento de todos modos:

Pages = history.InSetsOf<Message>(500); 
var firstPage = Pages.FirstOrDefault(); 
var index = Pages.IndexOf(x => x.SequenceEqual(firstPage)); 

Answer 2

¿Su clase T implementa el IComparable? De lo contrario, su comprobación de igualdad podría ser defectuosa, ya que el marco no sabe exactamente cuándo T = T. También lo haría sobrescribiendo a los iguales en su clase T, supongo.