我有一个公司实体

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)您不必在代码中的任何地方编写大量实现NotImplementedExceptionsInvalidOperationException接口的各种组合的子类(可能是2 ^ n个子类!)。

5)您不必使用枚举或“类型”字段,尤其是当您要求混合和匹配这些功能集时(您不仅需要枚举,还需要标记枚举)。使用类型系统表示不同的类型和行为,而不是枚举。

6)干。

这种方法的坏事:

1)显式的接口实现和覆盖显式的强制转换运算符并不完全是C#的基本知识,并且可能会使那些追随您的人感到困惑。

编辑:

好吧,我回答得太快而没有测试这个想法,这对接口不起作用。但是,请参阅我的其他答案以获取其他想法。

08-05 22:54
查看更多