情况:我的公司购买了一个大型收缩包装应用程序。它应该是可扩展的,yada,yada。它具有SQL和DLL形式的DB,DAL和BLL。它还有一个MVC项目(可扩展部分),但95%的“模型”部分都在DAL / BLL库中。
问题:我需要扩展位于BLL中的一个“模型”。它是一个User对象,有47个属性,0个方法,没有构造函数。我开始的是他们班级的简单妄想:
public class ExtendedUser : BLL.DTO.User
{
public bool IsSeller { get; set; }
public bool IsAdmin { get; set; }
}
如果我只创建一个新的ExtendedUser,这可以正常工作。然而,它被另一个调用BLL填充,如:
BLL.DTO.User targetUser = UserClient.GetUserByID(User.Identity.Name, id);
我尝试了直接的强力尝试,这当然会抛出一个施法异常:
ExtendedUser targetUser = (ExtendedUser)UserClient.GetUserByID(User.Identity.Name, id);
我在这个非常简单的OO概念上画了一个完整的空白。我不想编写一个接受现有User对象的构造函数,然后将每个属性复制到我的扩展对象中。我知道有一个正确的方法来做到这一点。有人可以打我的头,并告诉我显而易见的事情吗?
TIA
答案 0 :(得分:6)
如果您确实想使用继承,那么使用47个属性,像Automapper这样的东西可能会帮助您复制所有值 - http://automapper.codeplex.com/ - 这将允许您使用:
// setup
Mapper.CreateMap<BLL.DTO.User, ExtendedUser>();
// use
ExtendedUser extended = Mapper.Map<BLL.DTO.User, ExtendedUser>(user);
或者,您可能最好使用聚合而不是继承 - 例如
public class AggregatedUser
{
public bool IsSeller { get; set; }
public bool IsAdmin { get; set; }
public BLL.DTO.User User { get; set; }
}
答案 1 :(得分:4)
这种方法怎么样(基本上是聚合):
public sealed class ExtendedUser
{
public ExtendedUser(BLL.DTO.User legacyUser)
{
this.LegacyUser = legacyUser;
}
public BLL.DTO.User LegacyUser
{
get;
private set;
}
}
答案 2 :(得分:2)
我不想编写一个接受现有User对象的构造函数,然后将每个属性复制到我的扩展对象中。
这通常是执行此操作的“正确”方式,除非您具有对BLL的编译时访问权限。问题是演员阵容永远不会奏效 - ExtendedUser
是User
的具体类型,但每个User
都不是 ExtendedUser
,这将是演员成功所必需的。
您可以通过聚合处理此问题(包含User
的实例作为成员),但不能直接通过继承处理。
这通常在编译时通过Partial Classes处理。如果BLL设置为创建类(即:User
)作为分部类,则可以将自己的逻辑添加到单独的文件中,从而防止出现问题。这是许多大型框架,ORM等的常见做法。