C＃未能在IEnumerable中设置属性

Question

在一些C＃代码中看到了一些奇怪的行为，我不知所措。可能是我错过了一个重要的理解，所以希望有人可以为我开灯。

得到的代码块看起来像这样：

IEnumberable<myObject> objects = GetObjectsFromApiCall(); 

    for (int i = 0; i < objects.Count(); i++) 
     { 
      if (String.IsNullOrEmpty(objects.ElementAt(i).SubObject.Title)) 
      { 
       SubObject sub = GetSubObjectFromDatabase((long)objects.ElementAt(i).SubObject.Id); 
       if (sub != null) 
       { 
        objects.ElementAt(i).SubObject.Title = sub.Title; 
       } 
      } 
     } 

当你通过它一步，这段代码的一切似乎正常工作。 “对象”集合按预期填充。 “sub”被提取为已收集并具有一整套预期属性，包括已填充的Title属性。在执行过程中不会引发错误。

...但是... SubObject.Title属性（它只是有标准的get;设定;代码）存在于每个对象固执地保持为空。

我不知所措。任何人都可以解释发生了什么？

编辑：对于那些建议我不应该使用for循环和ElementAt，我开始使用foreach循环，但认为它可能是问题的来源，因为它每次都获取新的SubObject。现在修复，感谢您的帮助，并恢复了ForEach。

干杯， 马特

Answer 1

我会解决这个问题是这样的：

var objects = GetObjectsFromApiCall().ToList(); 

然后，你可以保持环路是（它的工作原理），或优化有点用的foreach和一些LINQ的其他答案的建议，但它确实并不重要：问题在于您试图更改IEnumerator上的元素，如@AhmetKakıcı指出的this question中所述。

Answer 2

首先，你不应该使用ElementAt()对于这种代码，使用

foreach (var o in objects) 
{ 
    if (string.IsNullOrEmpty(o.SubObject.Title)) 
    { 
     o.SubObject.Title = ...; 
    } 
} 

另外，应注意的是，如果你的方法返回一个动态IEnumerable那么每次您致电objects.Something()再次调用API并检索新鲜副本。如果是这种情况，则应使用.ToList()方法将枚举复制到列表中。

还有的不把副本列表中的一种方式 - 通过创建一个动态枚举是这样的：

objects = objects.Select(o => 
{ 
    if (string.IsNullOrEmpty(o.SubObject.Title)) 
    { 
     o.SubObject.Title = ...; 
    } 
    return o; 
}); 

至于没有被正确设置（如果以前的事情没有帮助）的值 - 尝试在设置器中添加throw new Exception(value)以获取Title属性 - 查看是否正在使用正确的值调用该属性。

Answer 3

试试这个

List<myObject> objects = GetObjectsFromApiCall().ToList(); 

foreach(var obj in objects.Where(o => string.IsNullOrEmpty(objects.SubObject.Title)).ToList()) 
{ 
    var subObject = GetSubObjectFromDatabase(obj.SubObject.Id); 
    if(subObject == null) continue; 

    obj.SubObject.Title = subObject.Title; 
} 

Answer 4

我来宾功能GetObjectsFromApiCall看起来像以下：

public IEnumberable<myObject> GetObjectsFromApiCall(){ 
    for(var i = 0; i < 10; i++) 
    { 
     yield return new myObject(); 
    } 
} 

如果我是正确的，每次打电话objects.ElementAt（I）函数来获取对象，您将通过“yield return new myObject()”获得一个新对象。

Answer 5

但是，如何检查Title属性是否发生了变化？你再拨打GetObjectsFromApiCall()吗？还是你foreach再次通过相同的objects实例？

IEnumerable实例可能会在每次“枚举”时创建并产生新对象。所以这里有一个简单的插图示例。对于这个示例，定义：

class SomeObject 
{ 
    public string Title { get; set; } 
} 

然后，我们将考虑两种类型的“源极”，第一阵列，然后这样定义的迭代器块：

static IEnumerable<SomeObject> GetSomeSequence() 
    { 
     yield return new SomeObject { Title = "Alpha", }; 
     yield return new SomeObject { Title = "Beta", }; 
     yield return new SomeObject { Title = "Gamma", }; 
    } 

然后这种方式测试：

static void Main() 
    { 
     IEnumerable<SomeObject> thingsToModify; 

     // set source to an array 
     thingsToModify = new[] { new SomeObject { Title = "Alpha", }, new SomeObject { Title = "Beta", }, new SomeObject { Title = "Gamma", }, }; 

     foreach (var t in thingsToModify) 
      Console.WriteLine(t.Title); 

     foreach (var t in thingsToModify) 
      t.Title = "Changed!"; 

     foreach (var t in thingsToModify) 
      Console.WriteLine(t.Title); // OK, modified 


     // set source to something which yields new object each time a new GetEnumerator() call is made 
     thingsToModify = GetSomeSequence(); 

     foreach (var t in thingsToModify) 
      Console.WriteLine(t.Title); 

     foreach (var t in thingsToModify) 
      t.Title = "Changed!";   // no-one keeps these modified objects 

     foreach (var t in thingsToModify) 
      Console.WriteLine(t.Title); // new objects, titles not modified 

    } 

结论：完全可以修改属于我们正在迭代的源的可变对象的状态。但某些类型的IEnumerable源在每次调用时都会产生新的数据副本，然后修改副本无用。