LINQを使って2 foreachを簡略化するにはどうすればよいですか？

Question

これをよりわかりやすい方法で実行するにはどうすればよいですか？

Answer 1

簡略化したいですか？どうして？シンプルでわかりやすく理解できます。あなたはより硬いであるが、確かによりクールに見えるすべての種類のゴブリンジグックを考え出すことができる。しない！思い切った衝動に抵抗してください。

Answer 2

return 
    from actual in actualProjections 
    from projection in projections 
    select Create(book, actual, projection) 
      .With_TargetNett(projection.DailyProjectedNet.ToString()); 

With_TargetNettが拡張子ですあなたが行うことができます

static Position With_TargetNett(this Position position, string targetNett) 
{ 
position.TargetNett = targetNett; 
return position; 
} 

Answer 3

1つのオプションは、あなた自身のクロス結合を実装し、その上で列挙することです。しかし、それは非常にきれいまだありません。

actualPositions.SelectMany(actual => projections.Select(project => new { actual, project })) 

しかし、あなたは.NET 4.0以降を使用している場合、あなたはクロスタプル（デフォルト）または独自のカスタムセレクタを使用して拡張メソッドに参加実装できます。

public static IEnumerable<Tuple<T1, T2>> CrossJoin<T1, T2>(this IEnumerable<T1> source, IEnumerable<T2> toJoin) 
{ 
    return source.CrossJoin(toJoin, Tuple.Create<T1, T2>); 
} 

public static IEnumerable<T3> CrossJoin<T1, T2, T3>(this IEnumerable<T1> source, IEnumerable<T2> toJoin, Func<T1, T2, T3> selector) 
{ 
    return source.SelectMany(tFirst => toJoin.Select(tSecond => selector(tFirst, tSecond))); 
} 

そして、これは非常に簡単に使用することができます。

foreach(var pair in actualPositions.CrossJoin(projections)) 
{ 
    var position = Create(book, pair.Item1, pair.Item2); 
    position.TargetNett = pair.Item2.DailyProjectedNet.ToString(); 
    yield return position; 
} 
// OR 
return actionPositions.CrossJoin(projections, (actual, project) => { /*Code in foreach loop above here, without the "yield". */ }); 

Answer 4

だけで楽しい

return 
(
    from actual in actualProjections 
    from projection in projections 
    select new {actual, projection} 
).Select(pair => 
    { 
    var position = Create(book, pair.actual, pair.projection); 
    position.TargetNett = pair.projection.DailyProjectedNet.ToString();   
    return position; 
    } 
); 

のために

Answer 5

この場合、Linqは読みやすさが低下します。 あなたが別々の方法で二foreach文を移動して、新しいメソッドにLINQ

return from actual in actualPositions 
     select GetBookPositionList(actual); 

を使用することができます。

private List<BookPosition> GetBookPositionList(ActualPosition actual) 
{ 
    foreach (var projection in projections) 
    { 
    var position = Create(book, actual, projection); 
    position.TargetNett = projection.DailyProjectedNet.ToString(); 
    yield return position; 
    } 
}