Czy istnieje dobry wzór do ujawnienia kolekcji ogólnej jako readonly?

Question

Mam więc te klasy, które eksponują kolekcję obiektów podrzędnych.

Nie chcę, aby inne klasy dodawały lub usuwały obiekty z kolekcji, ponieważ muszę je łączyć w zdarzenia w obiektach podrzędnych, tak aby były dodawane lub usuwane. Chcę móc wykonać dodatkowe przetwarzanie. Ale uwielbiam łatwość manipulowania generykami wewnętrznie.

Czy wspomniałem, że jest to aplikacja WPF, więc potrzebuję INotifySupport?

Najlepsze, co mogę wymyślić, to coś takiego.

public class foo : INotifyPropertyChanged 
{ 
    protected List<ChildFoo> _Children = new List<ChildFoo>(); 

    public foo() 
    { 
    } 

    public void AddChild(ChildFoo newChild) 
    { 
     DoAttachLogic(newChild); 
     _Children.Add(newChild); 
     NotifyPropertyChange("Children"); 
    } 

    public void RemoveChild(ChildFoo oldChild) 
    { 
     DoRemoveLogic(oldChild); 
     _Children.Remove(oldChild); 
     NotifyPropertyChange("Children"); 
    } 

    public ChildFoo[] Children 
    { 
     get 
     { 
      return _Children.ToArray(); 
     } 
    } 

} 

Czy istnieją poważne wady tego wzoru, których nie widzę?

Za każdym razem, gdy uzyskuje się dostęp do właściwości Children, otrzymujemy obciążenie związane z przekształcaniem listy w tablicę.

Wszelkie porady na ten temat byłyby świetne.

Answer 1

Powinieneś używać ObservableCollection jako pola w swojej klasie, wtedy masz pełny dostęp do modyfikacji kolekcji. Następnie wyeksponuj to jako właściwość ReadonlyObservableCollection. A jeśli nie zmienisz samej kolekcji (np. Nr children = new ObservableCollection(), powinieneś zrobić pole tylko do odczytu), to nie potrzebujesz żadnego rodzaju obiektu notifyPropertyChanged na tej właściwości, ponieważ nie zmienia się i kolekcja sama zajmuje się tymi zdarzeniami dla swoich dzieci.

public class Child 
{ 
    public int Value { get; set; } 
} 

class MyClassWithReadonlyCollection 
{ 
    private readonly ObservableCollection<Child> _children = new ObservableCollection<Child>(); 

    public MyClassWithReadonlyCollection() 
    { 
     _children.Add(new Child()); 
    } 

    //No need to NotifyPropertyChange, because property doesnt change and collection handles this internaly 
    public ReadOnlyObservableCollection<Child> Children { get { return new ReadOnlyObservableCollection<Child>(_children); } } 
} 

Answer 2

To jest to, co robię dla normalnego kodu:

Public Readonly Property Childern As ObjectModel.ReadOnlyCollection(Of Child) 
    Get 
     Return New ObjectModel.ReadOnlyCollection(Of Child)(_ChildernList) 
    End Get 
End Property 

Dla kodu WPF Chciałbym tylko narazić podklasy ObservableCollection.

Answer 3

Zmieniłem "dodaj dziecko" i "usuń dziecko" na chronione, ponieważ mówisz, że nie chcesz innych zajęć modyfikujących swoją kolekcję. Zmieniłem twoją listę na ObservableCollection, abyś mógł otrzymywać powiadomienia o zmianie kolekcji. Ponieważ używasz IList, nie ma potrzeby wywoływania ToArray(), po prostu dostęp bezpośrednio.

spróbuj tego:

public class foo : INotifyPropertyChanged 
{ 
    protected ObservableCollection<ChildFoo> _Children = new ObservableCollection<ChildFoo>(); 

public foo() { } 

protected void AddChild(ChildFoo oldChild) 
{ 
    DoAttachLogic(newChild); 
    _Children.Add(newChild); 
    NotifyPropertyChange("Children"); 
} 

protected void RemoveChild(ChildFoo oldChild) 
{ 
    DoRemoveLogic(oldChild); 
    _Children.Remove(oldChild); 
    NotifyPropertyChange("Children"); 
} 

public ChildFoo this[int n] 
{ 
    get 
    { 
     return _Children[n]; 
    } 
} 

} 

Answer 4

Można podklasy BindingList i ustawić AllowNew/AllowRemove false. W swoich metodach dodawania/usuwania dla dzieci można ustawić wartość true, wprowadzić zmiany, a następnie ustawić wartość false. (Oczywiście, musisz również ukryć dostęp do ustawień AllowNew/AllowRemove od osób dzwoniących z zewnątrz).

Inna opcja - podklasa Obserwowalna kolekcja i nadpisywanie metod InsertItem, RemoveItem, itp., Aby zachowywać się jak AddChild/RemoveChild zachowałaby się. Następnie osoby dzwoniące mogą nadal uzyskiwać do niego dostęp w znany sposób, ale nie ominąć niestandardowej logiki.

Podklasy istniejącej klasy kolekcji prawdopodobnie będą łatwiejsze (dla ciebie i konsumenta) niż zawijanie kolekcji w innej klasie.