ネストされたForEachを置換する場合は選択

Question

メソッドForEach()をSelect()などと置き換えて、ネスト拡張メソッドを使用して次のコードを1つの文字列に書き込むことはできますか？または、アルゴリズムを改善する別の方法がありますか？

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()); 

それはIFooを実装し、文字列が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

である私は全然ここで中間Listの必要性を見ないDictionary<string, Type>に追加することpathにアセンブリからすべてのクラスをロードする - あなただけ行うことができますこの：

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()) 

ところで、あなたはまたタイプがそれをインスタンス化しようとする前にabstractであるかどうかを確認したい場合があります。