Using LINQ with MVC, how can I include multiple parameters into a where clause?

Question

I am trying to run the below code, and it works properly until I try and add a second search value after using a comma to add a new value that gets added to the array of values. In SQL terms, I basically want to add the "AND" keyword after each loop iteration. Is this somehow possible?

var query = from s in db.Items 
                    join c in db.Categories on s.Category equals c.ID into cats from c in cats.DefaultIfEmpty()
                    join ma in db.Makes on s.Make equals ma.ID into maks from ma in maks.DefaultIfEmpty()
                    join mo in db.Models on s.Model equals mo.ID into mods from mo in mods.DefaultIfEmpty()
                    join st in db.Status on s.Usage_Status equals st.ID into stats from st in stats.DefaultIfEmpty()
                    join d in db.Departments on s.Department equals d.ID into deps from d in deps.DefaultIfEmpty()
                    select new { item = s, cat = c, make = ma, model = mo, status = st, department = d };

if (!String.IsNullOrEmpty(searchString))
{
    string[] searchValues = searchString.Split(',');

    for (var x = 0; x < searchValues.Length; x++)
    {
        var value = searchValues[x];
        query = query.Where(s => s.item.Asset_Tag_Nbr.Contains(value)
                           || s.item.Serial.Contains(value)
                           || s.cat.Name.Contains(value)
                           || s.make.Name.Contains(value)
                           || s.model.Name.Contains(value)
                           || s.item.Assigned_User.Contains(value));
    }
}

So assuming the searchString is This, That, you want to get the items where one of the properties contains both This and That? are you sure you want and and not or? BTW, searchValues[x] is already a string, the ToString is redundant. — Zohar Peled
– Zohar Peled, Commented Dec 11, 2018 at 16:32
if query is an IQueryable then you're already doing what you're asking. But then you say that's not working properly when you add another search value, so i'm guessing the problem is what @ZoharPeled said — Th0rndike
– Th0rndike, Commented Dec 11, 2018 at 16:39
I would want it to be an 'and' statement because I want the user to be able to narrow down the results. For example they would type "Desktop, Dell" and find only the results with those matching values. — sjohn285
– sjohn285, Commented Dec 11, 2018 at 16:40
This should work, check what is query. If its IQueryable around correct type it shoudl work, might be that @ZoharPeled pointed out the mistake in your logic — CodeSmith
– CodeSmith, Commented Dec 11, 2018 at 16:41
I think the issue is more of a linq syntax error. I'm not too experienced with linq, but maybe I can't run "query.Where" more than once? — sjohn285
– sjohn285, Commented Dec 11, 2018 at 16:48

sjohn285 · Accepted Answer · 2018-12-11 17:22:31Z

3

It turns out that the code was working as it should, and was just developer error. I was adding a space after the comma when entering search terms. Changing var value = searchValues[x] to var value = searchValues[x].Trim() did the trick and cleared up the empty spaces on the ends.

answered Dec 11, 2018 at 17:22

sjohn285

4151 gold badge6 silver badges23 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Using LINQ with MVC, how can I include multiple parameters into a where clause?

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related