c# - 检查是否有东西是空的

Question

我目前正在检查组合框和 numericupdowns 是否为空，但我正在使用 If 语句进行检查。

你能告诉我是否还有其他更快的方法吗？

这是当前代码

private void button1_Click(object sender, EventArgs e)
    {
        if(comboBox1.SelectedItem != null)
        {
            if(comboBox2.SelectedItem != null)
            {
                if(numericUpDown1.Value != 0)
                {
                    if(numericUpDown2.Value != 0)
                    {
                        if(numericUpDown3.Value != 0)
                        {
                            if(numericUpDown4.Value != 0)
                            {
                                string domacin = "" + comboBox1.GetItemText(comboBox1.SelectedItem);
                                int D_kosevaPrvoPoluvreme = (int)numericUpDown1.Value;
                                int D_kosevaDrugoPoluvreme = (int)numericUpDown3.Value;
                                int D_ukupnoKoseva = D_kosevaDrugoPoluvreme + D_kosevaPrvoPoluvreme;
                                int D_primljenihKosevaPrvoPoluvreme = (int)numericUpDown2.Value;
                                int D_primljenihKosevaDrugoPoluvreme = (int)numericUpDown4.Value;
                                int D_ukupnoPrimljenihKoseva = D_primljenihKosevaDrugoPoluvreme + D_primljenihKosevaPrvoPoluvreme;

                                string gost = "" + comboBox2.GetItemText(comboBox2.SelectedItem);
                                int G_kosevaPrvoPoluvreme = (int)numericUpDown2.Value;
                                int G_kosevaDrugoPoluvreme = (int)numericUpDown4.Value;
                                int G_ukupnoKoseva = G_kosevaDrugoPoluvreme + G_kosevaPrvoPoluvreme;
                                int G_primljenihKosevaPrvoPoluvreme = (int)numericUpDown1.Value;
                                int G_primljenihKosevaDrugoPoluvreme = (int)numericUpDown3.Value;
                                int G_ukupnoPrimljenihKoseva = G_primljenihKosevaDrugoPoluvreme + G_primljenihKosevaPrvoPoluvreme;

                                int rezultat;
                                Functions.odrediPobednika(out rezultat, D_ukupnoKoseva, G_ukupnoKoseva);

                                //SqlConnection con = new SqlConnection(@"Data Source=(LocalDB)\MSSQLLocalDB;AttachDbFilename=C:\Users\arist\Documents\VisualStudio2015\Projects\NBA\NBA\NBA.mdf;Integrated Security=True");
                                //SqlCommand cmd = new SqlCommand("", con);
                            }
                        }
                    }
                }
            }
        }            
    }

Answer 1

我会使用以下带有有意义消息的快速失败验证方法:

private void button1_Click(object sender, EventArgs e)
{

    if (comboBox1.SelectedItem == null)
    {
        MessageBox.Show("Meaningful message, user should know what he has done wrong", "Invalid xyz");
        return;
    }
    else if (comboBox2.SelectedItem == null)
    {
        MessageBox.Show("Meaningful message, user should know what he has done wrong", "Invalid xyz");
        return;
    }
    else if (numericUpDown1.Value == 0)
    {
        MessageBox.Show("Meaningful message, user should know what he has done wrong", "Invalid xyz");
        return;
    }
    else if (numericUpDown2.Value == 0)
    {
        MessageBox.Show("Meaningful message, user should know what he has done wrong", "Invalid xyz");
        return;
    }
    else if (numericUpDown3.Value == 0)
    {
        MessageBox.Show("Meaningful message, user should know what he has done wrong", "Invalid xyz");
        return;
    }

    // insert code
}

它既不更短也不更高效，但它易于阅读和调试，最重要的是:用户知道哪里出了问题。您还应该使用有意义的控件名称。

当然，您也可以使用 if 并使用 && 连接所有条件，但出于上述原因，我更喜欢我的第一种方法。

bool valid = comboBox1.SelectedItem != null 
          && comboBox2.SelectedItem != null 
          && numericUpDown1.Value != 0 
          && numericUpDown2.Value != 0 
          && numericUpDown3.Value != 0 
          && numericUpDown4.Value != 0;
if(valid){
    //Code
}