ASP.NET MVC brzytwa: atrybut warunkowy w HTML

Question 1

Poniższy kod nie wydaje się czysty. Jakieś sugestie dotyczące ulepszenia kodu?

<li @if(ViewData["pagename"].ToString()=="Business details"){ <text>class="active" </text> } >
        <a  @if(ViewData["pagename"].ToString()=="Business details"){ <text>style="color: white; background-color: #08C; border: 1px solid #08C;" </text> }
            href="@Url.Action("BusinessDetails", "Business")">Business Details</a>
    </li> 
    <li @if (ViewData["pagename"].ToString() == "Booking policies"){ <text>class="active"</text> }> 
        <a  @if (ViewData["pagename"].ToString() == "Booking policies")
               { <text>style="color: white; background-color: #08C; border: 1px solid #08C;" </text> }
            href="@Url.Action("BookingPolicies", "Business")">Booking policies</a> 
    </li>

Question 2

MVC ma wbudowane atrybuty warunkowe ...

<div @{if (myClass != null) { <text>class="@myClass"</text> } }>Content</div>
<div class="@myClass">Content</div>

Jeśli @myClass ma wartość null, po prostu w ogóle nie użyje atrybutu ...

Wiem, że może to nie rozwiązać Twojego obecnego problemu, ale jest godny uwagi!

http://weblogs.asp.net/jgalloway/archive/2012/02/16/asp-net-4-beta-released.aspx

Question 3

<li class="@(ViewBag.pagename == "Business details" ? "active" : null)">

Powinieneś zamienić inline style="..."na oddzielną nazwę klasy i użyć tam tej samej składni.

Byłoby jednak bardziej przejrzyste utworzenie oddzielnej metody rozszerzenia pomocnika HTML, która pobiera nazwę strony i akcji oraz generuje kod HTML w sposób ogólny.

Question 4

Używam małej metody pomocniczej, która warunkowo doda atrybut, jeśli wartość nie jest pusta i, jeśli jest zdefiniowana, gdy wartość wyrażenia funkcji logicznej wyniesie true:

public static MvcHtmlString Attr(this HtmlHelper helper, string name, string value, Func<bool> condition = null)
{
    if (string.IsNullOrEmpty(name) || string.IsNullOrEmpty(value))
    {
        return MvcHtmlString.Empty;
    }

    var render = condition != null ? condition() : true;

    return render ? 
        new MvcHtmlString(string.Format("{0}=\"{1}\"", name, HttpUtility.HtmlAttributeEncode(value))) : 
        MvcHtmlString.Empty;
}

Po zdefiniowaniu mogę użyć tej metody w moich widokach Razor:

<li @(Html.Attr("class", "new", () => example.isNew))>
...
</li>

Powyższy kod wyrenderuje się, <li class="new">...</li>jeśli example.isNew == true, jeśli nie, pominie cały classatrybut.

Question 5

W MVC4

<!DOCTYPE html>
<html>
<head>
</head>
<body>
    @{
        string css = "myDiv";
    }
    <div class='@css'></div>
</body>
</html>

lub

<!DOCTYPE html>
<html>
<head>
</head>
<body>
    @{
        string css = "class=myDiv";
    }
    <div @css></div>
</body>
</html>

Więcej tutaj: http://evolpin.wordpress.com/2012/05/20/mvc-4-code-enhancements/

Question 6

Podejście z metodą rozszerzenia TagWrap. Kod Twojego pytania wyglądałby tak:

@using (Html.TagWrap("li", condition ? new { @class = "active" } : null))
{
    var anchorAttrs = new Dictionary<string, object> { { "href", Url.Action("BusinessDetails", "Business") } };
    if(condition)
    {
        anchorAttrs["style"] = "color: white; background-color: #08C; border: 1px solid #08C;";
    }
    using (Html.TagWrap("a", anchorAttrs))
    {
        <text>Business Details</text>
    }
}

Metody rozszerzenia TagWrap

using Microsoft.AspNetCore.Mvc.ViewFeatures;

public static IDisposable TagWrap(this IHtmlHelper htmlHelper, string tagName, object data)
{
    return htmlHelper.TagWrap(tagName, HtmlHelper.AnonymousObjectToHtmlAttributes(data));
}

public static IDisposable TagWrap(this IHtmlHelper htmlHelper, string tagName, IDictionary<string, object> data)
{
    var tag = new TagBuilder(tagName);
    tag.MergeAttributes(data);

    htmlHelper.ViewContext.Writer.Write(tag.RenderStartTag());

    return new DisposableAction(() =>
        htmlHelper.ViewContext.Writer.Write(tag.RenderEndTag()));
}

Klasa pomocnicza używana do renderowania tagu zamykającego w Dispose

public class DisposableAction : IDisposable
{
    private readonly Action DisposeAction;

    public DisposableAction(Action action)
    {
        DisposeAction = action;
    }

    public void Dispose()
    {
        DisposeAction();
    }
}

Question 7

Na podstawie rozmrażania odpowiedz tutaj na adaptację, przyjmując objectzamiast a string:

    public static MvcHtmlString ConditionalAttr(this HtmlHelper helper, string attributeName, object value, Func<bool> condition)
    {
        if (string.IsNullOrEmpty(attributeName) || value == null)
        {
            return MvcHtmlString.Empty;
        }

        var render = condition != null ? condition() : true;

        return render ? 
            new MvcHtmlString($"{attributeName}=\"{HttpUtility.HtmlAttributeEncode(value.ToString())}\"") : 
            MvcHtmlString.Empty;
    }

W ten sposób nie musisz zamieniać innych typów danych w łańcuchy przed ich przekazaniem, oszczędzając fiew .ToString(). Jest jednak różnica : przekazanie pustego ciągu będzie nadal renderowane. Jako przykład:

@Html.ConditionalAttr("data-foo", "", () => Model.IsFooNeeded)

// Ouput:
data-foo=""

Question 8

@{ var classAttr= needClass ? "class=\"class-name\"" : "" }

a następnie w kodzie HTML

<li @Html.Raw(classAttr) >