Проблема с List.Add(): он сохраняет только последний добавленный элемент

Question

в строке кода для

tempList.Add(orderables); 

в этом полный код:

AssociatedComboItems ai = new AssociatedComboItems(); 
    List<Orderables> tempList = new List<Orderables>(); 
    Orderables orderables = new Orderables(); 

    foreach (var t in comboBox1.Items) 
    { 
     ai.ComboBoxItem = t.ToString(); 

     for (int i = 0; i < fpSpread1.ActiveSheet.RowCount; i++) 
     { 
      orderables.Display = fpSpread1.ActiveSheet.Cells[i, 1].Text; 
      orderables.ShowInDSR = (bool)fpSpread1.ActiveSheet.Cells[i, 0].Value; 
      orderables.DisplayOrder = i; 
      tempList.Add(orderables); 
     } 

     ai.AssociatedItems = tempList; 
     tempList.Clear(); 
     if(AssociatedItems == null) 
     AssociatedItems = new List<AssociatedComboItems>(); 
     AssociatedItems.Add(ai); 
    } 

, когда я положил контрольную точку на линии (tempList.Add (заказываемые данные);) при первом добавлении элемента к templist, и в нем будет один элемент. Во второй раз это будет правильный элемент в списке, но если я навешиваю tempList и хочу увидеть его содержимое, хотя он имеет два элемента, но оба они одинаковы, и оба они теперь являются вторым элементом, который добавляется к список, поэтому он перезаписал первый, ...

Я не могу понять, что с этим происходит, и почему это происходит.

Спасибо всем.

Answer 1

Вам необходимо создать экземпляр Orderablesв пределах петля for; в противном случае вы продолжаете использовать один и тот же экземпляр во всех итерациях (и каждый раз переписывая его свойства).

AssociatedComboItems ai = new AssociatedComboItems(); 
List<Orderables> tempList = new List<Orderables>(); 

foreach (var t in comboBox1.Items) 
{ 
    ai.ComboBoxItem = t.ToString(); 

    for (int i = 0; i < fpSpread1.ActiveSheet.RowCount; i++) 
    { 
     Orderables orderables = new Orderables(); // ← Instantiate here 
     orderables.Display = fpSpread1.ActiveSheet.Cells[i, 1].Text; 
     orderables.ShowInDSR = (bool)fpSpread1.ActiveSheet.Cells[i, 0].Value; 
     orderables.DisplayOrder = i; 
     tempList.Add(orderables); 
    } 

    ai.AssociatedItems = tempList; 
    tempList.Clear(); 
    if(AssociatedItems == null) 
    AssociatedItems = new List<AssociatedComboItems>(); 
    AssociatedItems.Add(ai); 
} 

Unrelated на вопрос: Вы могли бы найти object initializer синтаксис быть чище:

 Orderables orderables = new Orderables 
     { 
      Display = fpSpread1.ActiveSheet.Cells[i, 1].Text, 
      ShowInDSR = (bool)fpSpread1.ActiveSheet.Cells[i, 0].Value, 
      DisplayOrder = i, 
     }; 

Answer 2

Проблема заключается в том, что у вас есть только один экземпляр заказчиков, и вы продолжаете изменять один и тот же экземпляр и повторно добавлять его в список. Каждая ссылка в списке указывает на тот же объект. Переместите объявление порядка внутри внутренний цикл, и он устранит проблему.