我有一个公司实体
public class Company : Entity<Company>
{
public CompanyIdentifier Id { get; private set; }
public string Name { get; private set; }
..............
..........
}
公司可以是代理商或供应商,也可以是两者或都不是。 (有更多的类型)它的行为应该根据类型而改变。代理商可以获得佣金,供应商可以开具发票。 设计一个或多个实体或值(value)对象的最佳方式是什么?我可以选择添加一些 bool 类型并在方法中检查这些值,
public class Company : Entity<Company>
{
public CompanyIdentifier Id { get; private set; }
public string Name { get; private set; }
public bool IsAgent { get; private set; }
public bool IsSupplier { get; private set; }
..........
public void Invoice()
{
if(!IsSupplier)
{
throw exception.....;
}
//do something
}
public void GetCommission(int month)
{
if(!IsAgent)
{
throw exception.....;
}
//do something
}
..........
}
老实说,我不喜欢这样。是否有任何设计模式可以帮助克服这种情况?你会做什么,为什么要设计这个场景?
最佳答案
显式实现接口(interface),然后覆盖强制转换运算符以仅在有效时强制转换为该接口(interface)。
public class Company : ...., IAgentCompany, ISupplierCompany ... {
public double IAgentCompany.GetCommission(int month) {
/*do stuff */
}
public static explicit operator IAgentCompany(Company c) {
if(!c.IsAgent)
throw new InvalidOperationException();
return this;
}
}
接口(interface)的显式实现必须通过它们的接口(interface)调用,而不是具体类型:
// Will not compile
new Company().GetCommission(5);
// Will compile
((IAgentCompany)new Company()).GetCommission(5)
但是,现在我们已经重载了显式转换运算符。那是什么意思呢?我们不能在不强制转换为 IAgentCompany 的情况下调用 GetCommission,现在我们有一个保护措施来防止未标记为代理的公司进行强制转换。
这种方法的优点:
1) 您拥有定义不同类型公司的各个方面以及它们可以做什么的接口(interface)。接口(interface)隔离是个好东西,可以明确各类公司的能力/职责。
2) 您已经取消了对您要调用的不是所有公司都“全局”的每个函数的检查。你在转换时做一次检查,然后只要你把它放在一个类型为接口(interface)的变量中,你就可以愉快地与它交互而无需任何进一步检查。这意味着引入错误的地方更少,无用的检查也更少。
3) 您正在利用语言特性,并利用类型系统来帮助使代码更安全。
4) 你不必编写大量的子类来实现接口(interface)的各种组合(可能是 2^n 个子类!)你的代码。
5) 您不必使用枚举或“类型”字段,尤其是当您要求混合和匹配这些能力集时(您不仅需要枚举,还需要标志枚举).使用类型系统来表示不同的类型和行为,而不是枚举。
6) 它是干的。
这种方法的缺点:
1) 显式接口(interface)实现和重写显式强制转换运算符并不是 C# 编码知识的基础,可能会让后来者感到困惑。
编辑:
好吧,我没有测试这个想法就回答得太快了,这不适用于接口(interface)。但是,请参阅我的其他答案以获得另一个想法。
关于c# - 根据类型添加行为的最佳方式,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/25269207/