Я опубликовал этот обзор на codereview.stackexchange.com некоторое время назад ... Я чувствую, что он может быть более подходящим для stackoverflow, так как это больше вопрос, чем обзор кода.
Это займет немного объяснений, Пожалуйста, потерпите меня .
Я занимаюсь разработкой сайта электронной коммерции в ASP.NET MVC. Пользователи могут размещать на сайте рекламу разных типов.
Я использую наследование для определения типов моих объявлений, и этот вопрос касается использования преимуществ иерархической структуры для удаления повторяющегося кода в контроллерах и представлениях.
У меня разные типы объявлений: SimpleAd, Car и RealEstateRental.
Каждое объявление получено из AdBase, у которого есть все общие свойства:
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; }
}
Теперь другие объявления получены из этого базового класса:
public class SimpleAd : AdBase
{
public decimal Price { get; set; }
}
public class Car : AdBase
{
public decimal Price { get; set; }
public string Make { get; set; }
}
public class RealEstateRental : AdBase
{
public decimal WeeklyRent { get; set; }
public DateTime AvailableFrom { get; set; }
public short NoOfBedrooms { get; set; }
public short NoOfBathrooms { get; set; }
}
Я использую Entity Framework для взаимодействия с базой данных и использую шаблоны единиц работы и репозитория:
У меня есть общий AdBaseRepository со всеми распространенными методами рекламы:
public abstract class AdBaseRepository<TEntity> where TEntity : AdBase
{
protected readonly ApplicationDbContext Context;
public AdBaseRepository(ApplicationDbContext context)
{
Context = context;
}
public TEntity Get(long adBaseId)
{
return Context.AdBase.OfType<TEntity>()
.Where(r => r.IsActive == true && r.AdBaseId == adBaseId)
.FirstOrDefault();
}
// more common methods here...
}
Другие рекламные репозитории наследуются от вышеуказанного класса:
public class SimpleAdRepository : AdBaseRepository<SimpleAd>
{
public SimpleAdRepository(ApplicationDbContext context) : base(context)
{
}
}
public class CarRepository : AdBaseRepository<Car>
{
public CarRepository(ApplicationDbContext context) : base(context)
{
}
// methods which apply only to car here...
}
А это моя единица работы:
public class UnitOfWork
{
protected readonly ApplicationDbContext Context;
public UnitOfWork(ApplicationDbContext context)
{
Context = context;
SimpleAd = new SimpleAdRepository(Context);
RealEstateRental = new RealEstateRentalRepository(Context);
Car = new CarRepository(Context);
}
public SimpleAdRepository SimpleAd { get; private set; }
public RealEstateRentalRepository RealEstateRental { get; private set; }
public CarRepository Car { get; private set; }
public int SaveChanges()
{
return Context.SaveChanges();
}
public void Dispose()
{
Context.Dispose();
}
}
Я доволен всем до сих пор ... но проблема в том, что я не знаю, как использовать преимущества этой иерархии наследования в моих контроллерах и представлениях.
На данный момент у меня есть 3 контроллера: SimpleAdController, CarController и RealEstateRentalController:
public class SimpleAdController : ControllerBase
{
private UnitOfWork _unitOfWork;
public SimpleAdController(UnitOfWork unitOfWork)
{
_unitOfWork = unitOfWork;
}
[HttpGet]
// display specific ad
public ActionResult Display(long id)
{
SimpleAd simpleAd = _unitOfWork.SimpleAd.Get(id);
/*
* I have not included my ViewModel Classes in this question to keep
* it small, but the ViewModels follow the same inheritance pattern
*/
var simpleAdDetailsViewModel = Mapper.Map<SimpleAdDetailsViewModel>(simpleAd);
return View(simpleAdDetailsViewModel);
}
}
CarController и RealEstateRentalController имеют одинаковый метод Display, за исключением того, что тип объявления отличается (например, в CarController у меня есть):
public ActionResult Display(long id)
{
Car car = _unitOfWork.Car.Get(id);
var carViewModel = Mapper.Map<CarViewModel>(car);
return View(car);
}
То, чего я хотел добиться, - это создать AdBaseController, чтобы поместить в него все распространенные методы, что-то вроде этого:
public class AdBaseController : ControllerBase
{
private UnitOfWork _unitOfWork;
public AdBaseController(UnitOfWork unitOfWork)
{
_unitOfWork = unitOfWork;
}
// Display for generic ad type
[HttpGet]
public ActionResult Display(long id)
{
// SimpleAd simpleAd = _unitOfWork.SimpleAd.Get(id);
/*
* I need to replace the above line with a generic ad type...
* something like: _unitOfWork<TAd>.GenericAdRepository.Get(id)
*/
// var simpleAdDetailsViewModel = Mapper.Map<SimpleAdDetailsViewModel>(simpleAd);
// return View(simpleAdDetailsViewModel);
/*
* similarly I have to replace the above 2 lines with a generic type
*/
}
}
Если я сделаю вышеизложенное, то мои контроллеры рекламы могут наследовать его, и мне не нужно повторять один и тот же метод отображения в каждом из них ... но тогда мне нужно сделать мой UnitOfWork родовым .. или у вас 2 UoW (общие и неуниверсальные) ... что я не уверен, если это хорошая идея? Любая рекомендация о наличии AdBaseController?
Точно так же я повторяю много кода в моих представлениях. Например, это дисплей SimpleAdView:
<div class="row">
<div class="col-l">
@*this partial view shows Ad photos and is common code for all ad types*@
@Html.Partial("DisplayAd/_Photos", Model)
</div>
<div class="col-r">
<div class="form-row">
@*Common in all ads*@
<h5>@Model.Title</h5>
</div>
@*showing ad specific fields here*@
<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)
А это мой дисплей 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>
@*Common in all ads*@
@Html.Partial("DisplayAd/_AdBaseHeading", Model)
</div>
</div>
@*Common in all ads*@
@Html.Partial("DisplayAd/_Description", Model)
Опять же, я чувствую, что повторяю много кода в каждом представлении. Я попытался уменьшить количество повторяющихся кодов, поместив их в общие частичные представления. Я не уверен, есть ли лучший способ сделать это?