C# 优化绑定的DataGridView搜索功能

大家好,

我的代码运行得很好，但看起来很糟糕。其目的是从我的数据库中搜索和显示符合搜索条件的工作人员。我的表格上有一个

DataGridView

，里面装满了从数据库中检索到的工作人员数据。按下

搜索

按钮后，我只想显示符合

文本框/组合框

标准的工作人员列表。如果它们为空，则会再次显示完整列表

---

表单上的搜索功能：

private void btnSearch_Click(object sender, EventArgs e)
{
    FillList();
    List<Worker> list = new List<Worker>();
    bool match = false;

    foreach (var worker in workerBindingSource)
    {
        if (txtName.Text.Length > 0 && !((Worker)worker).Name.ToLowerInvariant().Contains(txtName.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (txtLastName.Text.Length > 0 && !((Worker)worker).LastName.ToLowerInvariant().Contains(txtLastName.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (txtOIB.Text.Length > 0 && !((Worker)worker).OIB.ToLowerInvariant().Contains(txtOIB.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (txtLocation.Text.Length > 0 && !((Worker)worker).Location.ToLowerInvariant().Contains(txtLocation.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (txtAddress.Text.Length > 0 && !((Worker)worker).Address.ToLowerInvariant().Contains(txtAddress.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (txtPhoneNumber.Text.Length > 0 && !((Worker)worker).PhoneNumber.ToLowerInvariant().Contains(txtPhoneNumber.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (!cboProfessionalQualification.SelectedValue.ToString().Equals("Empty") && !((Worker)worker).ProfessionalQualification.ToString().ToLowerInvariant().Contains(cboProfessionalQualification.SelectedValue.ToString().ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (!cboDegree.SelectedValue.ToString().Equals("Empty") && !((Worker)worker).Degree.ToString().ToLowerInvariant().Equals(cboDegree.SelectedValue.ToString().ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (match)
        {
            list.Add(WorkerDAO.ReadWorker(((Worker)worker).ID));
        }
    }

    SortableBindingList<Worker> sortableList = new SortableBindingList<Worker>(list);
    workerBindingSource.DataSource = sortableList;
}

public static Worker ReadWorker(int workerID)
{
    var worker = ReadEverything().Where(x => x.ID == workerID).FirstOrDefault();

    return worker;
}

代码：

private void btnSearch_Click(object sender, EventArgs e)
{
    FillList();
    List<Worker> list = new List<Worker>();
    bool match = false;

    foreach (var worker in workerBindingSource)
    {
        if (txtName.Text.Length > 0 && !((Worker)worker).Name.ToLowerInvariant().Contains(txtName.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (txtLastName.Text.Length > 0 && !((Worker)worker).LastName.ToLowerInvariant().Contains(txtLastName.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (txtOIB.Text.Length > 0 && !((Worker)worker).OIB.ToLowerInvariant().Contains(txtOIB.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (txtLocation.Text.Length > 0 && !((Worker)worker).Location.ToLowerInvariant().Contains(txtLocation.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (txtAddress.Text.Length > 0 && !((Worker)worker).Address.ToLowerInvariant().Contains(txtAddress.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (txtPhoneNumber.Text.Length > 0 && !((Worker)worker).PhoneNumber.ToLowerInvariant().Contains(txtPhoneNumber.Text.ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (!cboProfessionalQualification.SelectedValue.ToString().Equals("Empty") && !((Worker)worker).ProfessionalQualification.ToString().ToLowerInvariant().Contains(cboProfessionalQualification.SelectedValue.ToString().ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (!cboDegree.SelectedValue.ToString().Equals("Empty") && !((Worker)worker).Degree.ToString().ToLowerInvariant().Equals(cboDegree.SelectedValue.ToString().ToLowerInvariant()))
        {
            match = false;
            continue;
        }
        else
            match = true;

        if (match)
        {
            list.Add(WorkerDAO.ReadWorker(((Worker)worker).ID));
        }
    }

    SortableBindingList<Worker> sortableList = new SortableBindingList<Worker>(list);
    workerBindingSource.DataSource = sortableList;
}

public static Worker ReadWorker(int workerID)
{
    var worker = ReadEverything().Where(x => x.ID == workerID).FirstOrDefault();

    return worker;
}

---

方法

ReadEverything

返回数据库中所有工作人员的列表

我正在寻找一个LINQ解决方案（或任何其他更好的解决方案）来大幅减少我的代码并使其更具可读性，但由于我对编程和LINQ仍然非常熟悉，所以我自己无法解决这个问题。所以我想知道你们是否能帮我一点忙，或者至少给我指出正确的方向

---

提前感谢

如果您的datagridview已经拥有数据库中的所有数据，我建议您不要再去数据库进行搜索，因为您已经拥有了所有数据。只需在内存中过滤并显示它

如果出于某种原因需要访问数据库，则没有理由从数据库获取所有数据并在客户端对其进行过滤，只需将“workerId”传递给存储过程，让SP添加适当的where条件并返回过滤后的数据

要使用绑定源实现内存内筛选，请执行以下操作：

bindingSource.Filter = "columnname = 'value'";

private void button1_Click(object sender, EventArgs e)
{
    string searchValue = textBox1.Text;

     dataGridView1.SelectionMode = DataGridViewSelectionMode.FullRowSelect;
     bindingSource.Filter = string.Format("{0} = '{1}'","YourColumnName", searchValue );
     //here you can do selection if you need
}

To remove filter use the following

bindingSource.RemoveFilter();
//or
bindingSource.Filter = null;

---

最初作为答案发布。

当

从表中选择*时，
读取所有内容
是否命中数据库？为什么不直接使用
select*from-table，其中id=workerid
？workerBindingSource的类型是什么？它实际上是一个
System.Windows.Forms.BindingSource
？如果它是绑定源，则可以使用
BindingSoure.Filter
属性来筛选list@SriramSakthivel，这是一个很好的方法。再加上一个例子作为答案，你就得到了我的选票@SriramSakthivel ReadEverything使用存储过程从数据库中读取。是的，我在发布我的问题之前已经检查了原始答案，但我正在尝试这样做。如果可能的话，我只是想用linq优化代码=（你必须明白一件事。Linq不会优化任何东西，它使代码可读性和简单性，就是这样。永远不要说“我爱Linq”但是，在这种情况下，我建议您使用专门为此目的而设计的
bindingSource.Filter
。您需要生成查询来过滤数据，就这样。好的，谢谢，我很重视经验丰富的程序员的解释。我对您的答案投了赞成票，并将尝试这样做。我将等待其他答案还有一段时间，如果没有任何结果，我会接受你的回答。干杯！