C# Winforms - IF 语句的问题

Question

这就是我想要做的。我有一个带有多个复选框的表单。用户将选中其中的 1 个或 2 个或 12 个，然后单击一个按钮，然后会发生几件事，具体取决于选中的复选框。

    private void checkBox1_CheckedChanged(object sender, EventArgs e)
    {
        if (checkBox1.Checked == true)
        {
            checkBox1.Tag = "IP";
            Console.WriteLine(checkBox1.Tag);
        }
        else
        {
            checkBox1.Tag = null;
        }
    }
    private void CheckBox2_CheckedChanged(object sender, EventArgs e)
    {
        if (checkBox2.Checked == true)
        {
            checkBox2.Tag = "PR";
            Console.WriteLine(checkBox2.Tag);
        }
        else
        {
            checkBox2.Tag = null;

        }
    }

所以这只是我在此表单上的 24 个复选框中的 2 个的代码示例。一旦用户单击按钮，表单将使用 checkboxX.Tag 变量保存文件：

        private void button2_Click(object sender, EventArgs e)
    {

        if (string.IsNullOrWhiteSpace(myFile) &
            string.IsNullOrWhiteSpace(selectDate) &
            string.IsNullOrWhiteSpace(selectDept))
        {
            MessageBox.Show("Well, that didn't work. Check your info and try again!");
            return;
        }
        else File.Exists(myFile);
        {
            MessageBox.Show("Document(s) added! " + selectDept + selectWorker +
              " for " + dateTime.Year + "-" + dateTime.Month + "-" + dateTime.Day);
            MessageBox.Show("To add more docs, re-open this program");
            if (checkBox1.Checked)
            {
                string finalName = @"C:\testing\" + selectDept + selectWorker + @"\" + 
                  checkBox1.Tag + dateTime.Year + dateTime.Month + dateTime.Day + ".pdf";
                textBox2.Text = finalName;
                File.Copy(myFile, finalName, true);
            }
            if (checkBox2.Checked)
            {
                string finalName = @"C:\testing\" + selectDept + selectWorker + @"\" + 
                  checkBox2.Tag + dateTime.Year + dateTime.Month + dateTime.Day + ".pdf";
                textBox2.Text = finalName;
                File.Copy(myFile, finalName, true);
            }

因此，就目前而言，它适用于第一个且仅适用于第一个复选框。就像，文件将按照预期的方式复制和保存，但不是有 5 个相同的文件名为“IP20181012”、“PR20181012”、“FS20181012”等，我最终得到一个名为“IP20181012”的文件，然后另一个文件刚刚命名为“20181012”，这让我相信 checkBox2.Tag 变量从未设置过。我试过使用一个单独的变量（除了内置的 checkBox1.Tag 之一），并且发生了完全相同的问题。似乎只有 checkBox1.Tag 会正确设置，而其他 checkBoxX.Tag 变量都保持为空。即使我声明了单独的变量，然后尝试在复选框中设置它们，无论我做什么，都只会设置 checkBox1 的变量。

另外，这是使用 if 语句的好方法吗？我不需要它们嵌套。我可以把它们排成一行，这样点击按钮就会遍历它下面的所有 if 语句吗？

Answer 1

按钮的单击处理程序中的一堆 if 语句是一个很好的方法。而且您甚至不再需要使用 Tag 属性或处理复选框选中的事件，因为无论如何您都在检查选中的属性。

    if (checkBox1.Checked)
    {
        string finalName = @"C:\testing\" + selectDept + selectWorker + @"\IP"
            + dateTime.Year + dateTime.Month + dateTime.Day + ".pdf";
        textBox2.Text = finalName;
        File.Copy(myFile, finalName, true);
    }
    if (checkBox2.Checked)
    {
        string finalName = @"C:\testing\" + selectDept + selectWorker + @"\PR"
          + dateTime.Year + dateTime.Month + dateTime.Day + ".pdf";
        textBox2.Text = finalName;
        File.Copy(myFile, finalName, true);
    }

Answer 2

编写更简洁的代码会让事情变得更容易。一些建议：

1. 不要直接使用您的复选框，而是创建一个类（“CheckBoxConfigs.cs”，它将具有关于复选框功能的布尔/字符串标志（您的代码将更易于阅读/维护）；

2. 使所有复选框使用相同的处理程序（相同的“事件”函数），您可以在其中设置“CheckBoxConfigs.cs”标志（而不是使用“标记”属性，您将拥有一个具有角色的类处理标志）而不是为 24 个 CheckBox_CheckedChanged 函数创建代码；

Answer 3

您是如何为这些复选框创建事件处理程序的？

您是否双击了每个复选框？

或者您是否双击一个，创建事件处理程序，然后为其他每个复制并更改该事件处理程序？

您看不到的是，在代码隐藏文件 (MyForm.Designer.cs) 中，每个事件处理程序都有一行如下所示：

this.checkBox1.CheckedChanged += new System.EventHandler(this.checkBox1_CheckedChanged);

那是实际连接事件的线路。如果没有该行，您的复选框实际上不会响应任何内容。

如果您从 VB6 或 VBA 开始，那会有点令人困惑，因为在这些语言中，只需创建一个具有正确名称的方法就会神奇地导致这种连接发生。但在 C#（和 VB .NET）中它是显式的，即使它可能隐藏在 Designer 文件中。

但您也可以自己添加这些行。这是我一直这样做的方式，所以设计师没有隐藏任何东西。在表单的构造函数中，您可以这样键入它们：

public MyForm()
{
    checkBox1.Checked += checkBox1_Checked;
    checkBox2.Checked += checkBox2_Checked;
    [etc.]
}

Answer 4

首先将设计器中每个复选框的 Tag 属性设置为您想要的字符串。然后，摆脱 CheckChanged 处理程序。最后，在你的按钮处理程序中做这样的事情（我做了明显的修改（比如我不叫File.Copy））：

 private void button2_Click(object sender, EventArgs e)
 {
     var selectDept = "MyDept";
     var selectWork = "MyWork";
     var myFile = @"c:\somefolder\somefile.ext";
     var dateTime = DateTime.Now;

     var formattedDate1 = dateTime.ToString("yyyy-MM-dd");
     var formattedDate2 = dateTime.ToString("yyyyMMdd");
     MessageBox.Show($"Document(s) added {selectDept}{selectWork} for {formattedDate1}");
     var checkboxes = new[]
     {
         checkBox1,
         checkBox2,
         checkBox3,
         //you get the idea
     };
     foreach (var checkbox in checkboxes)
     {
         if (checkbox.Checked)
         {
             var finalName = $@"C:\testing\{selectDept}{selectWork}\{checkbox.Tag}{formattedDate2}.pdf";
             textBox1.Text = finalName;      //no sure what this is for
             Debug.WriteLine($"MyFile: [{myFile}], Final Name: [{finalName}]");
         }
     }
 }

如果我选中第一个和第三个复选框（其Tag 属性是“CK1”和“CK2”，我的输出中会出现以下内容：

MyFile: [c:\somefolder\somefile.ext], Final Name: [C:\testing\MyDeptMyWork\CK120181105.pdf]
MyFile: [c:\somefolder\somefile.ext], Final Name: [C:\testing\MyDeptMyWork\CK320181105.pdf]

我认为这非常接近您想要的