Question

我编写了以下方法来加载一个列表框，其中包含尚未加载的值，但是在分配以下内容时，我得到的Object引用未设置为对象异常的实例。任何信息都有帮助。谢谢。

lbxCabinetName.Items.Add(cabinetsCurrentNotUsed[i].ToString());

// Defined outside a method
List<string> cabinetsCurrentNotUsed; 

// Set value in the constructor
cabinetsCurrentNotUsed = new List<string>();

这是整个程序。

    private void selectCabinetToAdd()
    {

        // Loop through all server and form cabinet types to see if there are matches
        for (int x = 0; x < opticalFile.serverCabinetNames.Count; x++)
        {
            bool cabinetExists = false;
            for (int i = 0; i < opticalFile.CabinetValues.Count; i++)
            {
                if (opticalFile.serverCabinetNames[x].ToString() == opticalFile.CabinetValues[i].ToString())
                {
                    cabinetExists = true;
                }
            }
            // Add cabinets not used to cabinetsCurrentNotUsed List
            if (!cabinetExists)
            {
                cabinetsCurrentNotUsed.Add(opticalFile.serverCabinetNames[x].ToString());
            }
        }

        // Send cabinetsCurrentNotUsed List to list box
        for (int i = 0; i < cabinetsCurrentNotUsed.Count; i++)
        {
            lbxCabinetName.Items.Add(cabinetsCurrentNotUsed[i].ToString());
        }
    }

Answer 1

您正尝试向列表框添加null。

Insted of

for (int i = 0; i < cabinetsCurrentNotUsed.Count; i++)
{
     lbxCabinetName.Items.Add(cabinetsCurrentNotUsed[i].ToString());
}

使用

foreach (string s in cabinetsCurrentNotUsed)
{
    if(s != null)
          lbxCabinetName.Items.Add(s);
}

注意

这部分与问题无关。但是在设置cabinetExists = true;之后你的内部for循环中你可以突破内循环（如果满足至少一个条件，你可以确保cabinetExists是真的。你不必检查其余的项目在内循环中）

修改

private void selectCabinetToAdd() { foreach (string sc in serverCabinetNames) { bool cabinetExists = false; foreach (string cv in CabinetValues) { if (sc == cv) { cabinetExists = true; break; } } if (!cabinetExists) { cabinetsCurrentNotUsed.Add(sc); } } foreach (string ccnu in cabinetsCurrentNotUsed) { if (ccnu != null) lbxCabinetName.Items.Add(ccnu); } }

此外，如果您的listBox可以为null，请确保在填充列表框之前先检查它。

if(lbxCabinetName != null) { selectCabinetToAdd(); }

编辑2

动态添加控件

ListBox lbxCabinetName = new ListBox(); lbxCabinetName.Location = new System.Drawing.Point(10, 55); lbxCabinetName.Size = new System.Drawing.Size(130, 95); this.Controls.Add(lbxCabinetName);

Answer 2

字符串可以为空，所以在某些时候你必须做类似的事情：

cabinetsCurrentNotUsed.Add(null);

或者dbaseman说可能lbxCabinetName是空的但我认为情况并非如此。

顺便说一下，这不是一个真正的问题，但如果您使用的是通用的字符串列表，则ToString()调用不是必需的。你可以这样做：

lbxCabinetName.Items.Add(cabinetsCurrentNotUsed[i]);

将列表分配给列表框时，对象引用未设置为对象异常的实例

2 个答案: