List.Add() 问题仅保存最后添加的项目[重复]答案

【问题标题】：Issue with List.Add() only saving the last added item [duplicate]List.Add() 问题仅保存最后添加的项目[重复]
【发布时间】：2018-09-19 05:27:12
【问题描述】：

我注意到的问题是这行代码：

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(orderables); ) 上时，它第一次将项目正确添加到 templist 并且其中会有一个项目。第二次它会将正确的项目添加到列表中但是如果我将鼠标悬停在 tempList 上并想查看其内容，虽然它有两个项目，但它们都是相同的 - 它们都是现在是添加到列表中的第二项。它已经覆盖了第一个。

我无法弄清楚这出了什么问题以及为什么会发生。

【问题讨论】：

tempList.Clear();清空列表

标签： c#

【解决方案1】：

你需要在for循环中实例化Orderables内；否则，你会在所有迭代中重复使用同一个实例（并且每次都覆盖它的属性）。

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

与问题无关：您可能会发现object initializer 语法更简洁：

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

【讨论】：

谢谢。那解决了它。出于我的学习目的：我认为一个对象就足够了，因为每次我为其分配新值并将其传递给列表时……是因为它是引用类型吗？这就是原因？
没错。引用类型（类）与值类型（结构）不同，因为后者是在赋值时复制的（例如，当传递给 List<T>.Add 方法时），而前者只复制它们的 reference （指针），但是仍然会指向同一个对象（实例）。因此，通过引用它的任何变量对对象状态的更改也会“出现”在所有其他变量上。

【解决方案2】：

问题是您只有一个可订购的实例，并且您不断更改同一个实例并将其重新添加到列表中。列表中的每个引用都指向同一个对象。将 orderables 声明 inside 移动到内部 for 循环中，它将解决问题。

【讨论】：