使用 MvcContrib Grid 之类的东西是否会导致代码可读性倒退？

Question

我的意思是，现在我们已经采取了所有这些行动，以使用现代模板引擎尽可能地将 html 标记与代码分离（在过去，程序员通常只是在 php 中连接字符串，这很糟糕。）

然后我看了一下 co -worker生成html表格的代码，看起来是这样的：

<% Html.Grid(Model).Columns(column => {
    column.For(x => Html.ActionLink("Edit", "Edit", new { id = x.Id })).Attributes(width => "30px").DoNotEncode();
    column.For(x => Html.ActionLink("Delete", "Delete", new { id = x.Id }, new { @class = "delete" })).Attributes(width => "95px").DoNotEncode();
    column.For(x => x.Id).Named("Code");
    column.For(x => x.Name).Named("Name").HeaderAttributes(align => "left");
    column.For(x => x.CPF).Named("CPF");
})
.Attributes(width => "100%", border => "0", cellpadding => "0", cellspacing => "0", @class => "data-table")
.Empty("No users found!")
.RowStart(row => string.Format("<tr class='row{0}'>", row.IsAlternate ? "-alternating" : ""))
.Render();
%>

他觉得很棒，我觉得很丑，所以我想了解更多人的意见。

Answer 1

乍一看很难看，但我认为它相当简洁并且考虑了很多逻辑。例如，标准 MVC 输出可以执行与此类似的操作（没有标记）

    <table>
    <% dim index as integer = 0
      if model.count > 0 then %>
       <tr>
        <th>Edit</th>
        <th>Delete</th>
        <th>Code</th>
        <th align="left">Name</th>
        <th>CFP</th>
       for each item in model
         if index mod 2 = 0 then%>
           <tr>
         <%else%>
           <tr class ="alternate">
         <%end if%>
           <td><%= html.actionLink("Edit", "Edit", New With {.id = item.id})%></td>
           <td><%= html.actionLink("Delete", "Delete", New With {.id = item.id})%></td>
           <td><%: item.id%></td>
           <td><%: item.Name%></td>
           <td><%: item.CPF%></td>
         <% index += 1
            Next%>
    <%Else%>
        <tr><td>No Rows Found</td></tr>
    <%end if%>

</table>

，而且我很确定我也错过了其中的一些内容。传统上，我必须跟踪行数或以某种方式知道它是否是交替行，有一种情况是我没有记录并且通常有很多 <% %><% %><% %> code> 标签来处理这些代码块。

Answer 2

对于设计师来说，这是代码可读性的倒退，这里没有两种意见。

从开发人员的角度来看，这将取决于。有些人喜欢，有些人不喜欢。就我个人而言，我喜欢它，并且与更标准的 foreach 技术相比更喜欢它。为什么？

您从一个简单的 和一个 foreach 开始。然后有些用户说您需要处理交替的行样式。首先添加 ifs。然后另一个用户说您需要能够按给定列进行排序。您可以使用另一个 ifs 来处理这个问题。第三个用户要求您处理分页 =>您的视图中还有另一个 ifs 和 foreach 。你最终会得到意大利面条。

结论：用正确的枪瞄准正确的目标。对于简单的表，传统方法效果很好，但是一旦你开始做更高级的事情，就使用助手。

Answer 3

MVC Contrib Grid 看起来很可怕，因为它拥有大量的功能。它是一个瑞士陆军级 HTML 扩展，可以为所有人做所有事情。正如其他人指出的那样，手工操作更加困难或更笨拙。我记得回到桌面时，编写同样笨拙的代码来配置网格和电子表格控件以获得正确的效果。

但是，如果您的需求比较温和，则只需编写自己的需求即可。 Phil Haack 有一篇关于 基于代码的转发器。还有一些更轻的 网格视图助手可能会为您提供帮助。这些功能并不那么强大，但可能更适合您应用程序中的某些工作。

Answer 4

我同意它不是很漂亮，但是如果您开始定义表的外观约定，您可以开始重构扩展方法来清理它。然后它甚至会干掉你的表定义。这是一个示例（减去实际的扩展方法）。

<% Html.Grid(Model).Columns(column => {
    InsertEdit(column).Width(30);
    InsertDelete(column).Width(95);
    column.For(x => x.Id).Named("Code");
    column.For(x => x.Name).Named("Name").LeftAlignHeader();
    column.For(x => x.CPF).Named("CPF");
})
.ApplyDefaultStyle()
.Class("data-table")
.Empty("No users found!")
.DefaultAlternatingStyle()
.Render();
%>