Wie gestalte ich eine if-Anweisung innerhalb einer MVC-Ansicht

Ich hoffe diese Frage ist schnell und schmerzlos

Habe ich ein mvc-Ansicht, wo ich will, um entweder einen von zwei Werten abhängig von einer if-Anweisung. Dies ist, was ich in der Ansicht selbst:

 <%if (model.CountryId == model.CountryId) %>
        <%= Html.Encode(model.LocalComment)%> 
        <%= Html.Encode(model.IntComment)%>

Wenn wahre display-Modell.LocalComment, wenn falsche display-Modell.IntComment.

Diese nicht funktioniert, wie bekomme ich die beide Werte zeigt. Was mache ich falsch?

  • Wie C# - code im Allgemeinen sollten Sie nicht schreiben, if-Anweisungen ohne geschweifte Klammern {...}.
InformationsquelleAutor Cunners | 2011-01-01
  1. 11

    Ihre if - Anweisung immer den Wert true. Sie testen, ob model.CountryId gleich model.CountryId das ist immer wahr: if (model.CountryId == model.CountryId). Auch Sie sind, fehlt eine else - Anweisung. Es sollte wie folgt sein:

    <%if (model.CountryId == 1) { %>
    <%= Html.Encode(model.LocalComment) %> 
<% } else if (model.CountryId == 2) { %>
    <%= Html.Encode(model.IntComment) %>
<% } %>

    Natürlich müssen Sie ersetzen 1 und 2 mit den korrekten Werten.

    Persönlich würde ich schreiben, eine HTML-Hilfe für diese Aufgabe zu vermeiden, die tag-Suppe in den Blick:

    public static MvcHtmlString Comment(this HtmlHelper<YourModelType> htmlHelper)
{
    var model = htmlHelper.ViewData.Model;
    if (model.CountryId == 1)
    {
        return MvcHtmlString.Create(model.LocalComment);
    } 
    else if (model.CountryId == 2)
    {
        return MvcHtmlString.Create(model.IntComment);
    }
    return MvcHtmlString.Empty;
}

    Und dann in deinem Blick einfach:

    <%= Html.Comment() %>
    InformationsquelleAutor Darin Dimitrov
  2. 6

    Abgesehen von Darin der Punkt über die Krankheit ist immer die wahre, möchten Sie vielleicht zu prüfen, mit der bedingte operator:

    <%= Html.Encode(model.CountryId == 1 ? model.LocalComment : model.IntComment) %>

    (Passen Sie für was auch immer Ihre real Bedingung wäre, natürlich.)

    Ich persönlich finde das einfacher zu Lesen als die große Mischung von <% %> und <%= %>.

    • +1 für den oft übersehenen bedingte operator. Den null-coalescing operator - ?? - ist auch ganz nützlich.
    InformationsquelleAutor Jon Skeet
  3. 0

    Conditional Rendering in Asp.Net MVC-Ansichten 

     <% if(customer.Type == CustomerType.Affiliate) %>
   <%= this.Html.Image("~/Content/Images/customer_affiliate_icon.jpg")%>
 <% else if(customer.Type == CustomerType.Preferred) %>
   <%= this.Html.Image("~/Content/Images/customer_preferred_icon.jpg")%>
 <% else %>
   <%= this.Html.Image("~/Content/Images/customer_regular_icon.jpg")%>
    • Das ist ein hack. Eine schmutzige, schreckliche hack, der nur versucht sich zu verstecken code smell.
    • komm, "hack" ist ein bisschen stark hier. Seine grauen Bereich--sollte die Steuerung wissen, den Umgang mit dieser situation kann schwierig sein, für die Ansicht? Sollte die view-Anforderungen behandelt werden, indem eine neue Erweiterung Methode? Würde ich es nennen, hässlich und hart-zu-erhalten-als-Ihr-lohnt sich aber ich würde nicht nennen es eine schreckliche hack.
    • Eine Erweiterung Methode, die unweigerlich zwingt den Benutzer zu schreiben mehr HTML, und die Ergebnisse in den unteren Lesbarkeit und Wartbarkeit. Nein, ich denke, 'hack' ist eine faire Bewertung!
    InformationsquelleAutor Jahan Zinedine
Schreibe einen Kommentar