处理 ASP.NET MVC“标签汤”

Question

我今天正在开发一个 ASP.NET MVC 模板，在盯着那些荧光黄色 % 标签看了足够长的时间后，我基本上决定我已经受够了，所以我煞费苦心地修改了我的 ascx 文件来看看就像这样：

<%@ Control Language="C#" Inherits="System.Web.Mvc.ViewUserControl"         %>

<%  if (Model == null)
    {                                                                       %>
<%=     ViewData.ModelMetadata.NullDisplayText                              %>
<%  }
    else if (ViewData.TemplateInfo.TemplateDepth > 1)
    {                                                                       %>
<%=     ViewData.ModelMetadata.SimpleDisplayText                            %>
<%  }
    else
    {                                                                       %>
<%      foreach (var prop in ViewData.ModelMetadata.Properties.Where(
            pm => pm.ShowForDisplay && !ViewData.TemplateInfo.Visited(pm)))
        {                                                                   %>
<%          if (prop.HideSurroundingHtml)
            {                                                               %>
<%=             Html.Display(prop.PropertyName)                             %>
<%          }
            else
            {                                                               %>
<%              if (!String.IsNullOrEmpty(prop.GetDisplayName()))
                {                                                           %>
                    <span class="display-label">
<%=                     prop.GetDisplayName()                               %>
                    </span>
<%              }                                                           %>
                <span class="display-field">
<%=                 Html.Display(prop.PropertyName)                         %>
                </span>
<%          }                                                               %>
<%      }                                                                   %>
<%  }                                                                       %>

啊，终于可读了。唯一的问题是，手动完成此操作需要很长时间。我需要一种方法来自动化此操作。某种代码格式化解决方案。也许是宏或 Visual Studio 加载项或...？你有什么建议？

更新

我现在计划重构标记中的大部分逻辑（请参阅下面 Mike 的回答），但与此同时，我想出了一种更易于管理的方法来格式化具有以下内容的 ascx 文件：代码和 html 的混合体。这样代码会更加分散，但是首先像这样格式化代码要容易得多，而且使用起来也容易得多。

<%@ Control Language="C#" Inherits="System.Web.Mvc.ViewUserControl" %>

<% 
    if (Model == null)
    {
%>
        <%= ViewData.ModelMetadata.NullDisplayText %>
<%
    }
    else if (ViewData.TemplateInfo.TemplateDepth > 1)
    {
%>
        <%= ViewData.ModelMetadata.SimpleDisplayText %>
<%
    }
    else
    {
%>
<%
        foreach (var prop in ViewData.ModelMetadata.Properties.Where(
            pm => pm.ShowForDisplay && !ViewData.TemplateInfo.Visited(pm)))
        {
            if (prop.HideSurroundingHtml)
            {
%>
                <%= Html.Display(prop.PropertyName) %>
<%
            }
            else
            {
%>
                <div class="display-row">   
<%
                if (!String.IsNullOrEmpty(prop.GetDisplayName()))
                    {
%>
                        <div class="display-label">
                            <%= prop.GetDisplayName() %>
                        </div>
<%
                }
%>
                    <div class="display-field">
                        <%= Html.Display(prop.PropertyName) %>
                    </div>
            </div>
<%
            }
        }
    }
%>

Answer 1

我相信你的“标签汤”痛苦实际上是另一个问题的症状：你感觉到它是因为你的观点有逻辑。视图应该非常轻量级，很少或没有逻辑。您的逻辑应该位于控制器中，控制器可以根据逻辑决定渲染不同的视图。或者你可以将逻辑放入助手中。

请参阅Rob Conery 撰写的这篇文章

Answer 2

有了正确的视图模型（也可以选择使用 Spark 视图引擎），就不存在标签汤了。

特别是，如果使用模板。

关于这个例子不能说太多（甚至更少 - 整理它），但对我来说 - 通常这都是关于整洁感和正确构建事物的能力。

---

“我正在寻找代码格式化解决方案。” =>那么你必须检查火花。它会“html 化”您的视图，即使它们确实包含逻辑。

你的例子（没有重组任何东西）=>

<var nullText="ViewData.ModelMetadata.NullDisplayText"
     templateDepth="ViewData.TemplateInfo.TemplateDepth"
     simpleDisplayText="ViewData.ModelMetadata.SimpleDisplayText"
     props="ViewData.ModelMetadata.Properties.Where(pm => pm.ShowForDisplay && !ViewData.TemplateInfo.Visited(pm))" 
     m="ViewData.Model"/>
<if condition="m==null">
  ${nullText}
</if>
<elseif condition="templateDepth>1">
  ${simpleDisplayText}
</elseif>
<else>
  <for each="var prop in props">
    <if condition="prop.HideSurroundingHtml">
      ${Html.Display(prop.PropertyName)}
    </if>
    <else>
      <span if="!string.IsNullOrEmpty(prop.GetDisplayName()" class="display-field">
        ${prop.GetDisplayName()}
      </span>
      <span class="display-field">
        ${Html.Display(prop.ProperyName)}
      </span>
    </else>
  </for>
</else>

可能有些地方是错的，但你明白了。

Answer 3

对于我们当前的项目，我们通过编写 HTML 帮助程序来清理所有的 if-else：

public static void WriteIf(this HtmlHelper helper, bool condition, string truePart, string falsePart)
{
     helper.ViewContext.HttpContext.Response.Write(condition ? truePart : falsePart);
}

然后在 HTML 中，您会说：

<% Html.WriteIf(Model.IsTrue(), "TrueText", "FalseText"); %>