处理 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

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

见this article by Rob Conery

Answer 2

使用适当的视图模型（以及可选的火花视图引擎），没有标签汤。

特别是，如果使用模板。

关于这个例子不能说太多（更不用说 - 整理出来），但对我来说 - 通常都是关于整洁感和正确组织事物的能力。

---

“我正在寻找代码格式化解决方案。” => 那么你必须检查火花。即使它们确实包含逻辑，它也会“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"); %>