每次我创建一个具有集合属性的对象时,我都会以最好的方式来回去做?
带有getter的公共属性,它返回对私有变量的引用
显式get_ObjList和set_ObjList方法,每次返回并创建新的或克隆的对象
显式get_ObjList返回一个IEnumerator和一个带有IEnumerator的set_ObjList
如果集合是一个数组(即objList.Clone())而不是List,它会有所不同吗?
如果返回实际集合作为引用是如此糟糕,因为它创建依赖项,那么为什么返回任何属性作为引用?每当您将子对象作为引用公开时,除非父级具有属性更改事件,否则可以在父级"知道"的情况下更改该子对象的内部.是否有内存泄漏的风险?
并且,不要选项2和3打破序列化?这是一个catch 22还是你必须在有集合属性的时候实现自定义序列化?
通用的ReadOnlyCollection似乎是一般用途的一个很好的折衷方案.它包装IList并限制对它的访问.也许这有助于内存泄漏和序列化.然而,它仍然有枚举问题
也许它只是取决于.如果您不关心集合是否被修改,那么只需将它作为公共访问器公开在每个#1的私有变量上.如果您不希望其他程序修改集合,那么#2和/或#3会更好.
问题隐含的是为什么一种方法应该用于另一种方法,以及对安全性,内存,序列化等的影响是什么?
如何公开集合完全取决于用户与其进行交互的方式.
1)如果用户将添加和删除对象集合中的项目,那么最简单的只获取集合属性(原始问题中的选项#1):
private readonly CollectionmyCollection_ = new ...; public Collection MyCollection { get { return this.myCollection_; } }
此策略用于Items
WindowsForms和WPF ItemsControl
控件上的集合,用户可以在其中添加和删除希望控件显示的项目.这些控件发布实际的集合并使用回调或事件侦听器来跟踪项目.
WPF还公开了一些可设置的集合,以允许用户显示他们控制的项目集合,例如ItemsSource
属性ItemsControl
(原始问题中的选项#3).但是,这不是一个常见的用例.
2)如果用户只读取对象维护的数据,那么您可以使用只读集合,如Quibblesome建议:
private readonly ListmyPrivateCollection_ = new ...; private ReadOnlyCollection myPrivateCollectionView_; public ReadOnlyCollection MyCollection { get { if( this.myPrivateCollectionView_ == null ) { /* lazily initialize view */ } return this.myPrivateCollectionView_; } }
请注意,它ReadOnlyCollection
提供了基础集合的实时视图,因此您只需创建一次视图.
如果内部集合未实现IList
,或者您希望限制对更高级用户的访问,则可以通过枚举器包装对集合的访问:
public IEnumerableMyCollection { get { foreach( T item in this.myPrivateCollection_ ) yield return item; } }
这种方法易于实现,并且可以在不暴露内部集合的情况下提供对所有成员的访问.但是,它确实要求集合保持未修改,因为如果您尝试在修改集合后枚举集合,则BCL集合类将抛出异常.如果底层集合可能会更改,您可以创建一个可以安全枚举集合的光包装器,也可以返回该集合的副本.
3)最后,如果您需要公开数组而不是更高级别的集合,那么您应该返回数组的副本以防止用户修改它(原始问题中的选项#2):
private T[] myArray_; public T[] GetMyArray( ) { T[] copy = new T[this.myArray_.Length]; this.myArray_.CopyTo( copy, 0 ); return copy; // Note: if you are using LINQ, calling the 'ToArray( )' // extension method will create a copy for you. }
您不应该通过属性公开底层数组,因为您无法判断用户何时修改它.要允许修改数组,您可以添加相应的SetMyArray( T[] array )
方法,也可以使用自定义索引器:
public T this[int index] { get { return this.myArray_[index]; } set { // TODO: validate new value; raise change event; etc. this.myArray_[index] = value; } }
(当然,通过实现自定义索引器,您将复制BCL类的工作:)