C#：将 BusinessObject 传递给“BusinessLayer”构造函数或其方法？

Question

我负责支持 C# Winforms 应用程序，该应用程序使用 BusinessObjects（不包含逻辑，仅包含属性）和带有操作这些实体的类（“帮助程序”）的 BusinessLayer。

问题： 如果您将 BusinessObject 传递给 Helpers 构造函数，然后在构造函数内，实例化 Helper 的可公开访问的 Entity 变量 或者 您是否应该将实体传递给对其进行操作的方法？

场景 1：对于构造函数

Car myCar = new Car();
CarHelper ch = new CarHelper(myCar);
ch.Wash(suds);
ch.Upgrade(upgradeKit);
ch.Save();

场景 2：对于作用于实体的方法

Car myCar = new Car();
CarHelper ch = new CarHelper();
ch.Wash(myCar, suds);
ch.Upgrade(myCar, upgradeKit);
ch.Save(myCar);

场景 1 存在两个主要问题： A) 下一个开发人员必须深入研究 CarHelper 类，以认识到它有一个公共 Car 访问器属性，它在需要它的方法中引用该属性。这进一步混淆了 Helper 类，因为每个方法在执行其职责之前都需要检查“null”Car 属性...... B) 如果在操作之间存在一堆其他代码，则可能会不清楚 ch.Wash() 实际上在做什么......它甚至对 Car 对象起作用吗......？

大家觉得怎么样？？？

Answer 1

是否有任何原因导致您无法将逻辑移至 BusinessObject 中

Car myCar = new Car();
myCar.Wash(suds);
myCar.Upgrade(upgradeKit);
myCar.Save();

完全取消帮助程序类。阅读起来更具语义意义，并​​且无需检查空值。

需要维护的类数量也减少了一半

Answer 2

这是非常正确的...在我看来，将 BusinessObject 中的逻辑包装起来以使其具有自我意识也是最好的...但我不认为这是一个选项，因为：

在“应用程序”中，BusinessObjects 位于命名空间中(....ApplicationServices) 由 DAO 引用，因此它实际上无法调用 DAO 方法（因为这会导致循环依赖） - 因此它无法实现该功能 这

myCar.Wash(suds)
{
    this.CleanlinessRating = suds.CleaningAbilityRating;    
    // persist the level of Cleanliness to the DB
    CarDAO.Save(this);
}

似乎是整个应用程序背后的前提是 BusinessObjects 没有实现根本没有任何逻辑......它们只是信息的容器并且没有任何行为。

然后你有作用于实体的 BusinessLayer 类...

然后你有将实体中的更改保存到数据库的 DataLayer 类。

所以显然，让实体自我意识并实现自己的行为是一个很大的“不不”（在这个应用程序中）......我确信这是这里真正的问题。

但是，假设我无法改变这一点，你会怎么做？

1. 将实体传递给对其进行操作的方法？
   或者
2. 将实体包装到 Helper 类的构造函数中？

Answer 3

让您的 CarHelper 类扩展 Car

CarHelper helper = new Car();
helper.Wash(suds);
helper.Upgrade(upgradeKit);
helper.Save();

两全其美怎么样？