Question

This code exists in a View:

       if (Model.Group.IsPremium && null != Model.Group.ContactInfo)
       {
           Html.RenderPartial("ContactInfo", Model.Group.ContactInfo);
       }

at first glance, it’s presentation logic and so it’s ok. But it’s not sitting well with me.

The thing is, it’s a business requirement to display contact info if the group is classified as premium, which means they paid.

What do you guys think? Should this logic be moved into a HtmlHelper or abstracted away by some other means? Or is this the intended usage of the View? What’s the best thing to do with this code?

Answer 1

I’d produce a ViewModel that encapsulates this logic as a boolean DisplayContactInfo property. It depends on how “clean” you want your views.