组合复选框而不使用 if

本文关键字：if 复选框 组合 | 更新日期: 2023-09-27 18:34:26

我正在制作一个密码生成器，我有三个复选框：小写、大写和数字。现在我有一个IF语句，上面写着：

    IF (check1.checked == true & check2.checked == true & check3.checked == true)
       {
           length = Convert.ToInt32(lengthTextBox.Text);
                string password = "";
                int choice;
                Generate gen = new Generate();
                for (int i = 1; i <= length; i++)
                {
                    choice = rnd.Next(1, 4);
                    if (choice == 1)
                    {
                        password = password + gen.lower();
                    }
                    else if (choice == 2)
                    {
                        password = password + gen.upper();
                    }
                    else if (choice == 3)
                    {
                        password = password + gen.number();
                    }
                }
                passwordTextBox.Text = password;
       }
    Else IF (check1.checked == true & check2.checked == true)
    {
            length = Convert.ToInt32(lengthTextBox.Text);
                string password = "";
                int choice;
                Generate gen = new Generate();
                for (int i = 1; i <= length; i++)
                {
                    choice = rnd.Next(1, 3);
                    if (choice == 1)
                    {
                        password = password + gen.lower();
                    }
                    else if (choice == 2)
                    {
                        password = password + gen.upper();
                    }
                }
                passwordTextBox.Text = password;
    }

等等。这种方法效率非常低。我在制作收银机程序时遇到了这个问题（10 种浇头选择，检查您想要的配料，然后程序将价格加在一起。如果你取消选中浇头，价格将被减去该金额），所以我就放弃了。但现在，我决心找到一种隐喻去购物的方法。

组合复选框而不使用 if

虽然不清楚

你的代码是做什么的，但你似乎能够连续调用这些方法。那么为什么不这样做呢？

if (check1.Checked)
   Generate.lowercase();
if (check2.Checked)
   Generate.uppercase();
if (check3.Checked)
   Generate.number();

此外，您可能还应该为控件和变量使用更具描述性的名称。

if(check1.checked) Generate.lowercase();
if(check2.checked) Generate.uppercase();
...
因此，

您在这里真正需要做的是找到一种方法来使选择动态化。 一种方法是创建一个函数列表，其中每个函数都能够生成一个值。 一开始，您根据检查的内容使用相关函数填充列表，然后您可以有一个循环，只需从该列表中选择一个函数：

List<Func<char>> generators = new List<Func<char>>();
if (lowercaseCheckbox.Checked)
    generators.Add(() => gen.lowercase());
if (uppercaseCheckbox.Checked)
    generators.Add(() => gen.uppercase());
if (numberCheckbox.Checked)
    generators.Add(() => gen.number());
int length = Convert.ToInt32(lengthTextBox.Text);
StringBuilder password = new StringBuilder();
Random random = new Random();
for (int i = 0; i < length; i++)
{
    int choice = random.Next(generators.Count);
    password.Append(generators[choice]());
}
string result = password.ToString();

请注意，与其在循环中重复向字符串追加字符，不如使用 StringBuilder 来避免不断重新分配和复制值。

我还建议为checkbox变量使用更有意义的名称;它使代码更具可读性。

这取决于您使用的框架和应用的模式，但有更好的方法可以做到这一点。您可以使用位标志，并将复选框和它们放在一起。

FlagVariable output = FlagVariable.None;
if( topping1.IsChecked )
{
    output &= FlagVariable.Topping1;
}
if( topping2.IsChecked )
{
    output &= FlagVariable.Topping2;
}
// etc...

然后后来

if( output | FlagVariable.Topping1 )
{
    // it has topping 1
}

这是非常有效且可维护的。您仍然需要在复选框及其相应的标志值之间编写关联代码（这是取决于您的框架/模式的部分，因为在带有 MVVM 的 WPF 中，您可能能够避免大量 if-then 代码）。

你可以

简单地将它们分开：

if (check1.checked) Generate.lowercase();
if (check2.checked) Generate.upperrcase();
if (check3.checked) Generate.number();

从逻辑上讲，这种方式和分组方式之间没有。

在密码生成的情况下@Gerald Versluis的答案，绝对值得一看。

在购物的情况下，你有味道。"检查"口味，然后添加"检查"口味的价格，正如您已经意识到的那样，它只是不能很好地扩展。
在这种情况下，最好将口味添加到列表中，然后将其计算为总数。

这也可以应用于密码生成。添加不同的"密码"生成器，从而使其更好地扩展，并使用 C# 强大的 OOP 功能。这里的流行词是：Polymorhism

相关文章：