Question

Basically, I was always in the understanding that you should return the expose base types whenever you can and worry about implementation details internally, which makes sense…

But, I’m not sure what to do here. Basically, right now I have:

ReadOnlyObservableCollection<Foo> MyFoos {get; private set; }

I’m wondering if that should be returned as a ReadOnlyCollection<Foo> or an ICollection<Foo> because internally I never really use any observable parts or attempt to write to the collection. WPF seems to not care what I return, it still binds it and triggers the collection changed notification event properly. But, I read somewhere that I should design this to really have any consuming view handle my ViewModel.

So I’m a bit torn here. I’m thinking that leaving it as a ReadOnlyObservableCollection<T> makes the most sense to explicitly tell the consuming view what they can and can not do with the property, but I’m also under the impression that you should reduce down types to their base types when you can. So I’m not sure what to do here. Especially with the fact that WPF doesn’t care what type I return, it figures out that it’s observable.

Answer 1

I would probably leave it as ReadOnlyObservableCollection because that very specifically states what a consumer of your ViewModel is allowed to do with your collection. Also note that WPF doesn’t actually bind directly to your collection, it binds to the return value of CollectionViewSource.GetDefaultView, which returns an ICollectionView. ICollectionView has INotifyCollectionChanged in its contract.