所以,我把所有这些逻辑放在我看来,认为遵循DRY原则会有所帮助。
基本上,这会复制纸质表单,在我的视图模型中,我有一对名为XXXCondition
和XXXComment
的属性,这些属性在我的实际模型中组合成InspectionDetail
。
<table>
<% string secname ="";
foreach (PropertyInfo p in typeof(InspectionFormModel).GetProperties()) {
object[] sec = p.GetCustomAttributes(typeof(InspectionSection), false);
object[] name = p.GetCustomAttributes(typeof(DisplayNameAttribute), false);
string propname = p.Name;
/* Display a row for all view model properties having InspectionSection Attribute */
if ( sec.Length > 0) {
/* New Subheading */
if (((InspectionSection)sec[0]).Section.ToString() != secname) {
secname = ((InspectionSection)sec[0]).Section.ToString();
var secdesc = typeof(InspectionDetail.DetailTypes).GetMember(secname)[0].GetCustomAttributes(typeof(DescriptionAttribute), false);
string ssecdesc = (secdesc.Length == 0) ? secname : ((DescriptionAttribute)secdesc[0]).Description;
%> <tr><th><%= ssecdesc %></th><th>Condition</th><th>Remarks</th></tr><%
}
/* Use DisplayName attribute for item instead of property name if specified */
if (name.Length > 0)
{
propname = ((DisplayNameAttribute)name[0]).DisplayName;
}%>
<tr>
<td><label for="<%= p.Name %>"><%= propname %></label></td>
<td><%= Html.DropDownList(p.Name, (IEnumerable<SelectListItem>)ViewData["ItemConditions"]) %></td>
<td><%= Html.TextBox(p.Name.Replace("Condition", "Comment"), null, new {Class ="comment"}) %></td>
</tr>
<% } } %>
</table>
视图中的所有代码看起来都是一个明确的反模式。会有更好的地方隐藏这个吗?或者依赖于视图模型中的所有定义的订单属性是愚蠢的吗?
答案 0 :(得分:1)
您是否看过像MVC模板这样的内容:http://bradwilson.typepad.com/blog/2009/10/aspnet-mvc-2-templates-part-1-introduction.html?
您可能还应该创建一个自定义视图模型,并在各种属性等的控制器/操作方法中初始化它。