使用组合存在“is-a”关系的问题

Question

我正在开发一个人力资源系统，其中有会计员工和程序员员工。加入公司的第一个月，员工不被赋予任何角色。一个员工可以同时是会计师和程序员。以下代码展示了设计。

现在，我需要通过实现新功能来增强系统：

终止所有会计师（终止意味着将员工状态设置为IsActive = false）。问题在于，我不能直接将所有会计师设置为非活动状态而不进行检查。我需要检查他是否拥有其他角色。

如何重建这些类以更自然地使用OO来执行终止函数？

更新

我正在寻找一个具有EF数据库优先解决方案模型和数据库架构的答案，以响应@AlexDev的回答。

C＃代码

List<Accountant> allAccountants =  Get All accountants from database

public class Employee
{
    public int EmpID { get; set; }
    public DateTime JoinedDate { get; set; }
    public int Salary { get; set; }
    public bool IsActive { get; set; }
}


public class Accountant : Employee
{
    public Employee EmployeeData { get; set; }
}

public class Programmer : Employee
{
    public Employee EmployeeData { get; set; }
}

@AlexDev的回答

请提供需要翻译的具体内容。

public class Employee
{
...
IList<Role> Roles;
bool isActive;

public void TerminateRole(Role role)
{
    Roles.Remove(role);
    if(Roles.Count == 0)
    {
        isActive = false;
    }
}
}

public class Role
{
 abstract string Name { get;}
}

public class ProgrammerRole : Role
{
 override string Name { get { return "Programmer"; } }
}

参考资料

1. DDD方法访问外部信息
2. 优先使用组合而非继承？
3. 领域模型中的继承与枚举属性
4. Entity Framework：在存储库中获取子类对象

- LCJ

---

终止（isActive）的概念是针对员工而不是角色的。考虑一个拥有两个角色的员工。尽管所有会计师都被终止，他仍将继续成为一名活跃的员工。 - LCJ

首要任务应该是确保在 SQL 层面（表设计）上做得正确。之后，我会尝试将数据库中的内容建模为对象。但是，不要过度使用层次结构树。OOP 和 SQL 之间存在阻抗不匹配。问题在于 SQL 在很大程度上基于集合和数学。OOP 在很大程度上是炒作；它的基本原则相互冲突，正确设计一个矩形类需要花费很长时间 :）。每当 SQL 和 OOP 之间存在冲突时，让 OOP 弯曲以适应 SQL，而不是反过来。我会使用存储过程和静态函数。 - Hamish Grubijan

参考资料：http://programmers.stackexchange.com/questions/194560/a-design-decision-in-composition-or-aggregation?rq=1 - LCJ

Answer 1

如果您正在使用的结构需要某人既是会计师又是程序员来实现，那么您需要多重继承。此外，该系统可能会添加新角色，而C#中不存在这种情况。您应该考虑采用不同的设计。一种可能的方案：

public class Employee
{
    ...
    IList<Role> Roles;
    bool isActive;

    public void TerminateRole(Role role)
    {
        Roles.Remove(role);
        if(Roles.Count == 0)
        {
            isActive = false;
        }
    }
}

public class Role
{
    abstract string Name { get;}
}

public class ProgrammerRole : Role
{
    override string Name { get { return "Programmer"; } }
}

那么你可以为每种类型都继承Role类，并可以决定终止一个或所有角色。

- AlexDev

---

当一个人拥有两个角色时被终止，应该发生什么？ - AlexDev

编辑。另一个选项是将isActive作为只读属性，如bool isActive { get { return Roles.Count > 0; } }，但这不会映射到数据库中。 - AlexDev

谢谢。更新后的答案很有道理。使用EF Database First是否可以实现这个模型？它会是什么样子？ - LCJ

请问您能告诉我NHibernate中模型的样子吗？ - LCJ

@Lijo 我在一个新答案中发布了它。 - AlexDev

Answer 2

根据您在问题中添加的模式，我假设您不会对Role进行子类化，因此我正在撰写新答案。另外，如果您使用NHibernate，请不要忘记使用public virtual属性。

public class Employee
{
    ...
    public virtual IList<Role> Roles { get; set; }
    public virtual bool isActive { get; set; }

    public virtual void TerminateRole(Role role)
    {
        Roles.Remove(role);
        if(Roles.Count == 0)
        {
            isActive = false;
        }
    }
}

public class Role
{
    public virtual int RoleID { get; set; }
    public virtual string Name { get; set; } 
}

并且有映射：

public class EmployeeMap : ClassMap<Employee>
{
    public EmployeeMap()
    {
        Id(x => x.EmpId);
        Map(x => x.JoinedDate)
        Map(x => x.Salary);
        Map(x => x.IsActive);
        HasManyToMany(x => x.Roles).Cascade.AllDeleteOrphan().Table("EmployeeRole")
    }
}

public class RoleMap : ClassMap<Role>
{
    public RoleMap()
    {
        Id(x => x.RoleID);
        Map(x => x.RoleName);
    }
}

- AlexDev

Answer 3

public abstract class AbstractEmployee
{
    ...
    public abstract bool IsActiveAccountant { get; set; }
    public abstract bool IsActiveProgrammer { get; set; }
    public bool IsActive() { get { return bitwise or of all roles; } }
}

public class NewEmployee : AbstractEmployee
{
    ...
    public override bool IsActiveAccountant { get; set; }
    public override bool IsActiveProgrammer { get; set; }
}

public class Programmer : AbstractEmployee
{
    ...
    public override bool IsActiveAccountant { get; set; }
    public override bool IsActiveProgrammer { get; set; }
}

缺点：

• 每次添加新的系统级别角色时，您都必须修改类

优点：

• 您不需要寻找会计师
• 程序员可以拥有IsEmptyAccountant的空实现，因为这种角色对他们来说是无效的
• NewEmployee可以同时拥有多个角色

如果引入新角色的开销很大，我建议继续搜索。

- Roman Saveljev

---

这并不模拟真实的场景。我不喜欢在程序员类中使用IsActiveAccountant。 - LCJ

问题在于我无法直接将所有会计师设置为非活动状态而不进行检查 - 在现实世界中您也会进行检查，不是吗？ :) 我只是试图解决您需要检查的问题（尽管我个人认为这不是问题）。在对象上有限的角色列表中删除其中一个角色意味着需要进行检查。如果您的实际问题不是关于消除检查的需求，请原谅。 - Roman Saveljev

Answer 4

从我的回答中优先使用组合而非继承？开始

我首先会进行检查 - 是否存在“is-a”关系。如果存在，我通常会检查以下内容：

基类是否可以实例化。也就是说，基类是否可以是非抽象的。如果它可以是非抽象的，我通常会选择组合。

例如 1. 会计师是员工。但我不会使用继承，因为可以实例化一个员工对象。

例如 2. 书是销售物品。销售物品无法实例化 - 它是一个抽象概念。因此，我将使用继承。销售物品是一个抽象基类（或在 C# 中是接口）

- LCJ