Как бы вы повторно отразили этот помощник ASP.NET MVC 2 HTML?
-
29-09-2019 - |
Вопрос
Быстрый вопрос.
Как бы вы повторно отразили это 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.
Решение
Одна вещь, которую я вижу, много людей не хватает, это использование слов 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)
- Таким образом, вы сможете добавить/удалять/изменять номера телефонов и условия, не нарушая существующий код.