I am developing an e-commerce website in ASP.NET MVC. Users can post advertisements of different types on the site.

I am using inheritance to define my Ad types, and this review is mainly about taking advantage of the hierarchical structure to remove repeated code in Controllers and Views.

I have different Ad types: SimpleAd, Car and RealEstateRental.

Every Ad is derived from AdBase which has all the common properties:

public abstract class AdBase
{
public long AdBaseId { get; set; }
public bool IsActive { get; set; }
public long UserId { get; set; }
public string Title { get; set; }
public short AdDurationInDays { get; set; }
public string PhotosFolder { get; set; }
}


Now other Ads are derived from this base class:

public class SimpleAd : AdBase
{
public decimal Price { get; set; }
}

{
public decimal Price { get; set; }
public string Make { get; set; }
}

{
public decimal WeeklyRent { get; set; }
public DateTime AvailableFrom { get; set; }
public short NoOfBedrooms { get; set; }
public short NoOfBathrooms { get; set; }
}


I am using Entity Framework to interact with database and I am using Unit of Work and Repository patterns:

I have an Abstract base repository (the intention is to avoid rewriting the same function for each Ad type, so I have put the common functions here):

public interface IAdBaseRepository<TEntity> where TEntity : AdBase
{
}

{
{
}

{
.FirstOrDefault();
}
}


Other ad repositories inherit from the above class:

public interface ISimpleAdRepository : IAdBaseRepository<SimpleAd>
{
}

{
{
}
}

{
}

ICarRepository
{
public CarRepository(ApplicationDbContext context) : base(context)
{
}
}


And this is my Unit of Work:

public class UnitOfWork : IUnitOfWork
{

public UnitOfWork(ApplicationDbContext context)
{
Context = context;
RealEstateRental = new RealEstateRentalRepository(Context);
Car = new CarRepository(Context);
}

public IRealEstateRentalRepository RealEstateRental { get; private set; }
public ICarRepository Car { get; private set; }

public int SaveChanges()
{
return Context.SaveChanges();
}

public void Dispose()
{
Context.Dispose();
}
}


I am happy with everything so far... but the problem is I don't know how I can take advantage of this inheritance hierarchy in my Controllers and Views.

At the moment, I have 3 Controllers: SimpleAdController, CarController and RealEstateRentalController:

public class SimpleAdController : ControllerBase
{
private IUnitOfWork _unitOfWork;

{
_unitOfWork = unitOfWork;
}

[HttpGet]
public ActionResult Display(long id)
{
/*
* I have not included my ViewModel Classes in this code review to keep
* it small, but the ViewModels follow the same inheritance pattern
*/
}
}


CarController and RealEstateRentalController have similar display function, except the type of the Ad is different (e.g. in CarController I have):

    public ActionResult Display(long id)
{
Car car = _unitOfWork.Car.Get(id);
var carViewModel = Mapper.Map<CarViewModel>(car);
return View(car);
}


What I wanted to achieve was to create an AdBaseController to put all the common methods in it, something like this:

public class AdBaseController : ControllerBase
{
private IUnitOfWork _unitOfWork;

{
_unitOfWork = unitOfWork;
}

// Display for generic ad type
[HttpGet]
public ActionResult Display(long id)
{
/*
* I need to replace the above line with a generic ad type...
*/

/*
* similarly I have to replace the above 2 lines with a generic type
*/
}
}


If I do the above, then my Ad Controllers can inherit from it and I don't need to repeat the same Display Method in every one of them... but then I need to make my UnitOfWork generic... or have 2 UoW (generic and non-generic)... which I am not sure if it is a good idea? Any recommendation on having a AdBaseController?

Similarly I am repeating a lot of code in my Views. For example, this is the display SimpleAdView:

<div class="row">
<div class="col-l">
@*this partial view shows Ad photos and is common code for all ad types*@
</div>
<div class="col-r">
<div class="form-row">
<h5>@Model.Title</h5>
</div>

<div class="form-row">
<h5 class="price">$@Model.Price</h5> </div> @*Ad heading is common among all ad types*@ @Html.Partial("DisplayAd/_AdBaseHeading", Model) </div> </div> @*Ad Description is common among all ad types*@ @Html.Partial("DisplayAd/_Description", Model)  And this is my display CarView: <div class="row"> <div class="col-l"> @*Common in all ads*@ @Html.Partial("DisplayAd/_Photos", Model) </div> <div class="col-r"> <div class="form-row"> @*Common in all ads*@ <h5>@Model.Title</h5> </div> @*Price and Make are specific to Car*@ <div class="form-row"> <h5 class="price">$@Model.Price</h5>
</div>
<div class="form-row">
<h5 class="make">@Model.Make</h5>
</div>