Is it bad practice to use foreach inside the View?

Question

It’s a bad practice to use foreach within the View? If it is, how to do it?

Answer 1

Not!

Bad practice is to carry out business rules in the view. Another very common bad practice is code repetition, copy and paste always the same code instead of creating a partial view.

But, for foreach only, it is impossible to say. Bad practice may be on what you are doing in the loop, but not because you are using it.

Answer 2

Is bad practice?

No. The foreach can be used to display data in the View normally.

Example: Assuming that my model (Product-type class viewmodel) has a list of products and I wish to display the name and value of each product.

@model ProdutosViewModel
    ...
    @foreach produto in Model.Produtos
    {
        //Código para exibir dados do produto
        <tr>
         <td>@produto.Nome</td>
         <td>@produto.Valor</td>
        </tr>
    }
    ...

But if you have too many records to iterate it is interesting to use a pagination feature for example, even for a better user experience.

When people say no logic in the View, they are usually referring to business logic. Once you are using the foreach just to display information on the page is not a bad practice to use it in the View.