Return View with multiple parameter

Question

I am trying to create a filter system in C# mvc where i can use multiple input fields for filtering:

View:

<form asp-controller="Products" asp-action="Index">
<p>
Title: <input type="text" name="SearchString" />
Description: <input type="text" name="SearchString" />
<input type="submit" value="Filter" />
</p>
</form>

Model:

public class Product
{
public int Id { get; set; }
public string Title { get; set; }
public int Price { get; set; }
[Display(Name = "Product description")]
public string ProductDescription { get; set; }
public int Stock { get; set; }
}

Controller:

public async Task<IActionResult> Index(string searchString, string description)
{
var store = from m in _context.Products
select m;
if (!String.IsNullOrEmpty(searchString))
{
store = store.Where(s => s.Title.Contains(searchString));
}
if (!String.IsNullOrEmpty(description))
{
store = store.Where(s => s.ProductDescription.Contains(description));
}
return View(await store.ToListAsync());
}

I am not really sure how i can manage this the best possible way

Sachin Singh · Answer

please explain the conditions as well. Is it "Either Or" or :and" condition. Means, are both inputs are necessary fields or not? Also , you are doing In-Memory filtering , can you please explain why is it so, why are you not fltering directly on the server with Or condition. var store = _context.Products. Where(p=>p.Title.Contains(searchString) || p.Title.Contains(description)).ToList();

Kiran Mohanty · Answer

Hi , Try this. var store = _context.Products. Where(p=> (searchString.Length > 0 ? p.Title.Contains(searchString) : p) || (description.Length > 0 ? p.Title.Contains(description) :p) ).ToList();

osyris zosar · Answer

Hi Jignesh Kumar Thank you very much for your example lets say i have more Filtering fields, price, brand,name,delivery,Instock,popular,newProducts,outofstock. etc that would be a very long code because you will have to create a "else if statement" for every possible combination. I dont think that would be a good solution

Sachin Singh · Answer

Then simply use "Or" condition in one query and thats all.

osyris zosar · Answer

Hi Sachin its basicly a simple search on a E-commerce webpage to search for products. so you have the optiont to filter al products by name or price or brand etc.. so it could be that the user only uses one field or all of them

Jignesh Kumar · Answer

Hi, Please use this one, public async Task Index( string searchString, string description) { var store = from m in __employeeContext.Products select m; if (!String.IsNullOrEmpty(searchString) && !String.IsNullOrEmpty(description)) //Check condition for both paramter { store = store.Where(s => s.Title.Contains(searchString) && s.ProductDescription.Contains(description)); } else if (!String.IsNullOrEmpty(searchString)) { store = store.Where(s => s.Title.Contains(searchString)); } else if (!String.IsNullOrEmpty(description)) { store = store.Where(s => s.ProductDescription.Contains(description)); } return View(await store.ToListAsync()); }