Asp.Net MVC/Entity Framework. 태그 수프를 피하십시오?

Question

좋아,

엔티티 프레임 워크를 사용하는 새로운 MVC 프로젝트가 있습니다. 나는 메시지를 뱉어 내고있다. (이것은 게시판 스타일 섹션이다.) 이제는 테이블 출력의 행에 다른 클래스 스타일이 있어야하는 몇 가지 조건적인 요인에 따라 다르다.

컨트롤러에서 페이지로 전달되는 모델은 엔티티 모델입니다 (호출 메시지 및 데이터베이스와 같은 필드 포함) 이제

행 스타일을 얻는을에서 나는 다음,

했다

<% 
       int i = 0; 
       foreach (var message in ViewData.Model.MessageList) 
       { 
        string className = "rowEven"; 

        if (i % 2 == 0) { className = "rowOdd"; } 
        if (message.Deleted) { className = "deleted"; } 
        if (message.AuthorisedBy == null) { className = "notAuth"; } 
        if (message.Deleted) { className = "deleted"; } 

      %> 
        <tr class="<%=className%>"> 
         <td><%= Html.CheckBox("mc1")%></td> 
         <td> 
          <%= Html.ActionLink(message.Title, "Details", new { id = message.MessageID })%> 
         </td> 
         <td>User Name Here</td> 
         <td><%= Html.Encode(message.PublishDateTime.ToString())%></td> 
        </tr>     
      <% 
        i++; 
       } 
      %> 

어느 쪽이 꽤 못 생깁니 까, 어떤 제안이 더 좋은 방법이 있어야합니까?

Answer 1

이것은 가장 확실하게 프레젠테이션 논리이며보기는 속해 있습니다. 그러나, 당신은 더 나은 모두 Message 및 MessageList 받아보기 도우미, 해당 CSS 클래스 선택 코드를 이동할 것 :

public static string GetMessageCssClassName(this /* Don't remember :) */, Message message, MessageList messages) 
{ 
    var cssClassName = messages.IndexOf(message) % 2 == 0 ? 
     "rowOdd" : "rowEven"; 

    if(message.Deleted) cssClassName = "deleted"; 
    if(message.AuthorisedBy == null) cssClassName = "notAuth"; 
    if(message.Deleted) cssClassName = "deleted"; 
} 

을 그리고 지금 당신은 i 카운터를 제거하기, 당신의 <tr class="Html.GetMessageCssClassName(...)">에 그를 호출 할 수

<% foreach (var message in ViewData.Model.MessageList) { %> 
    <tr class="<%= Html.GetMessageCssClassName(message, ViewData.Model.MessageList) %>"> 
     <td><%= Html.CheckBox("mc1")%></td> 
     <td> 
      <! -- Remaining stuff here --> 
<% } %> 

Answer 2

도우미 메서드를 추가하는 것은 어떻습니까? 당신은 기본 Html 헬퍼 클래스에 자신의 확장 방법을 쓸 수 있습니다, 그래서 당신은 같은 것을 쓸 수있다 : 여기 모델 - 뷰 - 뷰 모델을 사용하여 권하고 것

... 
<tr class="<%= Html.GetMessageCssClass(i, message) %>"> 
... 

Answer 3

합니다. 이를 통해 뷰 모델 클래스에 많은 로직을 캡슐화 할 수 있습니다. 그런 다음 로직을 인라인으로 배치하는 대신 뷰 모델에서 메소드를 호출하면됩니다. 이것은 당신이 더 많은 클래스와 끝까지하는 동안 당신이 더 많은 읽을 수있는 마크 업을 얻을

 <% foreach (var messageViewModel in ViewData.Model.MessageList) { %> 
       <tr class="<%=message.RowClass%>"> 
        <td><%= Html.CheckBox("mc1")%></td> 
        <td> 
         <%= Html.ActionLink(message.Title, "Details", new { id = message.MessageID })%> 
        </td> 
        <td>User Name Here</td> 
        <td><%= Html.Encode(message.PublishedAt)%></td> 
       </tr>     
     <% } %> 

... 그것은 다음과 같이 보일 만들 것입니다 및 응용 프로그램이 훨씬 더 쓸만한입니다. 보기를 아주 간단한 창으로보기 모델로 변환합니다. 그런 다음 뷰 모델은 뷰에서만 필요한 모든 논리 및 특성을 캡슐화합니다.

here은 MVC의 모델 -보기 -보기 모델 패턴을 처리하는 기사입니다. 속성 및/또는 메소드 묶음으로 클래스를 만들고이 클래스에 필요한 모든 것을 채워주면됩니다. 그런 다음 모델을 직접 전달하지 않고 클래스를보기로 전달하십시오. AutoMapper 또는 다른 매핑 프레임 워크를 사용하여 모델을 뷰 모델에 자동으로 매핑 할 수도 있습니다.

Answer 4

많이 언급했듯이 Message 클래스의 확장 메서드를 작성하여 Namespace.Site 프로젝트 (mvc 프로젝트) 내에 배치 할 수 있습니다. 같은

뭔가 :

public static class Extensions 
{ 
    public static string CssClass(this Message message,int counter) 
    { 
     if (i % 2 == 0) 
      return "rowOdd"; 
     else if (message.AuthorisedBy == null) 
      return "notAuth";  
     else if (message.Deleted) 
      return "deleted"; 

     return "rowEven"; 
    } 
} 

사용법 :

<tr class="<%=message.CssClass(i)%>"> 
    <td><%= Html.CheckBox("mc1")%></td> 
    <td><%= Html.ActionLink(message.Title, "Details", new { id = message.MessageID })%></td> 
    <td>User Name Here</td> 
    <td><%= Html.Encode(message.PublishDateTime.ToString())%></td> 
</tr>