web-dev-qa-db-de.com

Benachrichtigen Sie ObservableCollection, wenn sich Element ändert

Ich habe auf diesem Link gefunden 

ObservableCollection merkt nicht, wenn sich ein Element darin ändert (auch mit INotifyPropertyChanged)

einige Techniken zum Benachrichtigen einer Observablecollection, dass ein Element geändert wurde. Die TrulyObservableCollection in diesem Link scheint das zu sein, wonach ich suche.

public class TrulyObservableCollection<T> : ObservableCollection<T>
where T : INotifyPropertyChanged
{
    public TrulyObservableCollection()
    : base()
    {
        CollectionChanged += new NotifyCollectionChangedEventHandler(TrulyObservableCollection_CollectionChanged);
    }

    void TrulyObservableCollection_CollectionChanged(object sender, NotifyCollectionChangedEventArgs e)
    {
        if (e.NewItems != null)
        {
            foreach (Object item in e.NewItems)
            {
                (item as INotifyPropertyChanged).PropertyChanged += new PropertyChangedEventHandler(item_PropertyChanged);
            }
        }
        if (e.OldItems != null)
        {
            foreach (Object item in e.OldItems)
            {
                (item as INotifyPropertyChanged).PropertyChanged -= new PropertyChangedEventHandler(item_PropertyChanged);
            }
        }
    }

    void item_PropertyChanged(object sender, PropertyChangedEventArgs e)
    {
        NotifyCollectionChangedEventArgs a = new NotifyCollectionChangedEventArgs(NotifyCollectionChangedAction.Reset);
        OnCollectionChanged(a);
    }
}

Wenn ich jedoch versuche, es zu verwenden, erhalte ich keine Benachrichtigungen in der Sammlung. Ich bin nicht sicher, wie ich dies korrekt in meinen C # -Code implementieren kann:

XAML:

    <DataGrid AutoGenerateColumns="False" ItemsSource="{Binding MyItemsSource, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}">
        <DataGrid.Columns>
            <DataGridCheckBoxColumn Binding="{Binding MyProperty, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}"/>
        </DataGrid.Columns>
    </DataGrid>

ViewModel:

public class MyViewModel : ViewModelBase
{
    private TrulyObservableCollection<MyType> myItemsSource;
    public TrulyObservableCollection<MyType> MyItemsSource
    {
        get { return myItemsSource; }
        set 
        { 
            myItemsSource = value; 
            // Code to trig on item change...
            RaisePropertyChangedEvent("MyItemsSource");
        }
    }

    public MyViewModel()
    {
        MyItemsSource = new TrulyObservableCollection<MyType>()
        { 
            new MyType() { MyProperty = false },
            new MyType() { MyProperty = true },
            new MyType() { MyProperty = false }
        };
    }
}

public class MyType : ViewModelBase
{
    private bool myProperty;
    public bool MyProperty
    {
        get { return myProperty; }
        set 
        {
            myProperty = value;
            RaisePropertyChangedEvent("MyProperty");
        }
    }
}

public class ViewModelBase : INotifyPropertyChanged
{
    public event PropertyChangedEventHandler PropertyChanged;

    protected void RaisePropertyChangedEvent(string propertyName)
    {
        if (PropertyChanged != null)
        {
            PropertyChangedEventArgs e = new PropertyChangedEventArgs(propertyName);
            PropertyChanged(this, e);
        }
    }
}

Wenn ich das Programm starte, habe ich die Checkbox 3 auf false, true, false wie in der Eigenschaftsinitialisierung. Wenn ich jedoch den Status einer der ckeckbox ändere, durchläuft das Programm item_PropertyChanged, aber niemals im MyItemsSource-Property-Code.

34
Pansoul

Die Stelle, die Sie als // Code to trig on item change... kommentiert haben, wird nur ausgelöst, wenn das Sammlungsobjekt geändert wird, z. B. wenn es auf ein neues Objekt oder auf null gesetzt wird.

Registrieren Sie mit Ihrer aktuellen Implementierung von TrulyObservableCollection etwas für das CollectionChanged-Ereignis von MyItemsSource, um die Eigenschaftsänderungen Ihrer Sammlung zu behandeln.

public MyViewModel()
{
    MyItemsSource = new TrulyObservableCollection<MyType>();
    MyItemsSource.CollectionChanged += MyItemsSource_CollectionChanged;

    MyItemsSource.Add(new MyType() { MyProperty = false });
    MyItemsSource.Add(new MyType() { MyProperty = true});
    MyItemsSource.Add(new MyType() { MyProperty = false });
}


void MyItemsSource_CollectionChanged(object sender, NotifyCollectionChangedEventArgs e)
{
    // Handle here
}

Ich persönlich mag diese Implementierung wirklich nicht. Sie lösen ein CollectionChanged -Ereignis aus, das besagt, dass die gesamte Sammlung zurückgesetzt wurde, wenn sich eine Eigenschaft ändert. Sicher, es wird die Benutzeroberfläche jederzeit aktualisiert, wenn sich ein Element in der Sammlung ändert. Ich kann jedoch feststellen, dass die Leistung schlecht ist, und es scheint nicht möglich zu sein, herauszufinden, welche Eigenschaft sich geändert hat. Dies ist eine der wichtigsten Informationen Normalerweise brauche ich etwas für PropertyChanged

Ich bevorzuge die Verwendung einer regulären ObservableCollection und die PropertyChanged-Ereignisse einfach mit den Elementen auf CollectionChanged zu verknüpfen. Wenn Ihre Benutzeroberfläche korrekt an die Elemente in der Variablen ObservableCollection gebunden ist, müssen Sie der Benutzeroberfläche nicht mitteilen, dass sie aktualisiert werden soll, wenn sich eine Eigenschaft für ein Element in der Auflistung ändert.

public MyViewModel()
{
    MyItemsSource = new ObservableCollection<MyType>();
    MyItemsSource.CollectionChanged += MyItemsSource_CollectionChanged;

    MyItemsSource.Add(new MyType() { MyProperty = false });
    MyItemsSource.Add(new MyType() { MyProperty = true});
    MyItemsSource.Add(new MyType() { MyProperty = false });
}

void MyItemsSource_CollectionChanged(object sender, NotifyCollectionChangedEventArgs e)
{
    if (e.NewItems != null)
        foreach(MyType item in e.NewItems)
            item.PropertyChanged += MyType_PropertyChanged;

    if (e.OldItems != null)
        foreach(MyType item in e.OldItems)
            item.PropertyChanged -= MyType_PropertyChanged;
}

void MyType_PropertyChanged(object sender, PropertyChangedEventArgs e)
{
    if (e.PropertyName == "MyProperty")
        DoWork();
}
73
Rachel

Ich habe diesen Fall mit statischer Aktion gelöst  


public class CatalogoModel 
{
    private String _Id;
    private String _Descripcion;
    private Boolean _IsChecked;

    public String Id
    {
        get { return _Id; }
        set { _Id = value; }
    }
    public String Descripcion
    {
        get { return _Descripcion; }
        set { _Descripcion = value; }
    }
    public Boolean IsChecked
    {
        get { return _IsChecked; }
        set
        {
           _IsChecked = value;
            NotifyPropertyChanged("IsChecked");
            OnItemChecked.Invoke();
        }
    }

    public static Action OnItemChecked;
} 

public class ReglaViewModel : ViewModelBase
{
    private ObservableCollection<CatalogoModel> _origenes;

    CatalogoModel.OnItemChecked = () =>
            {
                var x = Origenes.Count;  //Entra cada vez que cambia algo en _origenes
            };
}
6
sat1582

Ich weiß, es ist spät, aber vielleicht hilft das anderen. Ich habe eine Klasse NotifyObservableCollection erstellt, die das Problem der fehlenden Benachrichtigung des Elements selbst löst, wenn sich eine Eigenschaft des Elements ändert. Die Verwendung ist so einfach wie ObservableCollection.

public class NotifyObservableCollection<T> : ObservableCollection<T> where T : INotifyPropertyChanged
{
    private void Handle(object sender, PropertyChangedEventArgs args)
    {
        OnCollectionChanged(new NotifyCollectionChangedEventArgs(NotifyCollectionChangedAction.Reset, null));
    }

    protected override void OnCollectionChanged(NotifyCollectionChangedEventArgs e)
    {
        if (e.NewItems != null) {
            foreach (object t in e.NewItems) {
                ((T) t).PropertyChanged += Handle;
            }
        }
        if (e.OldItems != null) {
            foreach (object t in e.OldItems) {
                ((T) t).PropertyChanged -= Handle;
            }
        }
        base.OnCollectionChanged(e);
    }

Während Elemente hinzugefügt oder entfernt werden, leitet die Klasse das Ereignis PropertyChanged an das Objekt PropertyChanged der Auflistung weiter.

verwendungszweck:

public abstract class ParameterBase : INotifyPropertyChanged
{
    protected readonly CultureInfo Ci = new CultureInfo("en-US");
    private string _value;

    public string Value {
        get { return _value; }
        set {
            if (value == _value) return;
            _value = value;
            OnPropertyChanged();
        }
    }
}

public class AItem {
    public NotifyObservableCollection<ParameterBase> Parameters {
        get { return _parameters; }
        set {
            NotifyCollectionChangedEventHandler cceh = (sender, args) => OnPropertyChanged();
            if (_parameters != null) _parameters.CollectionChanged -= cceh;
            _parameters = value;
            //needed for Binding to AItem at xaml directly
            _parameters.CollectionChanged += cceh; 
        }
    }

    public NotifyObservableCollection<ParameterBase> DefaultParameters {
        get { return _defaultParameters; }
        set {
            NotifyCollectionChangedEventHandler cceh = (sender, args) => OnPropertyChanged();
            if (_defaultParameters != null) _defaultParameters.CollectionChanged -= cceh;
            _defaultParameters = value;
            //needed for Binding to AItem at xaml directly
            _defaultParameters.CollectionChanged += cceh;
        }
    }


public class MyViewModel {
    public NotifyObservableCollection<AItem> DataItems { get; set; }
}

Wenn sich jetzt eine Eigenschaft eines Elements in DataItems ändert, wird das folgende xaml eine Benachrichtigung erhalten, obwohl es an Parameters[0] oder an das Element selbst gebunden ist, mit Ausnahme der sich ändernden Eigenschaft Value des Elements (Konverter bei Triggern werden bei jeder Änderung als zuverlässig bezeichnet).

<DataGrid CanUserAddRows="False" AutoGenerateColumns="False" ItemsSource="{Binding DataItems}">
    <DataGrid.Columns>
        <DataGridTextColumn Binding="{Binding Parameters[0].Value}" Header="P1">
            <DataGridTextColumn.CellStyle>
                <Style TargetType="DataGridCell">
                    <Setter Property="Background" Value="Aqua" />
                    <Style.Triggers>
                        <DataTrigger Value="False">
                            <!-- Bind to Items with changing properties -->
                            <DataTrigger.Binding>
                                <MultiBinding Converter="{StaticResource ParameterCompareConverter}">
                                    <Binding Path="DefaultParameters[0]" />
                                    <Binding Path="Parameters[0]" />
                                </MultiBinding>
                            </DataTrigger.Binding>
                            <Setter Property="Background" Value="DeepPink" />
                        </DataTrigger>
                        <!-- Binds to AItem directly -->
                        <DataTrigger Value="True" Binding="{Binding Converter={StaticResource CheckParametersConverter}}">
                            <Setter Property="FontWeight" Value="ExtraBold" />
                        </DataTrigger>
                    </Style.Triggers>
                </Style>
            </DataGridTextColumn.CellStyle>
        </DataGridTextColumn>
2
WPFGermany

Die ObservableCollection und ihre Derivate erhöhen ihre Eigenschaftsänderungen intern. Der Code in Ihrem Setter sollte nur ausgelöst werden, wenn Sie der MyItemsSource-Eigenschaft einen neuen TrulyObservableCollection<MyType> zuweisen. Das heißt, es sollte nur einmal vom Konstruktor aus geschehen.

Ab diesem Zeitpunkt erhalten Sie Benachrichtigungen zu Eigenschaftsänderungen aus der Sammlung, nicht aus dem Setter in Ihrem Ansichtsmodell.

2
Jay

Sie können eine Erweiterungsmethode verwenden, um generisch über geänderte Eigenschaften eines Elements in einer Sammlung informiert zu werden.

public static class ObservableCollectionExtension
{
    public static void NotifyPropertyChanged<T>(this ObservableCollection<T> observableCollection, Action<T, PropertyChangedEventArgs> callBackAction)
        where T : INotifyPropertyChanged
    {
        observableCollection.CollectionChanged += (sender, args) =>
        {
            //Does not prevent garbage collection says: http://stackoverflow.com/questions/298261/do-event-handlers-stop-garbage-collection-from-occuring
            //publisher.SomeEvent += target.SomeHandler;
            //then "publisher" will keep "target" alive, but "target" will not keep "publisher" alive.
            if (args.NewItems == null) return;
            foreach (T item in args.NewItems)
            {
                item.PropertyChanged += (obj, eventArgs) =>
                {
                    callBackAction((T)obj, eventArgs);
                };
            }
        };
    }
}

public void ExampleUsage()
{
    var myObservableCollection = new ObservableCollection<MyTypeWithNotifyPropertyChanged>();
    myObservableCollection.NotifyPropertyChanged((obj, notifyPropertyChangedEventArgs) =>
    {
        //DO here what you want when a property of an item in the collection has changed.
    });
}
2
Mike de Klerk

Alle Lösungen hier sind korrekt, aber es fehlt ein wichtiges Szenario, in dem die Methode Clear () verwendet wird, die im OldItems kein NotifyCollectionChangedEventArgs liefert. Objekt.

dies ist der perfekte ObservableCollection.

public delegate void ListedItemPropertyChangedEventHandler(IList SourceList, object Item, PropertyChangedEventArgs e);
public class ObservableCollectionEX<T> : ObservableCollection<T>
{
    #region Constructors
    public ObservableCollectionEX() : base()
    {
        CollectionChanged += ObservableCollection_CollectionChanged;
    }
    public ObservableCollectionEX(IEnumerable<T> c) : base(c)
    {
        CollectionChanged += ObservableCollection_CollectionChanged;
    }
    public ObservableCollectionEX(List<T> l) : base(l)
    {
        CollectionChanged += ObservableCollection_CollectionChanged;
    }

    #endregion



    public new void Clear()
    {
        foreach (var item in this)            
            if (item is INotifyPropertyChanged i)                
                i.PropertyChanged -= Element_PropertyChanged;            
        base.Clear();
    }
    private void ObservableCollection_CollectionChanged(object sender, NotifyCollectionChangedEventArgs e)
    {
        if (e.OldItems != null)
            foreach (var item in e.OldItems)                
                if (item != null && item is INotifyPropertyChanged i)                    
                    i.PropertyChanged -= Element_PropertyChanged;


        if (e.NewItems != null)
            foreach (var item in e.NewItems)                
                if (item != null && item is INotifyPropertyChanged i)
                {
                    i.PropertyChanged -= Element_PropertyChanged;
                    i.PropertyChanged += Element_PropertyChanged;
                }
            }
    }
    private void Element_PropertyChanged(object sender, PropertyChangedEventArgs e) => ItemPropertyChanged?.Invoke(this, sender, e);


    public ListedItemPropertyChangedEventHandler ItemPropertyChanged;

}
1
bigworld12

Eine einfache Lösung ist die Verwendung von BindingList<T> anstelle von ObservableCollection<T>. In der Tat werden bei den Änderungen der BindingList-Relay-Elemente Änderungen vorgenommen. Wenn das Element mit einer Bindungsliste die Schnittstelle INotifyPropertyChanged implementiert, können Sie einfach Benachrichtigungen über das ListChanged-Ereignis erhalten.

Siehe auch diese SO Antwort.

0
Malick