4

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

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

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

4

2 回答 2

8

您需要在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,
};
于 2012-05-17T22:38:37.830 回答
2

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

于 2012-05-17T22:38:31.123 回答