我很好奇你是否可以重载控制器方法在ASP。净MVC。每当我尝试时,都会得到下面的错误。这两个方法接受不同的参数。这是做不到的事情吗?

当前对控制器类型“MyController”上的动作“MyMethod”的请求在以下动作方法之间是不明确的:


当前回答

每个控制器方法只允许有一个公共签名。如果您试图重载它,它将被编译,但您将得到您所经历的运行时错误。

如果您不愿意使用不同的动词(如[HttpGet]和[HttpPost]属性)来区分重载方法(这将工作),或者更改路由,那么剩下的就是您可以提供另一个具有不同名称的方法,或者您可以在现有方法内部分派。我是这样做的:

我曾经遇到过必须保持向后兼容性的情况。原来的方法需要两个参数,但新的方法只有一个参数。重载我所期望的方式没有工作,因为MVC不再找到入口点。

为了解决这个问题,我做了以下事情:

Changed the 2 overloaded action methods from public to private Created one new public method which contained "just" 2 string parameters. That one acted as a dispatcher, i.e.: public ActionResult DoSomething(string param1, string param2) { if (string.IsNullOrEmpty(param2)) { return DoSomething(ProductName: param1); } else { int oldId = int.Parse(param1); return DoSomething(OldParam: param1, OldId: oldId); } } private ActionResult DoSomething(string OldParam, int OldId) { // some code here return Json(result); } private ActionResult DoSomething(string ProductName) { // some code here return Json(result); }

当然,这是一个hack,应该稍后进行重构。但就目前而言,它对我有用。

你也可以创建一个dispatcher:

public ActionResult DoSomething(string action, string param1, string param2)
{
    switch (action)
    {
        case "update":
            return UpdateAction(param1, param2);
        case "remove":
            return DeleteAction(param1);
    }
}

可以看到,UpdateAction需要2个参数,而DeleteAction只需要一个参数。

其他回答

是的。我已经能够通过为每个控制器方法设置HttpGet/HttpPost(或等效的AcceptVerbs属性)来做到这一点,即HttpGet或HttpPost,但不是两者都有。这样它就可以根据请求的类型来判断使用哪个方法。

[HttpGet]
public ActionResult Show()
{
   ...
}

[HttpPost]
public ActionResult Show( string userName )
{
   ...
}

我的一个建议是,对于这种情况,应该有一个私有实现,你的两个公共Action方法都依赖于它,以避免重复代码。

每个控制器方法只允许有一个公共签名。如果您试图重载它,它将被编译,但您将得到您所经历的运行时错误。

如果您不愿意使用不同的动词(如[HttpGet]和[HttpPost]属性)来区分重载方法(这将工作),或者更改路由,那么剩下的就是您可以提供另一个具有不同名称的方法,或者您可以在现有方法内部分派。我是这样做的:

我曾经遇到过必须保持向后兼容性的情况。原来的方法需要两个参数,但新的方法只有一个参数。重载我所期望的方式没有工作,因为MVC不再找到入口点。

为了解决这个问题,我做了以下事情:

Changed the 2 overloaded action methods from public to private Created one new public method which contained "just" 2 string parameters. That one acted as a dispatcher, i.e.: public ActionResult DoSomething(string param1, string param2) { if (string.IsNullOrEmpty(param2)) { return DoSomething(ProductName: param1); } else { int oldId = int.Parse(param1); return DoSomething(OldParam: param1, OldId: oldId); } } private ActionResult DoSomething(string OldParam, int OldId) { // some code here return Json(result); } private ActionResult DoSomething(string ProductName) { // some code here return Json(result); }

当然,这是一个hack,应该稍后进行重构。但就目前而言,它对我有用。

你也可以创建一个dispatcher:

public ActionResult DoSomething(string action, string param1, string param2)
{
    switch (action)
    {
        case "update":
            return UpdateAction(param1, param2);
        case "remove":
            return DeleteAction(param1);
    }
}

可以看到,UpdateAction需要2个参数,而DeleteAction只需要一个参数。

据我所知,你只能有相同的方法时,使用不同的http方法。

i.e.

[AcceptVerbs("GET")]
public ActionResult MyAction()
{

}

[AcceptVerbs("POST")]
public ActionResult MyAction(FormResult fm)
{

}

为了克服这个问题,你可以编写一个ActionMethodSelectorAttribute来检查每个动作的MethodInfo,并将其与发布的表单值进行比较,然后拒绝任何与表单值不匹配的方法(当然,不包括按钮名称)。

举个例子:- http://blog.abodit.com/2010/02/asp-net-mvc-ambiguous-match/

但是,这不是一个好主意。

我喜欢这个贴在另一个帖子里的答案

这主要用于从另一个控制器继承,并希望重写来自基本控制器的操作

ASP。用不同的参数覆盖一个动作