Question

大家好，

我的代码运行得非常好，但看起来很讨厌。其目的是从我的数据库中搜索和显示符合搜索条件的工作人员。我的表单上有一个DataGridView，其中填写了数据库中的数据库。按Search按钮后，我只想显示符合TextBoxes/ComboBoxes条件的工作人员列表。如果它们是空的，它会再次显示完整列表。

表单上的搜索功能

Search feature

代码：

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;
}

带有“txt”前缀的代码中的所有内容都是TextBox，带有“cbo”前缀的所有内容都是ComboBox。 ComboBoxes包含用户可以选择的某些枚举。如果ComboBox显示枚举值“空”，则表示用户没有选择任何内容（与字符串的零长度值相同）。 SortableBindingList仅用于列标题单击的排序目的，在这种情况下没什么重要的。 ReadWorker方法只有在匹配所有搜索值时返回一个worker，然后将其添加到DataGridView之后显示的列表中。 FillList方法将整个数据库工作列表设置为DataBindingSource。

ReadWorker方法：

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

    return worker;
}

方法ReadEverything返回数据库中所有工作人员的列表。

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

提前致谢！

Answer 1

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

如果你出于某种原因需要点击数据库，没有理由从数据库获取所有数据并在客户端过滤它，只需将“workerId”传递给存储过程，让SP在适当的条件下添加并返回过滤后的数据。

使用Binding Source实现内存过滤：

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;

最初发布为答案here。

优化绑定的DataGridView搜索功能

1 个答案: