复选框的If-Else语句

Question

我有以下问题：

当我的程序检查一个确定的条件，如果复选框被选中，它会抛出MessageBox两次；根据调试器，它进入if，抛出messagebox，然后发出Checkbox1.Checked == false。之后，它再次执行整个if条件。

我尝试过关于MSDN的Checkbox1.Threestate文档，但是无法实现它。我能做些什么来解决这个复选框问题？

   private void checkBox1_CheckedChanged(object sender, EventArgs e)
    {

        string currPath = textBox1.Text;
        if (!textBox1.Text.Contains("\\"))
        {
            MessageBox.Show("Please define the input folder before starting");
            checkBox1.Checked = false;                
        }
        else if (!textBox2.Text.Contains("\\"))
        {
            MessageBox.Show("Please define the XML Output folder before starting");
            checkBox1.Checked = false;
        }
        else if (!textBox3.Text.Contains("\\"))
        {
            MessageBox.Show("Please define the Converted PPF Output Folder before starting");
            checkBox1.Checked = false;
        }
        else if (!textBox4.Text.Contains("\\"))
        {
            MessageBox.Show("Please define the Invalid PPF Output Folder before starting");
            checkBox1.Checked = false;
        }
        else
        {
            // calls the watcher
            prg.ProgramProcessing(textBox1.Text);
        }
    }

Answer 1

由于您更改了checkchange中的检查，因此它第二次触发它，因此您应该取消订阅该事件，然后重新订阅。

private void checkBox1_CheckedChanged(object sender, EventArgs e)
{
    checkBox1.CheckedChanged -= checkBox1_CheckedChanged;

    try
    {
        // Your conditions
    }
    finally
    {
        checkBox1.CheckedChanged += checkBox1_CheckedChanged;
    }
}

Answer 2

一旦设置了Checked = false，就会更改它的值，并再次触发Changed事件。

解决这一问题的一种方法是使用一个保护变量，以确保在处理事件时不会再次遍历代码：

private bool handlingCheckboxChanged;
private void checkBox1_CheckedChanged(object sender, EventArgs e)
{
  if(handlingCheckboxChanged) { return; }
  handlingCheckboxChanged = true;
  try {
    // Your original code
  } finally {
    handlingCheckboxChanged = false;
  }
}

更好的做法是首先确保用户不能选中复选框，例如，处理文本框的更改事件，并且只在满足所有条件后才启用复选框。

Answer 3

如果没有选中复选框，可以跳过它。

private void checkBox1_CheckedChanged(object sender, EventArgs e)
{
    if (!checkBox1.Checked) return; //<- this.

    string currPath = textBox1.Text;
    if (!textBox1.Text.Contains("\\"))
    {
        MessageBox.Show("Please define the input folder before starting");
        checkBox1.Checked = false;                
    }
    else if (!textBox2.Text.Contains("\\"))
    {
        MessageBox.Show("Please define the XML Output folder before starting");
        checkBox1.Checked = false;
    }
    else if (!textBox3.Text.Contains("\\"))
    {
        MessageBox.Show("Please define the Converted PPF Output Folder before starting");
        checkBox1.Checked = false;
    }
    else if (!textBox4.Text.Contains("\\"))
    {
        MessageBox.Show("Please define the Invalid PPF Output Folder before starting");
        checkBox1.Checked = false;
    }
    else
    {
        // calls the watcher
        prg.ProgramProcessing(textBox1.Text);
    }
}