Question

I’m working on a basic Issue Management System in order to learn ASP.NET MVC. I’ve gotten it up and running to a fairly decent level but I’ve run into a problem.

I have a controller named Issue with a view called Open. /Issue/Open lists all of the open issues currently logged on the system. I’ve defined a route like so:

    routes.MapRoute(          'OpenSort',                                                         // Route name         'Issue/Open/{sort}',                                                // URL with parameters         new { controller = 'Issue', action = 'Open', sort = 'TimeLogged' }  // Parameter defaults     ); 

This is working fine so far, using the following code in IssueController.cs:

public ActionResult Open(string sort) {                 var Issues = from i in db.Issues where i.Status == 'Open' orderby i.TimeLogged ascending select i;      switch (sort)     {         case 'ID':             Issues = from i in db.Issues where i.Status == 'Open' orderby i.ID ascending select i;             break;          case 'TimeLogged':             goto default;          case 'Technician':             Issues = from i in db.Issues where i.Status == 'Open' orderby i.Technician ascending select i;             break;          case 'Customer':             Issues = from i in db.Issues where i.Status == 'Open' orderby i.Customer ascending select i;             break;          case 'Category':             Issues = from i in db.Issues where i.Status == 'Open' orderby i.Category ascending select i;             break;          case 'Priority':             Issues = from i in db.Issues where i.Status == 'Open' orderby i.Priority ascending select i;             break;          case 'Status':             Issues = from i in db.Issues where i.Status == 'Open' orderby i.Status ascending select i;             break;          default:             break;     }                  ViewData['Title'] = 'Open Issues';     ViewData['SortID'] = sort.ToString();      return View(Issues.ToList()); } 

This is working fine (although, I wonder if there is a better way to handle my definition of the query than a switch?) but now I want to be able to do two things on the Open Issues view:

1. Sort by any of the headings – OK
2. Filter on certain headings (Technician, Customer, Category, Priority, Status) – ??

I can’t figure out how to pass two parameters to the Controller so I can organise my queries. I’ve also just realised that unless I figure out how to generate my queries on the fly I am going to need (number of sort options) * (number of filter options) in my switch.

Argh, can anyone point me in the right direction? Cheers!

Answer 1

1. Remove sort from the route. Just use a route without a parameter.
2. Add query string parameters to the query for the sort, filter, etc. So your query will look like:

http://example.com/Issue/Open?sort=ID&filter=foo

public ActionResult Open(string sort, string filter) 

The MVC framework will fill in the arguments from the query string parameters. Make sure and use nullable types (like string) for any of these query string parameter arguments which might not be filled in.

I actually think this is a ‘more correct’ way to write the URL. The URL itself identifies the resource (open issues); the query string parameters customize how to display the resource.

As far as the number of queries go, remember that you do not have to build the entire query at once. You can use the .OrderBy extension method to re-order an existing IQueryable<T>, and similarly with .Where.

var Issues = from i in db.Issues where i.Status == 'Open' select i;  switch (sort) {     case 'ID':         Issues = Issues.OrderBy(i => i.ID);         break;      // [...]      default:         Issues = Issues.OrderBy(i => i.TimeLogged); }