你會如何重構這個Asp.net MVC 2 Html Helper？

Question

快速問題。

你會如何重構這個Asp.net MVC 2 HtmlHelper？ 特別是在這種情況下使用TagBuilder類是否有意義？

 public static MvcHtmlString BusinessDisplayContacts(this HtmlHelper helper, string phone, string cellPhone, 
     string fax, string website, string email, bool hideEmail) 
    { 
     StringBuilder sb = new StringBuilder(); 

     sb.AppendLine("<ul>"); 
     if (!string.IsNullOrEmpty(phone)) { 
      sb.AppendLine("<li class=\"tel\">"); 
      sb.AppendLine("<span class=\"type\">Work</span>:"); 
      sb.AppendFormat("<span class=\"value\">{0}</span>",phone);    
      sb.AppendLine("</li>"); 
     } 
     if (!string.IsNullOrEmpty(cellPhone)) { 
      sb.AppendLine("<li class=\"tel\">"); 
      sb.AppendLine("<span class=\"type\">Cell</span> Phone:"); 
      sb.AppendFormat("<span class=\"value\">{0}</span>",cellPhone); 
      sb.AppendLine("</li>"); 
     } 
     if (!string.IsNullOrEmpty(fax)) { 
      sb.AppendLine("<li class=\"tel\">"); 
      sb.AppendLine("<span class=\"type\">Fax</span>:"); 
      sb.AppendFormat("<span class=\"value\">{0}</span>",fax); 
      sb.AppendLine("</li>"); 
     } 
     if (!string.IsNullOrEmpty(website)) { 
      sb.AppendFormat("<li><a class=\"url\" href=\"{0}\">{0}</a></li>",website); 
     } 
     if (!hideEmail && !string.IsNullOrEmpty(email)) { 
      sb.AppendFormat("<li><a class=\"email\" href=\"mailto:{0}\">{0}</a></li>",email); 
     } 
     sb.AppendLine("</ul>"); 

     if (sb.Length < 10) 
     { 
      return MvcHtmlString.Create(""); 
     } 
     else { 
      return MvcHtmlString.Create(sb.ToString()); 
     } 
    } 

在此先感謝。

更新：
感謝所有的建設性意見。 最後，我決定按照@ queen3的建議將上面的代碼移到強類型的局部視圖中。

Answer 1

有一件事我看到人們錯過了很多，正在使用C＃逐字字符串來處理類似的東西......例如，

sb.AppendLine("<li class=\"tel\">"); 
sb.AppendLine("<span class=\"type\">Work</span>:"); 
sb.AppendLine(string.Format("<span class=\"value\">{0}</span>",phone));    
sb.AppendLine("</li>"); 

可以做成

sb.AppendFormat(@" 
<li class=""tel""> 
    <span class=""type"">Work</span>: <span class=""value"">{0}</span> 
</li> 
", phone); 

其方式更具有可讀性。另一件事：我會把所有的字符串+ bool放在一個對象內，比如ContactInfo或者其他東西，把你的助手的簽名改爲BusinessDisplayContacts(this HtmlHelper helper, ContactInfo info) - 這樣你就可以添加/刪除/修改電話號碼和條件，而不需要打破現有的代碼。