MVC Razor @foreach

Słyszałem, że posiadanie @foreach wewnątrz widoku to nie-nie. Oznacza to, że widok nie powinien zawierać żadnej logiki. Jaka jest najlepsza praktyka dotycząca tego, gdzie powinna znajdować się logika @foreach?

    @foreach.. 

Jaka jest najlepsza praktyka dotycząca tego, gdzie powinna znajdować się logika @foreach?

Nigdzie, po prostu się go pozbądź. Możesz użyć edytora lub szablonów wyświetlania.

Na przykład:

@foreach (var item in Model.Foos)
{
    <div>@item.Bar</div>
}

można by idealnie zastąpić szablonem wyświetlania:

@Html.DisplayFor(x => x.Foos)

a następnie zdefiniujesz odpowiedni szablon wyświetlania (jeśli nie podoba ci się domyślny ). Więc zdefiniowałbyś szablon wielokrotnego użytku, ~/Views/Shared/DisplayTemplates/Foo.cshtmlktóry będzie automatycznie renderowany przez framework dla każdego elementu kolekcji Foos ( IEnumerable<Foo> Foos { get; set; }):

@model Foo
<div>@Model.Bar</div>

Oczywiście dokładnie te same konwencje dotyczą szablonów edytorów, których należy używać w przypadku, gdy chcesz wyświetlić niektóre pola wejściowe umożliwiające edycję modelu widoku, w przeciwieństwie do wyświetlania go jako tylko do odczytu.

Kiedy ludzie mówią, że nie umieszczaj logiki w widokach, zwykle odnoszą się do logiki biznesowej, a nie renderowania logiki. Moim skromnym zdaniem uważam, że używanie @foreach w widokach jest w porządku.

Używam, @foreachgdy wysyłam encję zawierającą listę jednostek (na przykład w celu wyświetlenia 2 siatek w 1 widoku)

Na przykład, jeśli wysyłam jako model jednostkę Foo, która zawiera Foo1(List<Foo1>)iFoo2(List<Foo2>)

Mogę odnieść się do pierwszej listy z:

@foreach (var item in Model.Foo.Foo1)
{
    @Html.DisplayFor(modelItem=> item.fooName)
}

odpowiedź dla @DarinDimitrov na przypadek, w którym użyłem foreach w widoku brzytwy.

<li><label for="category">Category</label>
        <select id="category">
            <option value="0">All</option>
            @foreach(Category c in Model.Categories)
            {
                <option title="@c.Description" value="@c.CategoryID">@c.Name</option>
            }
        </select>
</li>

Odpowiedź nie zadziała, gdy użyjesz przeciążenia do wskazania szablonu @Html.DisplayFor(x => x.Foos, "YourTemplateName).

Wydaje się być zaprojektowany w ten sposób, zobacz ten przypadek . Również wyjątek, który daje framework (o typie nie był zgodny z oczekiwaniami) jest dość mylący i oszukał mnie za pierwszym razem (dzięki @CodeCaster)

W takim przypadku musisz użyć@foreach

@foreach (var item in Model.Foos)
{
    @Html.DisplayFor(x => item, "FooTemplate")
}