List.Add() 仅保存最后添加的项目的问题

本文关键字：添加 项目 问题 最后 保存 Add List | 更新日期: 2023-09-27 18:34:28

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

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上并想查看其内容，尽管它有两个项目，但它们都是相同的 - 它们现在都是添加到列表中的第二个项目。它覆盖了第一个。

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

List.Add() 仅保存最后添加的项目的问题

您需要在 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);
}

与以下问题无关：您可能会发现对象初始值设定项语法更简洁：

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

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

相关文章：