IEnumerable - foreach döngüsünün içindeki nesneleri güncelle

Question

Nesnelerin birçoğunu oluşturan ve her bir nesnenin Priority özelliğini ayarlamak için bunların üzerinden yineleyen gerçekten basit bir programım var.

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; } 
} 

Değiştirilmiş önceliklerle nesneleri içerecek Ana yöntemde IEnumerable bekliyorum.

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 davranışın nedeni nedir ve ben çalışıyorum benim öncelikleri almak için buraya neyi değiştirmelisiniz: Ancak, hepsi varsayılan değeri 0. Burada günlük olduğunu var?

Answer 1

Bunu yaptığınızda: Sadece bir lazily değerlendirilen yineleyici oluştururken

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

, bu ObjectWithPriorty size proje depolamak için bir dizi/liste tahsis etmez. Yineleyiciyi numaralandırdığınız her seferde, değerleri yinelemek ve her yineleme için bir ObjectWithPriority yansıtmak, ancak bunları atar.

Yapmak istediğiniz şey, iletmeden önce sorguyu yansıtmaktır, bu nedenle daha sonra önceden ayrılmış liste/diziyi gerçekten değiştireceksiniz. Bu Enumerable.ToList veya Enumerable.ToArray kullanılarak elde edilebilir:

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

Yapabilirsin ek kullanım Enumerable.Range yerine istendiği gibi tembel numaralarını proje olacak Sabit boyutlu bir dizi, tahsis:

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

Answer 2

Yanıt Yanı Sıra Yuval Itzchakov ait:

: Yapmak istediğiniz

yapabildin senin nesnelerin öncelik tembel-yük

sadece bir nesne için ApplyPriorities() Yöntem tanımlayın basıp-Yöntem kullanmak VEYA ObjectWithPriority sınıf wich için bir temsilci eklemek için aşağıdaki kodu gösterildiği gibi Öncelik hesaplar:

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

daha iyi neler olduğunu anlamak için programda, bir sorgusu olarak bu ifadenin

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

düşünmek gerekir, değil nesneleri dizisi/listeye/koleksiyonu olarak.

Ancak bu programda bir sorguya ihtiyacınız olmadığını kodunuzdan açıkça göreceksiniz. Sonlu sayıda nesneye sahip bir koleksiyona ihtiyacınız var ve foreach'u kullanarak her geçişinizde aynı nesneleri döndürür.

Yapılması gereken en iyi şey, IEnumerable<ObjectWithPriority> yerine ICollection<ObjectWithPriority> kullanmak olacaktır. Bu, programın mantığını daha iyi temsil eder ve sizin tökezlediğiniz gibi bazı yanlış/yanlış yorumlamalardan kaçınmanıza yardımcı olur.

aşağıdaki gibi kod modifiye edilebilir: Eğer LINQ hakkında içselleştirmelidir

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