Méthode d'extension IEnumerable et Datareader, suggestions requises

Question

est la méthode d'extension permettant de convertir un lecteur de données en objet à l'aide de la classe "DataReaderInjection" basée sur l'injecteur de valeur OMU.

public static IEnumerable<T> MapTo<T>(this IDataReader reader) where T : new() { 
    return reader.MapTo<T>(null); 
} 

public static IEnumerable<T> MapTo<T>(this IDataReader reader, Action<IDataReader, T> customMappingFunction) where T : new() { 
    using (reader) 
     while (!reader.IsClosed && reader.Read()) { 
      var nextItem = new T(); 
      nextItem.InjectFrom<DataReaderInjection>(reader); 
      if (customMappingFunction != null) 
       customMappingFunction(reader, nextItem); 
      yield return nextItem; 
     } 
} 

cette méthode a un bug car une fois que vous appelez cette méthode et datareader est fermé, prochain appel retournerait un vide IEnumerable, Pour résoudre ce problème je l'ai changé le code ci-dessus à la suite.

public static class DataAccessExtensions 
{ 
    public static IEnumerable<T> MapTo<T>(this IDataReader reader) where T : new() { 
     return reader.MapTo<T>(null); 
    } 

    private static Dictionary<int, IEnumerable> dataReaderCache = new Dictionary<int, IEnumerable>(); 
    private static object lockObj = new object(); 

    public static IEnumerable<T> MapTo<T>(this IDataReader reader, Action<IDataReader, T> customMappingFunction) where T : new() { 
     lock (lockObj) 
     { 
      if (dataReaderCache.ContainsKey(reader.GetHashCode())) 
       return dataReaderCache[reader.GetHashCode()] as IEnumerable<T>; 
     } 
     List<T> finalList = new List<T>(); 
     using (reader) { 
      while (!reader.IsClosed && reader.Read()) { 
       var nextItem = new T(); 
       nextItem 
        .InjectFrom<DataReaderInjection>(reader); 
       if (customMappingFunction != null) 
        customMappingFunction(reader, nextItem); 
       finalList.Add(nextItem); 
      } 
     } 
     lock (lockObj) 
     { 
      dataReaderCache.Add(reader.GetHashCode(), finalList); 
     } 
     return finalList; 
    } 

}

Qu'est-ce que vous en pensez cela fonctionnera

Answer 1

... cette méthode a un bug car une fois que vous appelez cette méthode et datareader est fermé, prochain appel retournerait un vide IEnumerable

Ce n'est pas un bogue, les gestionnaires de données sont des itérateurs uniquement en avant, vous ne pouvez les répéter qu'une seule fois. Si vous devez parcourir plus d'une fois les résultats, mettez en cache l'IEnumerable que vous renvoyez. La manière la plus simple de faire ceci est de l'envelopper dans un List<T>, ou vous pouvez le mettre en cache paresseusement en utilisant Rx's MemoizeAll()