Question

我有一个if语句，可以在表单上的所有文本框完成时运行一些代码，

我检查所有文本框目前不为空的方式是

if (txtUserId.Text != ""
                    && txtFirstName.Text != ""
                    && txtLastName.Text != ""
                    && txtCity.Text != ""
                    && txtSTate.Text != ""
                    && txtCountry.Text != "")
                {
                    // Some code
                }

有没有更好的方法来写这个？

Answer 1

摘要检查函数：

bool IsFilled(TextBox tb) { return tb.Text != ""; }

然后你可以使用旧的，简化的代码，或者这个技巧：

var textBoxes = new [] { txtUserId, txtFirstName, ... };
if (textBoxes.All(tb => IsFilled(tb)) { ... }

你得到的文字框越多，这可能就越具有可扩展性。

您还可以编写textBoxes.All(IsFilled)，因为方法组转换为委托而起作用。这略微性能更高，更短。但我发现方法组转换难以理解和误导。其他人可能会问你“这是做什么的？”这表明代码味道。我不推荐它。

Answer 2

TextBox[] boxes = new[] { txtUserId, txtFirstName, txtLastName, txtCity, txtSTate, txtCountry };

if (boxes.All(x => x.Text != ""))
{
    // Some code
}

Answer 3

您可以在表单中循环显示TextBox。

private bool AreThereAnyEmptyTextBoxes()
{
    foreach (var item in this.Controls)
    {
        if (item is TextBox)
        {
            var textBox = (TextBox)item;
            if (textBox.Text == string.Empty)
            {
                return true;
            }
        }
    }

    return false;
}

Answer 4

假设您使用的是Windows窗体控件，可以这样写：

public bool AllTextEntered(params Control[] controls)
{
    return controls.All(control => (control != null) && (control.Text != ""));
}

然后你可以这样打电话：

if (AllTextEntered(txtUserId, txtFirstName, txtLastName, txtCity, txtSTate, txtCountry))
{
    // ...
}

使用params的优点是允许您根据需要传递任意数量的控件，而无需手动编写代码将它们全部放入传递给AllTextEntered()的数组中。

另请注意，(control != null)测试可能并非真正需要。

Answer 5

您也可以在这些情况下使用反射。当我通过多个字段的用户详细信息过滤结果时，我遇到了类似的问题。如果用户未在搜索框中键入任何内容，则查询将返回数据库中的所有结果，这些结果并非预期结果。我问了这个问题并第一次见到反思。

例如：

你有一个UserDetail模型，如下所示

public class UserDetail{

  public string FirstName {get;set;}

  public string LastName {get;set;}

  public string Country {get;set;}
}

然后假设您有userDetail对象，并且您希望通过使用LINQ来检查以确保UserDetail对象中的所有属性都不为null或为空。

return userDetail.GetType().GetProperties()
    .Where(pi => pi.GetValue(userDetail) is string)
    .Select(pi => (string) pi.GetValue(userDetail))
    .All(value => !String.IsNullOrEmpty(value));

如果userDetail对象字符串属性都不为null或为空，则此方法将返回true。如果所有属性都包含某些内容，则输出为true。

你的逻辑是：

public bool AllUserDetailsContainSomething(UserDetail userDetail){

 return userDetail.GetType().GetProperties()
        .Where(pi => pi.GetValue(userDetail) is string)
        .Select(pi => (string) pi.GetValue(userDetail))
        .All(value => !String.IsNullOrEmpty(value));

}

然后你可以调用这个方法

if(AllUserDetailsContainSomething(userDetail)){
 //do something
}

减少if语句中的条件

5 个答案: