Это плохая практика?
Да. . Фактически он нарушает шаблон MVC - View не должен перезванивать через модель, а только получать, чтобы выполнить свою единственную работу: рендеринг HTML.
Если вам нужна дополнительная информация, а не только одна сущность, заполните ViewModel всей необходимой вам информацией, а затем передайте ее в View.
Кроме того, не просматривайте IEnumerable
в модели, используйте Шаблон отображения:
@Html.DisplayForModel()
Преимуществом этого является отсутствие явного цикла, использующего преимущества соглашений MVC и придерживающегося иерархии моделей при привязке модели.