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);) にブレークポイントを置くと、最初に項目が正しく追加され、templist1 つの項目が含まれます。2回目は正しいアイテムをリストに追加しますが、カーソルを合わせてその内容を表示したい場合tempList、2つのアイテムがありますが、両方とも同じです-両方ともリストに追加された2番目のアイテムです. 最初のものを上書きしました。

これで何が問題になっているのか、なぜそれが起こっているのかわかりません。

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

問題は、注文可能なインスタンスが1つしかないため、同じインスタンスを変更してリストに再度追加し続けることです。リスト内の各参照は、同じオブジェクトを指しています。orderables宣言を内側のforループ内に移動すると、問題が修正されます。

于 2012-05-17T22:38:31.123 に答える