有没有更好的方法来构建这个 if 语句

本文关键字：if 语句 构建 更好 方法 有没有 | 更新日期: 2023-09-27 18:32:32

这

对我来说似乎一团糟，我的头脑告诉我必须有更好的方法。

我在网页上有 6 个控件。

if (printer_make_1.Text != "" && printer_model_1.Text != "" && printer_make_2.Text != "" && printer_model_2.Text != "" && printer_make_3.Text != "" && printer_model_3.Text != "")
{
  // Do something
}

最好/最有效的方法是什么？

有没有更好的方法来构建这个 if 语句

如果要

提高可读性或在其他地方使用相同的逻辑，可以重构为方法：

public Boolean AllControlsHaveAValue() {
    return (printer_make_1.Text != ""
        && printer_model_1.Text != ""
        && printer_make_2.Text != ""
        && printer_model_2.Text != ""
        && printer_make_3.Text != ""
        && printer_model_3.Text != "");
}

然后就问：

if (AllControlsHaveAValue()) {
    // do something
}

重组从您的数据开始：避免printer_make_1、printer_make_2、...

class PrinterData
{
   public string Make { get; set; }
   public string Model { get; set; }
}
PrinterData[] printers = new PrinterData[3];  //or use a List<>
printers[0] = new PrinterData { Make = "PH", Model = "1A" };
...
if (printers.All(p => ! (p.Make == "" || p.Model == "")) )
  ...

if(new[] { printer_make_1, printer_model_1 ...}.All(l => l.Text != string.Empty)
{
    //do something
}

您可能希望将其拆分为更具可读性：

var labels = new[] { printer_make_1, printer_model_1 ... };
if(labels.All(l => l.Text != string.Empty))
{
    //do something
}

我通常将该测试放入一个方法中并调用它以使 if 更易于阅读

private boolean AreAllPrinterFieldsFilled()
{
    return (printer_make_1.Text != "" 
        && printer_model_1.Text != "" 
        && printer_make_2.Text != "" 
        && printer_model_2.Text != "" 
        && printer_make_3.Text != "" 
        && printer_model_3.Text != "");
}

然后在 if 中：

if (AreAllPrinterFieldsFilled)
{
    // Do something
}

有很多方法可以实现这一点 - 没有一种是优雅的。做你（以及那些可能跟在你身后的人）最容易读的事情。

我可能会采用这种方法：

string makeText = String.Concat(printer_make_1.Text, printer_make_2.Text, printer_make_3.Text);
string modelText = String.Concat(printer_model_1.Text, printer_model_2.Text, printer_model_3.Text);
if (makeText.Length != 0 && modelText.Length != 0)
{
    // Do something
}

if (!Enumerable.Range(1, 3)
    .Any(i => ((TextBox)FindControl("printer_make_" + i)).Text == "" || 
        ((TextBox)FindControl("printer_model_" + i)).Text == "") {...}

它允许您稍后扩展打印机品牌和型号的数量，但不是强类型。

private bool CheckAllEmpty(params TextBox[] textBoxes)
{
    return textBoxes.All(tb => tb.Text != null);
}
private void Foo()
{
    ...
    if (CheckAllEmpty(tb1, tb2, tb3))
    {
        mbox("tb1, tb2 and tb3 are all empty");
    }
    else
    {
        mbox("tb1, tb2 or tb3 isn't empty");
    }
    ...
}
相关文章：