ObservableCollection CollectionChanged событие, проблема

Question

ObservableCollection CollectionChanged событие, проблема

1

Когда я добавляю новый элемент в ObservableCollection CollectionChanged он работает отлично,

Вот код MVVM

public List<BOMForDesignMasterModel> BOMCollection { get; set; } 

public ObservableCollection<BOMForDesignMasterModel> BOM
    {
        get { return _BOM; }
        set
        {
            _BOM = value;
            NotifyPropertyChanged("BOM");
            BOM.CollectionChanged += BOM_CollectionChanged;
        }
    }
public DesignMasterModel DesignMaster
    {
        get
        {
            return _DesignMaster;
        }
        set
        {
            if (value != null)
            {
                _DesignMaster = value;
                BOM.Clear();
                BOM = new ObservableCollection<BOMForDesignMasterModel>(BOMCollection.Where(x => x.DesignMasterId == value.DesignMasterId));
                NotifyPropertyChanged("DesignMaster");
            }
        }
    }
void BOM_CollectionChanged(object sender, System.Collections.Specialized.NotifyCollectionChangedEventArgs e)
    {
        if (e.NewItems != null)
            foreach (BOMForDesignMasterModel item in e.NewItems)
                item.PropertyChanged += item_PropertyChanged;
        if (e.OldItems != null)
            foreach (BOMForDesignMasterModel item in e.OldItems)
                item.PropertyChanged -= item_PropertyChanged;

    }
    public String TotalWeight { get { return _TotalWeight; } set { _TotalWeight = value; NotifyPropertyChanged("TotalWeight"); } }
    void item_PropertyChanged(object sender, PropertyChangedEventArgs e)
    {
        if (e.PropertyName == "Weight")
        {
            TotalWeight = BOM.Sum(x => x.Weight).ToString();
        }
    }

Здесь я использую сбор копий на основе условия,

BOM = new ObservableCollection<BOMForDesignMasterModel>(BOMCollection.Where(x => x.DesignMasterId == value.DesignMasterId));

когда мне понравилось это событие item_PropertyChanged, не работающее с копируемыми элементами.

Как я могу исправить эту проблему?

Sam Alex 06 апр. 2015, в 20:54

Источник

0

Вы действительно регистрируетесь для BOM_CollectionChanged где-нибудь? я не вижу этого, но вы, очевидно, не включили инструментальные части кода.
Theodosius Von Richthofen 06 апр. 2015, в 18:37
0

Зачем все регистрироваться. В наборе BOMForDesignMasterModel Weight просто вызовите NotifyPropertyChanged ("TotalWeight"); И имейте это, вычисляют общий вес.
paparazzo 06 апр. 2015, в 18:45
1

@SamAlex не уверен, что я понимаю вашу проблему, но когда вы создаете список, он не вызовет событие CollectionChanged для элементов, которые вы передаете в конструктор, потому что a) ему не нужно b) никакой обработчик события CollectionChanged еще не назначен. По сути, в установщике свойства BOM , где вы назначаете обработчик CollectionChanged , список уже полностью создан и содержит начальные элементы.
dkozl 06 апр. 2015, в 19:04
1

Просто небольшая точка, но каждый раз, когда вызывается установщик BOM вы регистрируете обработчик для назначаемой коллекции, но кажется, что вы не _BOM зарегистрированный обработчик из текущего значения, то есть _BOM . Хорошей практикой является удаление нежелательных обработчиков, в противном случае вы можете столкнуться с ситуацией, когда объекты не GCed из-за ссылок на обработчики событий.
MotoSV 06 апр. 2015, в 19:23

Показать ещё 2 комментария

Теги:

c#

wpf

mvvm

1 ответ

Ещё вопросы

Вы действительно регистрируетесь для BOM_CollectionChanged где-нибудь? я не вижу этого, но вы, очевидно, не включили инструментальные части кода.
Зачем все регистрироваться. В наборе BOMForDesignMasterModel Weight просто вызовите NotifyPropertyChanged ("TotalWeight"); И имейте это, вычисляют общий вес.
@SamAlex не уверен, что я понимаю вашу проблему, но когда вы создаете список, он не вызовет событие CollectionChanged для элементов, которые вы передаете в конструктор, потому что a) ему не нужно b) никакой обработчик события CollectionChanged еще не назначен. По сути, в установщике свойства BOM , где вы назначаете обработчик CollectionChanged , список уже полностью создан и содержит начальные элементы.
Просто небольшая точка, но каждый раз, когда вызывается установщик BOM вы регистрируете обработчик для назначаемой коллекции, но кажется, что вы не _BOM зарегистрированный обработчик из текущего значения, то есть _BOM . Хорошей практикой является удаление нежелательных обработчиков, в противном случае вы можете столкнуться с ситуацией, когда объекты не GCed из-за ссылок на обработчики событий.

goobering · Accepted Answer · 2015-04-06T21-09-00.000Z

Я думаю, это ваша проблема:

BOM = new ObservableCollection<BOMForDesignMasterModel>(BOMCollection.Where(x => x.DesignMasterId == value.DesignMasterId));

Когда вы новичок в ObservableCollection, как это, вы не меняете его содержимое, которое должно срабатывать NotifyPropertyChanged. Вы меняете экземпляр ObservableCollection, а это не так.

Хотя вы не можете просто обновлять ObservableCollection, вы можете использовать Clear/Add/Remove для изменения его содержимого по мере необходимости. Чтобы ваш сеттер не был слишком сухим, я бы рекомендовал переместить эту логику в свой собственный метод и найти другой способ его запуска.

Это будет выглядеть примерно так:

private void RefreshBOM(int designMasterId)
{
    BOM.Clear();
    var bomResult = BOMCollection.Where(x => x.DesignMasterId == designMasterId);
    if(bomResult != null)
    {
        foreach(var result in bomResult)
        {
            BOM.Add(result);
        }
    }
}