Trying to filter Database Data based on ID

osyris zosar

5y674.NET

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

Code:

public async Task 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?

2 Replies

Know the answer? Post it — somebody with the same question will find it here.

osyris zosar5y ago
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
Sachin Singh5y ago
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);

      }