Remplacer ForEach imbriqué avec Sélectionner si applicable

Question

Est-il possible de remplacer l'utilisation de la méthode ForEach() par Select() ou autre chose pour écrire le code suivant dans une chaîne avec des méthodes d'extension imbriquées? OU peut-être existe-t-il d'autres façons d'améliorer l'algorithme?

var list = new List<IStatementParser>(); 

System.IO.Directory.GetFiles(path, "*.dll") 
    .ForEach(f => System.Reflection.Assembly.LoadFrom(f) 
     .GetTypes() 
     .Where(t => !t.IsInterface && typeof(IFoo).IsAssignableFrom(t)) 
     .ForEach(t => list.Add((IFoo)Activator.CreateInstance(t)))); 

return list.ToDictionary(k => k.Name, v => v.GetType()); 

Il charge toutes les classes d'ensembles en path qui implémente IFoo et les ajoute à Dictionary<string, Type> où la chaîne est IFoo.Name

Answer 1

var foos = 
    from dllFile in Directory.GetFiles(path, "*.dll") 
    from type in Assembly.LoadFrom(dllFile).GetTypes() 
    where !type.IsInterface && typeof(IFoo).IsAssignableFrom(type) 
    select (IFoo) Activator.CreateInstance(type); 

return foos.ToDictionary(foo => foo.Name, foo => foo.GetType()); 

Answer 2

Je ne vois pas la nécessité d'un intermédiaire List tout ici - vous pouvez juste faire ceci:

return (from dll in Directory.GetFiles(path, "*.dll") 
     let asm = Assembly.LoadFrom(dll) 
     from t in asm.GetTypes() 
     where !t.IsInterface && typeof(IFoo).IsAssignableFrom(t) 
     select (IFoo)Activator.CreateInstance(t) 
     ).ToDictionary(foo => foo.Name, foo => foo.GetType()) 

Par ailleurs, vous pouvez également vérifier si un type est abstract avant d'essayer de instancier.