Question

我在消息框中显示变量时遇到问题。我想要做的是在消息框中显示Combobox尚未填写，它将在消息框的列表中显示，然后阻止用户保存到数据库。该错误表明它是使用未分配的变量，但我已将其分配在'if'语句的顶部。

private void btnSaveDB_Click(object sender, EventArgs e)
    {
        if (cmbPolType.SelectedItem == null || 
            cmbPolNum.SelectedItem == null ||
            cmbTPReg.SelectedItem == null || 
            cmbLossType.SelectedItem == null || 
            cmbLossDesc.SelectedItem == null ||
            cmbInsdFault.SelectedItem == null)
        {
            string polType, polNum, lossType, lossDesc, tpReg, insdFault = null;

            if (cmbPolType.SelectedItem==null)
            {
                polType = "Policy Type";
            }

            if (cmbPolNum.SelectedItem==null)
            {
                 polNum = "Policy Number";
            }
            if (cmbLossType.SelectedItem==null)
            {
                lossType = "Loss Type";
            }
            if (cmbLossDesc.SelectedItem ==null)
            {
                lossDesc = "Loss Description";
            }
            if (cmbTPReg.SelectedItem==null)
            {
                tpReg = "TP Reg";
            }
            if (cmbInsdFault.SelectedItem==null)
            {
                insdFault = "Insd at Fault";
            }

            MessageBox.Show("You have not selected options for the following: " + lossDesc );
        }

Answer 1

没有 lossDesc 以这种方式初始化以及其他字符串变量而不是 insdFault 。（错误消息指向 lossDesc ，因为它是代码其余部分中使用的唯一一个）。

我建议使用简单的List<string>，而不是初始化每个错误消息，并在测试结束时输入所有错误消息

List<string> missingData = new List<string>();
if (cmbPolType.SelectedItem == null)
    missingData.Add("Policy Type");
if (cmbPolNum.SelectedItem == null)
    missingData.Add("Policy Number");
if (cmbLossType.SelectedItem == null)
    missingData.Add("Loss Type");
if (cmbLossDesc.SelectedItem == null)
    missingData.Add("Loss Description");
if (cmbTPReg.SelectedItem == null)
    missingData.Add("TP Reg");
if (cmbInsdFault.SelectedItem == null)
    missingData.Add("Insd at Fault");

if(missingData.Count > 0)
{
   MessageBox.Show("You have not selected options for the following: " + 
                    Environment.NewLine + 
                    string.Join(Environment.NewLine, missingData.ToArray()));
}
else
{

   ... save to database ? ....
}

这消除了使用和初始化一堆字符串变量的需要，并使用string.Join方法将单个字符串中的整个错误消息与每个错误放在一个单独的行中。

在messagebox中显示if语句变量

1 个答案: