- html - 出于某种原因,IE8 对我的 Sass 文件中继承的 html5 CSS 不友好?
- JMeter 在响应断言中使用 span 标签的问题
- html - 在 :hover and :active? 上具有不同效果的 CSS 动画
- html - 相对于居中的 html 内容固定的 CSS 重复背景?
我发布了this review不久前在 codereview.stackexchange.com 上...我觉得它可能更适合 stackoverflow,因为它更多的是一个问题而不是代码审查。
需要一些解释,请耐心等待。
<小时/>我正在 ASP.NET MVC 中开发一个电子商务网站。用户可以在网站上发布不同类型的广告。
我使用继承来定义我的广告类型,这个问题是关于利用分层结构删除 Controller 和 View 中的重复代码。
我有不同的广告类型: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();
}
}
到目前为止,我对一切都很满意...但问题是我不知道如何在 Controller 和 View 中利用这种继承层次结构。
目前,我有 3 个 Controller :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
*/
}
}
如果我执行上述操作,那么我的广告 Controller 可以继承它,并且我不需要在每个 Controller 中重复相同的显示方法...但随后我需要创建我的 UnitOfWork
通用...或有 2 个 UoW(通用和非通用)...我不确定这是否是一个好主意? 关于 AdBaseController
的任何建议?
同样,我在 View 中重复了很多代码。例如,这是显示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)
我再次感觉我在每个 View 中重复了很多代码。我尝试通过将重复代码放入公共(public)部分 View 中来减少重复代码的数量。 我不确定是否有更好的方法来做到这一点?
最佳答案
从技术上来说这是可能的。对于类似的实体,您可以引入枚举并使用它来指示您在 Controller 中处理的实体类型。您可以创建通用 View 来处理类似的广告(但当然,您需要根据模型广告类型显示/隐藏相应的 UI 元素)。这是 Controller
的伪代码来说明这个想法:
using System.Threading.Tasks;
using AutoMapper;
using MyNamespace.Data;
using Microsoft.AspNetCore.Mvc;
using MyNamespace.ViewModels;
namespace MyNamespace
{
public enum AdType
{
[Description("Simple Ad")]
SimpleAd = 0,
[Description("Car")]
Car = 1,
[Description("Real Estate Rental")]
RealEstateRental = 2
}
public class AdController : Controller
{
private readonly ApplicationDbContext _context;
private readonly IMapper _mapper;
public AdController(
ApplicationDbContext context,
IMapper mapper)
{
_context = context;
_mapper = mapper;
}
[HttpGet("Ad/{type}")]
public IActionResult Index(AdType? type = AdType.SimpleAd)
{
switch (type)
{
case AdType.RealEstateRental:
return RedirectToAction("RealEstateRental");
case AdType.Car:
return RedirectToAction("Car");
case AdType.SimpleAd:
default:
return RedirectToAction("SimpleAd");
}
}
[HttpGet("Ad/Car")]
public IActionResult Car()
{
return View("Index", AdType.Car);
}
[HttpGet("Ad/RealEstateRental")]
public IActionResult RealEstateRental()
{
return View("Index", AdType.RealEstateRental);
}
[HttpGet("Ad/SimpleAd")]
public IActionResult SimpleAd()
{
return View("Index", AdType.SimpleAd);
}
[HttpGet("Ad/List/{type}")]
public async Task<IActionResult> List(AdType type)
{
// var list = ... switch to retrieve list of ads via switch and generic data access methods
return list;
}
[HttpGet("Ad/{type}/Details/{id}")]
public async Task<IActionResult> Details(AdType type, int id)
{
var ad = // ... switch by type to retrieve list of ads via switch and generic data access methods
if (ad == null) return NotFound($"Ad not found.");
// for instance - configure mappings via Automapper from DB entity to model views
var model = _mapper.Map<AdViewModel>(ad);
// Note: view will have to detect the exact ad instance type and show/hide corresponding UI fields
return View(model);
}
[HttpGet("Ad/{type}/Add/")]
public IActionResult Add(AdType type)
{
var ad = // ... switch by type to validate/add new entity
return View(_mapper.Map<AdEditModel>(ad));
}
[HttpPost("Ad/{type}/Add/")]
public async Task<IActionResult> Add(AdEditModel model)
{
// detect ad type and save
return View(model);
}
[HttpGet("Ad/{type}/Edit/{id}")]
public async Task<IActionResult> Edit(AdType type, int id)
{
// similar to Add
return View(model);
}
[HttpPost("Ad/{type}/Edit/{id}")]
public async Task<IActionResult> Edit(AdEditModel model)
{
// similar to Add
return View(model);
}
// And so on
}
}
但我应该注意,UI 相关代码中的继承最终会导致更多的问题而不是好处。代码变得更加复杂,难以维护并保持干净。因此,将所有 Views
和 Controllers
分开更有意义,即使它们的代码非常接近。您可以开始优化 DI 服务(又名业务逻辑
)或类似层下的“重复代码”使用。
UI级别的重复代码
问题应该通过提取组件来解决(又名控件
、部分 View
、 View 组件
)。 Controller 继承是可能的,但会使代码更难维护。
关于asp.net-mvc - 利用 Controller 和 View 中的继承,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/52284912/
我想做的是让 JTextPane 在 JPanel 中占用尽可能多的空间。对于我使用的 UpdateInfoPanel: public class UpdateInfoPanel extends JP
我在 JPanel 中有一个 JTextArea,我想将其与 JScrollPane 一起使用。我正在使用 GridBagLayout。当我运行它时,框架似乎为 JScrollPane 腾出了空间,但
我想在 xcode 中实现以下功能。 我有一个 View Controller 。在这个 UIViewController 中,我有一个 UITabBar。它们下面是一个 UIView。将 UITab
有谁知道Firebird 2.5有没有类似于SQL中“STUFF”函数的功能? 我有一个包含父用户记录的表,另一个表包含与父相关的子用户记录。我希望能够提取用户拥有的“ROLES”的逗号分隔字符串,而
我想使用 JSON 作为 mirth channel 的输入和输出,例如详细信息保存在数据库中或创建 HL7 消息。 简而言之,输入为 JSON 解析它并输出为任何格式。 最佳答案 var objec
通常我会使用 R 并执行 merge.by,但这个文件似乎太大了,部门中的任何一台计算机都无法处理它! (任何从事遗传学工作的人的附加信息)本质上,插补似乎删除了 snp ID 的 rs 数字,我只剩
我有一个以前可能被问过的问题,但我很难找到正确的描述。我希望有人能帮助我。 在下面的代码中,我设置了varprice,我想添加javascript变量accu_id以通过rails在我的数据库中查找记
我有一个简单的 SVG 文件,在 Firefox 中可以正常查看 - 它的一些包装文本使用 foreignObject 包含一些 HTML - 文本包装在 div 中:
所以我正在为学校编写一个 Ruby 程序,如果某个值是 1 或 3,则将 bool 值更改为 true,如果是 0 或 2,则更改为 false。由于我有 Java 背景,所以我认为这段代码应该有效:
我做了什么: 我在这些账户之间创建了 VPC 对等连接 互联网网关也连接到每个 VPC 还配置了路由表(以允许来自双方的流量) 情况1: 当这两个 VPC 在同一个账户中时,我成功测试了从另一个 La
我有一个名为 contacts 的表: user_id contact_id 10294 10295 10294 10293 10293 10294 102
我正在使用 Magento 中的新模板。为避免重复代码,我想为每个产品预览使用相同的子模板。 特别是我做了这样一个展示: $products = Mage::getModel('catalog/pro
“for”是否总是检查协议(protocol)中定义的每个函数中第一个参数的类型? 编辑(改写): 当协议(protocol)方法只有一个参数时,根据该单个参数的类型(直接或任意)找到实现。当协议(p
我想从我的 PHP 代码中调用 JavaScript 函数。我通过使用以下方法实现了这一点: echo ' drawChart($id); '; 这工作正常,但我想从我的 PHP 代码中获取数据,我使
这个问题已经有答案了: Event binding on dynamically created elements? (23 个回答) 已关闭 5 年前。 我有一个动态表单,我想在其中附加一些其他 h
我正在尝试找到一种解决方案,以在 componentDidMount 中的映射项上使用 setState。 我正在使用 GraphQL连同 Gatsby返回许多 data 项目,但要求在特定的 pat
我在 ScrollView 中有一个 View 。只要用户按住该 View ,我想每 80 毫秒调用一次方法。这是我已经实现的: final Runnable vibrate = new Runnab
我用 jni 开发了一个 android 应用程序。我在 GetStringUTFChars 的 dvmDecodeIndirectRef 中得到了一个 dvmabort。我只中止了一次。 为什么会这
当我到达我的 Activity 时,我调用 FragmentPagerAdapter 来处理我的不同选项卡。在我的一个选项卡中,我想显示一个 RecyclerView,但他从未出现过,有了断点,我看到
当我按下 Activity 中的按钮时,会弹出一个 DialogFragment。在对话框 fragment 中,有一个看起来像普通 ListView 的 RecyclerView。 我想要的行为是当
我是一名优秀的程序员,十分优秀!