Collectionview filtering on checkboxes and textbox text

Question

0.00/5 (No votes)

See more:

I'm looking at Filtering a CollectionView based on firstly, some Text inside a TextBox and secondly whether certain CheckBoxes are checked or not. What I really want to avoid is having a super long filter method that says something like,

If CheckBox1 is checked
{
    If CompanyContains SearchBox.Text && CompanyMeetsCheckBox1Spec
}
If CheckBox2 is checked
{
    If CompanyContains SearchBox.Text && CompanyMeetsCheckBox2Spec
}

and so on and so forth for however many number of CheckBoxes I may end up with. I may even have multiple CheckBoxes checked so could end up with;

If CheckBox1 is checked && CheckBox2 is checked
{
    If CompanyContains SearchBox.Text && CompanyMeetsCheckBox1Spec &&                 CompanyMeetsCheckBox2Spec 
}

It really could get quite a long. Could I ask for some suggestion on how to "neatly" avoid an incredibly long filtering method?.. I hope I explained myself correctly.

What I have tried:

This is the method I am using so far to filter based on SearchBox Text;

private void SearchBoxCallApplyFilters(object sender, TextChangedEventArgs e)
{
    CollectionViewSource.GetDefaultView(Companies).Refresh();
}

private bool FilterCompanies(object obj)
{
    Var company = obj as CompanyModel;
    if (searchBox.Text == string.Empty)
    {
        return true;
    }
    else if (company.Name.ToLower().Contains(CharactersOnly(searchBox.Text.ToLower())))
    {
        return true;
    }
    return false;
}

Posted 7-Sep-16 5:18am

DRD94

Updated 7-Sep-16 6:53am

Add a Solution

Comments

Ralf Meier 7-Sep-16 14:07pm

For me your requirement is complete unclear - sorry ...
Please explain with an example what you try to achieve.

Of course it is possible to get results with a loop. For this it is necessary that corresponding controls should have corresponding names. Do you have it like this ?

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Henrik Jonsson · Accepted Answer · 2016-09-07T06:53:00

Provided that all checkbox conditions must be fullfilled ("ANDed" together) you at least can reduce the problem to one if statement per checkbox and avoid repetition of the text check in the following way:

C#

private bool FilterCompanies(object obj)
{
    Var company = obj as CompanyModel;
    if (searchBox.Text == string.Empty)
    {
        return true;
    }
    else if (company.Name.ToLower().Contains(CharactersOnly(searchBox.Text.ToLower())))
    {
        if( chkBox1.IsChecked && MeetsCheckBox1Spec(Company) == false )
        {
           return false;
        }
        if( chkBox2.IsChecked && MeetsCheckBox2Spec(Company) == false )
        {
           return false;
        }
        ...

        return true;
    }
    return false;
}

If you have more checkboxes that can be handled in this way or have more complex logic between you may have to think about whether your solution is user-friendly at all.