Как добавить свойство ICollection при итерации типа объекта?

Question

Мне нужно добавить к собственности ICollection<string> класса, в котором у меня есть IEnumerable. Вот полная программа, которая иллюстрирует проблему:

using System; 
using System.Collections.Generic; 
using System.Linq; 

namespace CollectionAddingTest 
{ 
    public class OppDocumentServiceResult 
    { 
     public OppDocumentServiceResult() 
     { 
      this.Reasons = new List<string>(); 
     } 

     public Document Document { get; set; } 

     public bool CanBeCompleted 
     { 
      get 
      { 
       return !Reasons.Any(); 
      } 
     } 

     public ICollection<string> Reasons { get; private set; } 
    } 

    public class Document 
    { 
     public virtual string Name { get; set; } 
    } 

    public class Program 
    { 
     private static void Main(string[] args) 
     { 
      var docnames = new List<string>(new[] {"test", "test2"}); 

      var oppDocResult = docnames 
       .Select(docName 
         => new OppDocumentServiceResult 
           { 
            Document = new Document { Name = docName } 
           }); 

      foreach (var result in oppDocResult) 
      { 
       result.Document.Name = "works?"; 
       result.Reasons.Add("does not stick"); 
       result.Reasons.Add("still does not stick"); 
      } 

      foreach (var result in oppDocResult) 
      { 
       // doesn't write "works?" 
       Console.WriteLine(result.Document.Name); 

       foreach (var reason in result.Reasons) 
       { 
        // doesn't even get here 
        Console.WriteLine("\t{0}", reason); 
       } 
      } 
     } 
    } 
} 

Я бы ожидать, что каждый OppDocumentServiceResult бы его ссылки Document.Name набор свойств для работ?, и каждый OppDocumentServiceResult должен иметь две причины. Однако ни то, ни другое не происходит.

Что такое свойство «Причины», которое я не могу добавить к нему?

Answer 1

Фиксированный как это, преобразование в список вместо того, чтобы в IEnumerable:

var oppDocResult = docnames 
     .Where(docName => !String.IsNullOrEmpty(docName)) 
     .Select(docName 
      => new OppDocumentServiceResult 
      { 
       Document = docName 
      }).ToList(); 

Я могу только догадываться (это выстрел в темноте действительно!), что причина в том, что в IEnumerable элементы похожи на «прокси» реальных элементов? в основном Enumerable, определяемый запросом Linq, подобен «обещанию» для получения всех данных, поэтому каждый раз, когда вы повторяете, вы возвращаете исходные элементы? Это не объясняет, почему нормальное свойство до сих пор торчит ...

Таким образом, исправление там, но объяснение, которое я боюсь, не ... не от меня, по крайней мере :(

Answer 2

ForEach() определяется только против List<T> вы не быть в состоянии использовать его для ICollection<T>.

Вы должны варианты:

((List<string>) Reasons).ForEach(...) 

Или

Reasons.ToList().ForEach(...) 

---

Тем не менее, мой предпочтительный подход

Я бы определил это расширение, которое может помочь автоматизировать это для вас без потерь ресурсов:

public static class ICollectionExtensions 
{ 
    public static void ForEach(this ICollection<T> collection, Action<T> action) 
    { 
     var list = collection as List<T>; 
     if(list==null) 
      collection.ToList().ForEach(action); 
     else 
      list.ForEach(action); 
    } 
} 

Теперь я могу использовать ForEach() против ICollection<T>.

Answer 3

Просто изменить код внутри класса

public List<string> Reasons { get; private set; } 

Answer 4

Вопрос ваш первоначальный Select вы инстанцирование новых OppDocumentServiceResult объектов Добавить ToList и вы должны быть хорошо идти:.

var oppDocResult = docnames 
    .Select(docName 
      => new OppDocumentServiceResult 
        { 
         Document = new Document { Name = docName } 
        }).ToList(); 

Как Servy указал я добавил немного более подробно на мой ответ, но, к счастью the comment он оставил на Tallmaris' answer принимает забота об этом. В своем ответе Джон Ск eet далее расширяется по причине, но то, что она сводится к «заключается в том, что oppDocResult является результатом запроса LINQ с использованием отложенного выполнения».

Answer 5

Проблема заключается в том, что oppDocResult является результатом запроса LINQ с использованием отложенного выполнения ,

Другими словами, каждый раз, когда вы перебираете его, выполняется запрос и создаются OppDocumentServiceResult. Если вы поместите диагностику в конструктор OppDocumentServiceResult, вы увидите это.

Так что объекты OppDocumentServiceResult, которые вы повторяете в конце, отличаются от тех, к которым вы добавили причины.

Теперь, если вы добавите ToList() вызов, то, что материализуется запрос в «простой» коллекции (а List<OppDocumentServiceResult>). Каждый раз, когда вы перебираете этот список, он будет возвращать ссылки на одни и те же объекты, поэтому, если вы добавите причины при первом переходе по ним, вы распечатываете причины, когда вы повторите их снова, вы получите результаты, которые вы «Ищем.

См. this blog post (среди многих результатов поиска для «отложенного выполнения LINQ») для получения более подробной информации.