java – 业务规则 – 它们在OOP中的位置？
|
我有一节课：时间表.
public class Schedule {
private int locationNum;
private int cost;
private String costReason;
private Date weekOfChange;
private Date dayOfChange;
private String changeReason;
// and all those getters and setters
public Schedule(int locationNum,int cost,String costReason,Date weekOfChange,Date dayOfChange,String changeReason) throws ApplicationException {
//change is all or nothing - all attributes are present or none
if((weekOfChange!=null && dayOfChange!=null && changeReason!=null) || (weekOfChange==null && dayOfChange == null && changeReason == null)) {
this.weekOfChange = weekOfChange;
this.dayOfChange = dayOfChange;
this.changeReason = changeReason;
}
else { throw new ApplicationException();}
//similary another if block to ensure that if cost is specified
//then there exists the corresponding reason code for it.
}
}
到目前为止,我喜欢我的Schedule课程.但是,我没有完成检查,我将不得不做一些其他检查： >是locationNum数据库中的有效商店编号. 通常情况下,我不会在Schedule类中写这些,显然,我不能从这个类中调用DAO.所以,我会有一个业务层和某种验证器类,接受一个类型为Schedule的对象,并按顺序执行一堆数据库验证,并收集显示/任何错误. 现在,这是我的问题： >如果您将Schedule视为POJO并认为对象不负责验证自身 – 我必须将构造函数中的所有代码移动到业务层的验证器类.但是,如果我这样做,是不是安排贫血？这是他们所谓的违反单一责任原则吗？ 感谢您参与此讨论. 解决方法
如果你真的拥有“所有那些getter和setter”,那么你需要比构造函数更好地验证你的类.如果你的类的不变量是所有weekOfChange,dayOfChange和changeReason必须为null或者不是所有都必须不为null,那么你的setter很快就会把你的类置于无效状态.你的意思是拥有制定者还是你的意思是你的阶级是不可改变的？
在担心验证之前,应该对你的班级进行分析.查看给定状态的所有有效状态和不变量.然后,您将了解您的类是否应该是可变的或不可变的.如果你有相互依赖的共变量(比如weekOfChange,dayOfChannge和changeReason),将它们打包到自己的类中并使用Schedule类中的组合是有意义的.这会将关于这些事物字段的“规则”放在一个地方并简化时间表. 与其他合作领域相同(如成本和成本原因).然后,Schedule由自我验证类组成.如果这两个都是不可变的,并且Schedule也是不可变的,那么你的自我就更容易了. 所以,回答你的问题：一个类定义它的状态和不变量更好,并且在可行的情况下只向其合作者公开最小值.附表的内部状态的责任应该依赖于附表,并且只需稍加设计就可以相对容易地进行. 所以你有了 class Schedule {
private Change change;
private Cost cost;
private Location location;
public Schedule(Location location,Cost cost,Change change) {
this.change = change;
this.cost = cost;
this.location = location;
}
}
和colloborators一样 public class Change {
private Date weekOfChange; //shoudln't these two fields be one,a date is a date after all??
private Date dayOfChange; //shoudln't these two fields be one??
private String changeReason;
public Change(Date weekOfChange Date dayOfChange,String changeReason) {
this.weekOfChange = weekOfChange;
...etc.
}
}
虽然我强烈建议您通过防御性复制客户端代码传递的任何可变值来保护您的类不变量. （编辑：李大同） 【声明】本站内容均来自网络，其相关言论仅代表作者个人观点，不代表本站立场。若无意侵犯到您的权利，请及时与联系站长删除相关内容! |