Routing based on query string parameter name

Question

I'm trying to configure routing in my MVC4 WebAPI project.

I want to be able to search for products based on their name or their type like so:

/api/products?name=WidgetX - returns all products named WidgetX /api/products?type=gadget - returns all products of type gadget

The routes are configured like this:

config.Routes.MapHttpRoute(
    name: "Get by name",
    routeTemplate: "api/products/{name}",
    defaults: new { controller = "ProductSearchApi", action = "GetProductsByName", name = string.Empty }
);

config.Routes.MapHttpRoute(
    name: "Get by type",
    routeTemplate: "api/products/{type}",
    defaults: new { controller = "ProductSearchApi", action = "GetProductsByType", type = string.Empty }
);

The problem is that the name of the query string parameter seems to be ignored so the first route is always the one used, regardless the name of the query string parameter. How can I modify my route to get it right?

cuongle · Accepted Answer · 2012-09-27 11:45:22Z

33

What you need is just only one route below because query string is not used as routing parameters:

config.Routes.MapHttpRoute(
    name: "Get Products",
    routeTemplate: "api/products",
    defaults: new { controller = "ProductSearchApi" }
);

And, then define two methods like below:

GetProductsByName(string name)
{}

GetProductsByType(string type)
{}

Routing mechanism is smart enough to route your url to your correct action based on the name of query string whether the same with input parameters. Of course on all methods with prefix are Get

You might need to read this: http://www.asp.net/web-api/overview/web-api-routing-and-actions/routing-and-action-selection

edited Sep 27, 2012 at 11:45

answered Sep 27, 2012 at 11:23

cuongle

75.5k30 gold badges155 silver badges212 bronze badges

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

Comments

Nick · Accepted Answer · 2012-09-27 11:13:28Z

5

You don't need to include your query parameters in the route. There should only be one simple route map to cover the Http Methods on all of your ApiControllers:

routes.MapHttpRoute(
    name: "DefaultApi",
    routeTemplate: "api/{controller}/{id}",
    defaults: new { id = RouteParameter.Optional }
);

The only time you need to adjust the routes is if you want to move a parameter into the actual path, which you don't seem to be doing. Then your GET http method to search by two fields would be:

public IEnumerable<Product> Get(string name, string type){
    //..your code will have to deal with nulls of each parameter
}

If you want to explicitly search by one field at a time then you should think about using different controllers for different purposes. Ie, a SearchProductByTypeController that has a single Get(string type) method. The route would then be /api/SearchProductByTypeController?type=gadget

edited Sep 27, 2012 at 11:13

answered Sep 27, 2012 at 11:04

Nick

6,6485 gold badges48 silver badges64 bronze badges

2 Comments

rickythefox Over a year ago

Thanks, sometimes the issue is not in the code but in the architecture. ;)

mo. Over a year ago

What kind of resource is a SearchProductByTypeController? :P

hagensoft · Accepted Answer · 2012-09-27 10:56:32Z

0

try changing string.Empty for RouteParameter.Optional

answered Sep 27, 2012 at 10:56

hagensoft

1,50713 silver badges14 bronze badges

1 Comment

rickythefox Over a year ago

RouteParameter would make me use urls like /products/abc which would of course not meet the requirement. I need to be able to use query string parameters.

laszlokiss88 · Accepted Answer · 2012-09-27 11:06:11Z

0

Are you sure that the controllers are ok? I mean, the name of the params.

    public string GetProductsByName(string name)
    {
        return "Requested name: " + name;
    }

    public string GetProductsByType(string type)
    {
        return "Requested type: " + type;
    }

answered Sep 27, 2012 at 11:06

laszlokiss88

4,0793 gold badges22 silver badges26 bronze badges

Collectives™ on Stack Overflow

Routing based on query string parameter name

4 Answers 4

Comments

2 Comments

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

2 Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related