Я разместил этот обзор на 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">[email protected]</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">[email protected]</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)
Опять же, я чувствую, что повторяю много кода в каждом представлении. Я попытался уменьшить количество повторяющихся кодов, поместив их в общие частичные представления. Я не уверен, есть ли лучший способ сделать это?