Controller with Index action with optional parameter

Question

I'd like to have an Index action for a "Users" controller that takes an optional parameter.

/Users/{id}

Or

/Users

I'd like to avoid:

/Users/Index/{id}

Or

/Users/Index/

I added a Route to map /Users/{id} to the Index action.

routes.MapRoute( "Users",
                 "Users/{id}",
                 new { controller = "Users", action = "Index", id =
                 UrlParameter.Optional});

That works fine. I ran into a problem when I added another action, "Add", also with an optional parameter, to the Users controller. The route I added earlier misinterprets Add as a parameter to the Index action. The "Index" action gets triggered for /Users/Add.

How can I get the best of both?

Thanks.

McGarnagle · Accepted Answer · 2012-06-22 05:26:56Z

2

Set the constraints property to only allow numbers. That should fix your routing issue.

routes.MapRoute( "Users",
    "Users/{id}",
    new { controller = "Users", action = "Index", id = UrlParameter.Optional },
    new { id = "[0-9]+" }
);

Edit.

You can do it differently without using a regex constraint. Add the route Users/Add explicitly, adding it before the Index route:

routes.MapRoute( "Users",
    "Users/Add/{id}",
    new { controller = "Users", action = "Add", id = UrlParameter.Optional }
);

edited Jun 22, 2012 at 5:26

answered Jun 22, 2012 at 5:20

McGarnagle

103k31 gold badges235 silver badges264 bronze badges

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

2 Comments

McGarnagle Over a year ago

@JoelSkrepnek indeed I do not. Please see my edit. You can always "hard wire" the routes like this; the key is to put the more specific ones first.

Joel Skrepnek Over a year ago

Thanks. I'll see how this goes over with the boss :).

Dmitry Efimenko · Accepted Answer · 2012-06-22 05:20:21Z

0

You do no need to create routes for each action on your controller. A default route that is in global.asax at the beginning of creation of a project is enough for your needs:

routes.MapRoute(
    "Default", // Route name
    "{controller}/{action}/{id}", // URL with parameters
    new { controller = "Account", action = "Index", id = UrlParameter.Optional } // Parameter defaults
);

answered Jun 22, 2012 at 5:20

Dmitry Efimenko

11.2k7 gold badges65 silver badges85 bronze badges

1 Comment

Joel Skrepnek Over a year ago

I created the route to avoid /Users/Index/{id}.

Collectives™ on Stack Overflow

Controller with Index action with optional parameter

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related