我有一个大约有20个属性的类,但我会为这个问题简化它:
public class Product
{
public int Id { get; set; }
public string Name { get; set; }
}
我希望有一个类或属性来识别我的班级是否脏。我的意思是确定它的任何值是否已经改变了?
我可以采取3种设计方法:
1) 在类中设置属性时,我会检查属性是否为IsDirty。
public string Name
{
get { return this._name; }
set { if (this._name != value) { this.IsDirty = true; this._name = value; }
}
2) 从类外部设置属性时,我会检查属性是否为IsDirty。
e.g。
if (p.Name != newName)
{
p.IsDirty = true;
p.Name = newName;
}
这种方法迫使我在客户端类中添加大量ifs。有些属性甚至是集合甚至是引用对象,因此行数甚至会增加。
3) 当对象准备好保存时,我通过获取克隆对象并检查相等性来检查是否有任何属性IsDirty。
这会有较差的性能,因为我必须克隆或再次加载原始对象,然后逐个比较属性。
哪一个是最好的设计?或者是否有任何其他设计模式可以帮助解决这个问题?
答案 0 :(得分:5)
另一种选择是实现INotifyPropertyChanged接口。
请注意,这将有助于您使事情更整洁,并且您的API更清晰,但就变更后跟踪的内部实施而言,仍由您来实施。我认为这最适合您的Option #1
答案 1 :(得分:1)
选项1显然是最好的:它将跟踪肮脏的责任放在它所属的位置:在对象内部。选项2已经出局,因为正如您所提到的,您正在将此责任强加给您的课程的客户。如上所述,选项3还有其他问题。
顺便说一下,您应该使用DynamicProxy查看代理实现。这将使您的代码看起来像这样:
public class Product
{
public virtual int Id { get; set; }
public virtual string Name { get; set; }
}
通过明智地使用拦截器,您可以获得所需的行为。你可以编写一个拦截器来拦截任何“集合”并在里面做一些逻辑,比如设置一个IsDirty标志。
答案 2 :(得分:0)
另一个想法是将其设为GoF Observable
,让感兴趣的Observer
方注册他们对变更的兴趣。这是一种更基于事件的方法。
答案 3 :(得分:0)
这是最好的解决方案,非常好地符合SRP原则,我创建了以下类:
ProductWithChangeDetection;这使用Decorator模式将此新功能添加到现有产品对象
ProductChangeDetector;这包含检查和通知的逻辑。目前只暴露ChangeDetected属性,但如果需要更多复杂性,则应实现INotifyPropertyChange接口。
ProductEquitable;这实现了IEquitable并且有一些重载用于检查两个对象/属性是否相等