Trying to filter Database Data based on ID

Question

Hello I am Trying to Filter Database Data base on ID but i cant get it to work;

Code:

public async Task<ActionResult> Index(string searchString, string description)
{
var AllProducts = from m in _context.Products
select m;
List<int> IDList = new List<int>();
IDList.Clear();
int count = 0;
filter base on Title
if (!String.IsNullOrEmpty(searchString))
{
count++;
var FilterSearch = from Products in AllProducts
where Products.Title.Contains(searchString)
select Products.Id;
foreach (int Product in FilterSearch)
IDList.Add(Product);
}
filter based on description
if (!String.IsNullOrEmpty(description))
{
count++;
var FilterDesc = from Products in AllProducts
where Products.ProductDescription.Contains(description)
select Products.Id;
foreach (int Products in FilterDesc)
IDList.Add(Products);
}
making sure that only data that have ID simulair appear
if (IDList.Count != 0)
{
var filter2 = IDList.GroupBy(x => x)
.Where(g => g.Count() == count)
.Select(y => y.Key)
.ToList();
//under this line is what I think the problem
AllProducts = AllProducts.Where(x => x.Id.Equals(filter2));
}
return View(await AllProducts.ToListAsync());
}

everything works except for this code on line 47:

AllProducts = AllProducts.Where(x => x.Id.Equals(filter2));

when I type something in both search fields (description and searchStrings)

It return back nothing

Only when both search fields are empty they show all the data
I just need the Data Database that have simulair ID's as those of the numbers in filter2 list

How do i manage that?

Sachin Singh · Answer

you are comparing an int with List which is not possible , reverse the logic and it will work AllProducts = AllProducts.Where(x => filter2.Contains(x.Id)); BTW you are overrdoing things, your query should be as simple as public async Task Index(string searchString, string description) { var AllProducts = _context.Products.Where(x=>x.Title==searchString|| x.ProductDescription==description).ToListAsync(); return View(await AllProducts); }

osyris zosar · Answer

Thank you so much It finally worked. I would like to use much less code and i have tried your code the problem is that if i dont use any search field i get no results and when i type in both search fields it only shows me the results based on 1 filter I know this is the standart code: public async Task Index(string searchString, string description) { var store = from m in _context.Products select m; if (!String.IsNullOrEmpty(searchString)) { store = filtered.Where(s => s.Title.Contains(searchString)); } if (!String.IsNullOrEmpty(description)) { store = store.Where(s => s.ProductDescription.Contains(description)); } return View(await store.ToListAsync()); } but it can only use 1 search field at a time I could add: if (!String.IsNullOrEmpty(description) && !String.IsNullOrEmpty(searchString) ) { //some filter code } but its not practicale when you have 10 search fields. you will have to write an "If statements" for every single case. I dont know how else write less code. If someone knows please let me know