IEnumerable - foreachループ内のオブジェクトを更新する

Question

私は、たくさんのオブジェクトを作成し、各オブジェクトのPriorityプロパティを設定するために繰り返します。

static void Main(string[] args) 
{ 
    foreach (var obj in ObjectCreator.CreateObjectsWithPriorities()) 
     Console.WriteLine(String.Format("Object #{0} has priority {1}", 
             obj.Id, obj.Priority)); 
} 

class ObjectCreator 
{ 
    public static IEnumerable<ObjectWithPriority> CreateObjectsWithPriorities() 
    { 
     var objs = new[] { 1, 2, 3 }.Select(i => new ObjectWithPriority() { Id = i }); 
     ApplyPriorities(objs); 
     return objs; 
    } 

    static void ApplyPriorities(IEnumerable<ObjectWithPriority> objs) 
    { 
     foreach (var obj in objs) 
     { 
      obj.Priority = obj.Id * 10; 
      Console.WriteLine(String.Format("Set priority of object #{0} to {1}", obj.Id, obj.Priority)); 
     } 
    } 
} 

class ObjectWithPriority 
{ 
    public int Id { get; set; } 
    public int Priority { get; set; } 
} 

私は修正の優先順位を持つオブジェクトを格納するための主な方法でIEnumerableを期待しています。

Set priority of object #1 to 10 
Set priority of object #2 to 20 
Set priority of object #3 to 30 
Object #1 has priority 0 
Object #2 has priority 0 
Object #3 has priority 0 

sucheの行動の理由は何であり、私が働いて、私の優先順位を取得するために、ここで何を変更する必要があります。しかし、それらのすべては、デフォルト値0ここで ログがあるがありますか？

Answer 1

あなたがこれを行うときは：あなたは、単に遅延評価イテレータを作成している

var objs = new[] { 1, 2, 3 }.Select(i => new ObjectWithPriority() { Id = i }); 

、これはObjectWithPriortyプロジェクト格納する配列/リストを割り当てません。反復子を列挙するたびに、反復処理ごとに値を繰り返し反復し、ObjectWithPriorityを投影しますが、それらは破棄します。

あなたがやりたいことは、渡す前にクエリーを具体化することです。後で実際に割り当てられたリスト/配列を変更します。これはEnumerable.ToListまたはEnumerable.ToArrayを使用して達成することができます。

public static IEnumerable<ObjectWithPriority> CreateObjectsWithPriorities() 
{ 
    var objs = new[] { 1, 2, 3 }.Select(i => new ObjectWithPriority() { Id = i }) 
           .ToList(); 
    ApplyPriorities(objs); 
    return objs; 
} 

あなたが代わりに要求されるように遅延して数字を投影する固定サイズの配列を、割り当ての追加使用Enumerable.Range可能性：

回答に加えて、

var objs = Enumerable.Range(1, 3).Select(i => new ObjectWithPriority { Id = i }) 
           .ToList(); 

Answer 2

をYuval Itzchakovの：

：

あなたができたあなたのオブジェクトのレイジーロード優先にしたい場合

ただ一つの目的のために、あなたのApplyPriorities()メソッドを定義し、選択-方法でそれを使用するか、または以下のコードに示すようなあなたのObjectWithPriorityクラスウィッヒは、優先度を算出するにデリゲートを追加します。

class ObjectWithPriority 
{ 
    public int Id { get; set; } 

    private int? priority; 
    public int Priority { 
     get 
     { 
      return (priority.HasValue ? priority.Value : (priority = PriorityProvider(this)).Value); 

     } 

     set { priority = value; } 
    } 

    Func<ObjectWithPriority, int> PriorityProvider { get; set; } 

    public ObjectWithPriority(Func<ObjectWithPriority, int> priorityProvider = null) 
    { 
     PriorityProvider = priorityProvider ?? (obj => 10 * obj.Id); 
    } 
} 

Answer 3

より良い何が起こっているのかを理解するためにあなたのプログラムの中で、あなたはこの表現クエリとしてではなく、オブジェクトのシーケンス/リスト/コレクションとして

var objs = new[] { 1, 2, 3 }.Select(i => new ObjectWithPriority() { Id = i }); 

を考える必要があります。

しかし、この特定のプログラムではクエリを必要としないことはあなたのコードから明らかです。有限個のオブジェクトを持ち、foreachを使用してループするたびに同じオブジェクトを返すコレクションが必要です。

IEnumerable<ObjectWithPriority>の代わりにICollection<ObjectWithPriority>を使用することをお勧めします。これは、プログラムのロジックをよりよく表現し、あなたが遭遇したような間違いや誤解を避けるのに役立ちます。

次のようにコードを変更することができます：あなたがLINQについての内部化しなければならない

public static ICollection<ObjectWithPriority> CreateObjectsWithPriorities() 
{ 
    IEnumerable<ObjectWithPriority> queryThatProducesObjectsWithPriorities = new[] { 1, 2, 3 }.Select(i => new ObjectWithPriority() { Id = i }); // just for clarification 
    ICollection<ObjectWithPriority> objectsWithPriorities = queryThatProducesObjectsWithPriorities.ToList(); 
    ApplyPriorities(objectsWithPriorities); 
    return objectsWithPriorities; 
} 

static void ApplyPriorities(ICollection<ObjectWithPriority> objs) 
{ 
    foreach (var obj in objs) 
    { 
     obj.Priority = obj.Id * 10; 
     Console.WriteLine(String.Format("Set priority of object #{0} to {1}", obj.Id, obj.Priority)); 
    } 
}