将 BaseViewData 类作为 BaseController 类的属性是个坏主意吗？

Question

我项目中的每个控制器类都派生自一个基本控制器类，恰当地命名为 BaseController。

所有视图数据都包含在一个名为 BaseViewData 的类中（将来可能成为更具体的视图数据类的基本控制器）。

我在 BaseController 上创建了一个 BaseViewData 属性，因为每个控制器都需要访问强类型基础视图数据中的数据（并且 BaseController 做了一些工作来预填充一些 BaseViewData 属性）。

我这样做是因为：

1. 如果我更改了属性，我 会得到编译时错误 检查以解决更多损坏的代码 迅速地。

2. 练习 DRY，我已经 设法整合了很多代码 以前分散的 贯穿每个控制器。

但是，这是我第一次尝试这样做。所以我可能会忽略一个准备抬起丑陋脑袋的问题。所以：

让 BaseViewData 类成为 BaseController 类的属性是个坏主意吗？如果是，为什么？

更新 1：

我的 BaseController 看起来像（还有更多，但这应该能说明问题）：

public class BaseController
{
  public string Language {get; set;}
  public string Locale {get; set;}
  public BaseViewData Data {get; set;}

  protected override void OnActionExecuting(ActionExecutingContext filterContext)
  {
    var l = (RouteData.Values["language"] != null) ? RouteData.Values["language"].ToString() : "en";
    if (l.ToLower().Contains("en"))
    {
      l = "en";
    }
    else
     l = "ja";

    Data.Language = l;
  }
}

我的 BaseViewData 看起来像这样（同样，还有更多...）：

public class BaseViewData
{
  public string Language {get;set;}
  public string Locale {get;set;}
  public bool IsOwner {get;set;}
  public string Menu1 {get;set;}
  public string Menu2 {get;set;}
  public string Menu3 {get;set;}

  public IPagedList<TYPE> ListOfTYPE {get;set;}
  etc...
}

Answer 1

您的想法的菜单组件可能不是必需的，使用 ASP.NET MVC 2 Beta，您现在可以使用 Http.RenderAction 直接从视图调用控制器操作（例如从存储库并返回部分视图。

查看Haacked更多信息...

此外，对于语言/区域设置等更简单的内容，如果您使用 ASP.NET 配置文件提供程序（可通过控制器和视图访问），则可能不需要这样做。

Answer 2

对于我曾经在 ASP.NET MVC 中工作过的唯一网站，这正是我们所做的。这样做的好处还在于，我们能够在 BaseViewData 类中保存母版页所需的值。因为每个 View 都有一个派生 BaseViewData 的实例，所以我们可以安全地使用 Master Page 中 BaseViewData 中的数据。

Answer 3

恕我直言，你正在创造一个怪物。

随着您的应用程序的发展，越来越多的功能和全局屏幕将融入基本视图模型中。不可避免的是像 .aspx.cs 文件这样的神代码隐藏类，MVC 试图避免。

最好使用 MVC2 的 RenderAction 或 MVC Contrib 中的 SubController 之类的东西，即使这意味着稍微打破 MVC 模式。

看看像 CNN 甚至 Stackoverflow.com 这样的网站，你会在不知不觉中找到十几种方法。