我有一个公司实体
public class Company : Entity<Company>
{
public CompanyIdentifier Id { get; private set; }
public string Name { get; private set; }
..............
..........
}
公司可以是代理商或供应商,也可以是两者兼而有之。 (有更多类型)其行为应根据类型进行更改。代理商可以获得佣金,供应商可以开票。
设计一个或多个实体或值对象的最佳方法是什么?我可以选择添加一些布尔类型并在方法中检查这些值,
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
}
..........
}
老实说,我不喜欢这样。是否有任何设计模式可以帮助克服这种局面?您将做什么,为什么要设计此场景呢?
最佳答案
显式实现接口,然后覆盖强制转换运算符以仅在有效时强制转换为该接口。
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;
}
}
接口的显式实现必须通过其接口而不是具体类型进行调用:
// Will not compile
new Company().GetCommission(5);
// Will compile
((IAgentCompany)new Company()).GetCommission(5)
但是,现在我们已经重载了显式强制转换运算符。那是什么意思呢?我们必须在不强制转换为IAgentCompany的情况下才能致电GetCommission,现在我们有了防范措施,可以防止未标记为代理的公司强制转换。
关于这种方法的好处:
1)您具有定义不同类型公司的方面及其功能的界面。界面隔离是一件好事,并且可以使每种类型的公司的能力/职责明确。
2)您已经取消了对所有公司都不是“全局”的每个函数的检查。强制转换时进行一次检查,然后只要将其包含在作为接口类型的变量中,就可以愉快地与它进行交互,而无需任何进一步的检查。这意味着减少了引入错误的地方,并且减少了无用的检查。
3)您正在利用语言功能,并利用类型系统来帮助使代码更加防弹。
4)您不必在代码中的任何地方编写大量实现
NotImplementedExceptions
或InvalidOperationException
接口的各种组合的子类(可能是2 ^ n个子类!)。5)您不必使用枚举或“类型”字段,尤其是当您要求混合和匹配这些功能集时(您不仅需要枚举,还需要标记枚举)。使用类型系统表示不同的类型和行为,而不是枚举。
6)干。
这种方法的坏事:
1)显式的接口实现和覆盖显式的强制转换运算符并不完全是C#的基本知识,并且可能会使那些追随您的人感到困惑。
编辑:
好吧,我回答得太快而没有测试这个想法,这对接口不起作用。但是,请参阅我的其他答案以获取其他想法。