在Controller操作方法中重用代码的最佳方法

时间:2013-08-10 17:34:39

标签: asp.net-mvc asp.net-mvc-3 asp.net-mvc-4

我在相同的控制器类中有一些看起来几乎相同的代码,例如设置viewbags来填充所有下拉列表,相同的代码适用于我的Post并获得Create和Edit操作方法。 所以我在控制器类的末尾创建了一个私有方法,如下所示: -

private void populateViewBags()
{
    string controllerName = RouteData.Values["controller"].ToString();
    ViewBag.PossibleDataCenters = repository.AllDataCenter().OrderBy(a => a.Name).ToList();
    ViewBag.PossibleZones = repository.AllZone().OrderBy(a => a.Name).ToList();
    List<string> s = new List<string>();
    s.Add(controllerName.ToLower());
    ViewBag.Products = repository.GetProducts(s).OrderBy(a => a.COMPONENTNAME).ToList();
    ViewBag.Sites = repository.GetSDOrg().OrderBy(a => a.NAME).ToList();
    ViewBag.Customers = repository.FindAccountDefinition(null).ToList();
}

我在我的action方法中调用了这个方法。那么重用代码的正确方法是什么? 感谢

1 个答案:

答案 0 :(得分:3)

有两种标准方法可以做到这一点。

第一种方法 - 覆盖控制器类的OnActionExecuting和/或OnActionExecuted方法:

public class HomeController: Controller
{
    protected override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        string controllerName = RouteData.Values["controller"].ToString();
        ViewBag.ControllerName = controllerName;
    }

    protected override void OnActionExecuted(ActionExecutedContext filterContext)
    {
        string controllerName = RouteData.Values["controller"].ToString();
        ViewBag.ControllerName = controllerName;
    }
}

您还可以创建实现这些方法的抽象基本控制器,然后从抽象控制器继承具体的控制器,这样就不会在每个控制器中复制代码。

第二种方法 - 制作自定义ActionFilter属性并装饰需要执行其他操作的每个控制器。

public class MyActionFilterAttribute: ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        string controllerName = filterContext.RouteData.Values["controller"].ToString();
        filterContext.Controller.ViewBag.ControllerName = controllerName;
    }

    public override void OnActionExecuted(ActionExecutedContext filterContext)
    {
        string controllerName = filterContext.RouteData.Values["controller"].ToString();
        filterContext.Controller.ViewBag.ControllerName = controllerName;
    }
}

然后装饰控制器,如:

[MyActionFilter]
public class HomeController: Controller
{
    // ....
}

更新:过滤器方法的额外灵活性,如果您需要基于每个操作进行过滤,而不是控制器中的所有操作,那么它也是可能的:

public class HomeController: Controller
{
    [MyActionFilter]
    public ActionResult MyAction()
    {
        //...
    }
}