CODEWITHSUNDEEP
.netasp.net-mvcaction
user135498
user135498

Reputation: 6133

.NET MVC Call method on different controller

Can anybody tell me how to call a method on a different controller from within an action method? I don't want to redirect. I want to call a method on a different controller that returns a string and use the response within my action method.

Upvotes: 47

Views: 88546

Answers (7)

user1624372
user1624372

Reputation:

You can use the following approach to invoke a method on the other controller:

var otherController = DependencyResolver.Current.GetService<OtherController>();
var result = otherController.SomeMethod();

This worked for me in ASP.NET MVC5. Hope that it will work for you as well.

Upvotes: 23

daniel
daniel

Reputation: 101

I was looking for the same thing, but seriously people, why the need to write such complicated answers.

Here's a post that will answer it very simply: Using Html.ActionLink to call action on different controller

Basically you just have to use this overload of the actionlink: ActionLink(HtmlHelper, String, String, String, Object, Object)

So you will have: ActionLink("linkText", "actionName", "controllerName", routeValues, htmlAttributes)

If you don't have any routeValues (which are the Inputs for the Action) or htmlAttributes, you have to set them as null.

Here's an example call: @Html.ActionLink("Add New Student", "Create", "Student", null, new { @class = "btn btn-primary" })

Upvotes: -6

Drew Noakes
Drew Noakes

Reputation: 310907

You can achieve this via the Action method of HtmlHelper.

In a view, you would do it like this:

@Html.Action("OtherAction")

However it's not straightforward to obtain an instance of HtmlHelper in an action method (by design). In fact it's such a horrible hack that I am reluctant to even post it...

var htmlHelper = new HtmlHelper(new ViewContext(
                                      ControllerContext, 
                                      new WebFormView(ControllerContext, "HACK"),
                                      new ViewDataDictionary(),
                                      new TempDataDictionary(),
                                      new StringWriter()),
                                new ViewPage());

var otherViewHtml = htmlHelper.Action("ActionName", "ControllerName");

This works on MVC 3. You might need to remove the StringWriter arg from the ViewContext constructor for MVC 2, IIRC.

</hack>

Upvotes: 13

awrigley
awrigley

Reputation: 13581

I haven't used Castle Windsor IoC, but the theory is that you should be able to create a custom Controller factory class, and then instruct the MVC framework to use this custom controller factory, by registering it in the Global.asax.css file, in the Application_Start event:

protected void Application_Start()
{
   RegisterRoutes(RouteTable.Routes);
   ControllerBuilder.Current.SetControllerFactory(new MyCustomControllerFactor());
}

[See Pro Asp.Net MVC 2 Framework, Steven Sanderson, Apress, pages 64 - 66]

That way, you can instantiate your controllers from anywhere in your code.

The notion of NOT calling the actions of another controller from the "current" controller, or from other code is quite wrong. Controllers are just classes. They only become "Controllers" when invoked in a special way by the MVC Framework.

Therefore, the right and wrong of this boils down to WHY are you doing this, not WHETHER you should or not.

If you are just using a controller as a class, then this is fine. If you are trying to use this to send a response to the user, then you should use a RedirectToAction as suggested above.

There are a number of reasons to use a controller as a class rather than as a Controller. For example, when testing your controller. Therefore, treating your controllers from as a class is necessary as opposed to wrong.

A non testing scenario example of using a Controller as a Class:

I am writing a mini framework that leverages the MVC framework's templating capabilities to produce the HTML for HTML emails, something that all web apps need to do, for one reason or another (eg, order confirmation emails).

Very roughly, you instantiate your MailManagerController (for simplicity, presume you are not using IoC) in your NormalController's action (that needs to send an email) and then do:

MailManagerController mailmanager = new MailManagerController();
string html = mailmanager.OrderConfirmation(order).RenderToString();
Postman.SendEmail(html, order.UserEmailAddress, "MyApp order confirmation");

Where RenderToString is an extension method on ViewResultBase that renders the output of an Action (that returns a ViewResultBase object) to a string, and Postman is a static class that deals with sending emails once you have the text.

The beauty of this technique is that you can use the MVC framework to produce templated emails, because the OrderConfirmation Action will have an associated view which is nothing if not an html template for your the email you are going to send.

Upvotes: 9

Pete
Pete

Reputation: 12553

Sounds in my ears like you should refactor your application, and extract the functionality that generates the string out to a new seperate class (or reuse an existing class, if you have one that fits) and let both controllers use that class.

Upvotes: 42

twk
twk

Reputation: 3120

Looks like you're trying to do something the controllers aren't designed for. Design your required method as an public method in some class and invoke from both controller actions.

Upvotes: 2

Nick
Nick

Reputation: 5908

Could you just instantiate the controller in your action method and call the other method you need?

public ActionResult YourActionMethod()
{
   SomeController c = new SomeController();
   ActionResult result = c.SomeMethod();

   return View();
}

Upvotes: 10

Related Questions