ASP.NET MVC 3 中 ViewBag.Title 的替代方案

Question

默认情况下，ASP.NET MVC 3 的新项目模板将以下内容添加到默认布局（razor 中的母版页）：

<title>@ViewBag.Title</title>

然后，视图必须包含以下内容来分配页面标题，例如：

@{
     ViewBag.Title = "Log On";
}

也许这只是我自己的偏好但我发现使用 ViewBag 来保存标题有点错误（我认为太多了魔术字符串的味道）。所以我的问题是：对于使用 ASP.NET MVC 3 和 razor（使用动态属性包）的人来说，这是推荐的最佳实践吗？或者您是否选择更强类型的东西（可能涉及自定义基类？）

Answer 1

我不认为 ASP.NET MVC 3 附带的默认标题处理功能有什么不好，可以这样做。

我个人这样做（下面写）来处理标题，我不认可下面的代码或说它比默认功能更好，它只是一个偏好。

Master

<title>
    @RenderSection("Title");
</title>

View

@section Title
{
    write title
}

我可以建议改进默认功能的一件事

@{
    string pageTitle = @ViewBag.Title ?? "Title Not Set";
}
<title>@pageTitle</title>

因此，每当您忘记将其添加到 viewbag 中时，页面都会显示 title= Title Not Set

创建一个基类，然后创建所有从该基类继承的控制器也可以完成。但我认为它为 title 带来了很大的痛苦。

Answer 2

标题的 ViewBag 完全没问题（我什至会说这是拥有 ViewBag 的目的） - 动态并不是绝对的邪恶。 “标题”是众所周知的，不太可能改变，甚至在视图模板中预定义。我个人使用以下标题：

<title>@(ViewBag.Title == null ? string.Empty : ViewBag.Title + " | ")Site Name</title>

如果您担心输入错误 ViewBag.Title，您可以通过创建自定义 WebViewPage 使其成为强类型，但您仍然必须使用 ViewBag 或者可能是该强类型属性内的 HttpContext.Items，因为在渲染 IIRC 期间创建了多个 WebViewPage 实例。

我建议坚持使用ViewBag，创建自己的WebViewPage，因为这看起来有点过分 - 如果您已经有自定义WebViewPage，甚至可以在其上创建单个属性code> 在我看来只是毫无价值的复杂化——而这来自于经常过度设计事物的人。

Answer 3

我也根本不使用 ViewBag。

在 _Layout.shtml 的顶部 ...

@model <YourWebAppNameSpace>.Models.Base.EveryPageViewModel

在 _Layout.shtml 中 ...

<title>@Model.Title</title>

在您的模型中 ...

/// <summary>
/// Index View Model
/// </summary>
public class IndexViewViewModel : EveryPageViewModel {

}

在 EveryPageViewModel 中

/// <summary>
/// Every Page View Model
/// </summary>
public abstract class EveryPageViewModel {
    /// <summary>
    /// Title
    /// </summary>
    public string Title { get; set; }
    /// <summary>
    /// Sub Title
    /// </summary>
    public string SubTitle { get; set; }
}

在您的控制器操作中

    /// <summary>
    /// Index
    /// </summary>
    /// <returns></returns>
    public ActionResult Index() {
        var model = new IndexViewViewModel();
        model.Title = "Home";
        return View(model);
    }

Answer 4

我喜欢创建 PageTitle ActionFilter 属性，而不是编辑单个 ViewBags

用法：保持视图相同

<title>@ViewBag.Title</title>

对于控制器范围的页面标题：

[PageTitle("Manage Users")]
public class UsersController : Controller {
    //actions here
}

对于单个视图：

public class UsersController : Controller {
    [PageTitle("Edit Users")]
    public ActionResult Edit(int id) {
          //method here
    }
}

属性代码：

public class PageTitleAttribute : ActionFilterAttribute
{
    private readonly string _pageTitle;
    public PageTitleAttribute(string pageTitle)
    {
        _pageTitle = pageTitle;
    }

    public override void OnActionExecuted(ActionExecutedContext filterContext)
    {
        base.OnActionExecuted(filterContext);
        var result = filterContext.Result as ViewResult;
        if (result != null)
        {
            result.ViewBag.Title = _pageTitle;
        }
    }
}

易于管理，工作起来就像一个魅力。

Answer 5

就我个人而言，我认为这种情况是 ViewBag 的可接受使用。它仅限于“众所周知”的属性，并且将来可能不会造成任何问题。最后，一切都是为了务实并找到尽可能快的方法。在我看来，拥有一个需要设置标题的基类代码太多，不值得类型安全。

祝你好运！

Answer 6

我想说只要你想设置的只是Title，用ViewBag就可以了。好吧，也许不仅如此——最多 2-3 个属性。

但是，如果您开始发现在每个控制器操作中设置越来越多的（通用）属性，我会选择强类型的“ViewModelBase 类”。但这只是我。

Answer 7

我们更喜欢强大的标题设置......来自我们的 BaseController 类的一些示例。 （页面定义封装视图模式）

protected override ViewResult View(string viewName, string masterName, object model)
{
    if (model is Page)
    {
        ViewBag.Title = ((Page)model).Title;
        //HACK: SEO
        //TODO: SEO
    }
    return base.View(viewName, masterName, model);
}

Answer 8

使用 ViewBag.Title = "My Title"; 没有任何问题。

您所做的就是使用动态属性。

问题实际上是信息应该在哪里“声明”。

即，对于当前的目的来说，它在哪里最容易实现。

如果它是在每页的基础上，那么那就是正确的地方。

但是，如果页面标题可以从模型派生，那么您应该这样做。

在这种情况下，我可能会使用您使用的 ViewModel 的基类，并在其中创建一个 PageTitle 属性，其中包含从模型中的属性派生页面标题的逻辑。

所以：

<title>Model.PageTitle</title>

总之，马匹适合课程，不要害怕使用动态属性......只要您了解它们是什么以及它们做什么。