为什么Linq.Enumerable.Where打破了我的ObservableCollection

时间:2013-06-27 15:47:44

标签: wpf linq ienumerable observablecollection ieditableobject

背景

我正在编写一个WPF应用程序,严格遵循MVVM模式。我有一个BaseRepository类作为连接到不同数据库的通用接口(EF不是一个选项),一切正常;这只是一个技术问题。

我使用一个名为NotifyingCollection的包装ObservableCollection来订阅IEditableObject的ItemEndEdit事件(我的ViewModelBase实体包装器实现了INotifyPropertyChanged和IEditableObject成员)。

在编辑WPF DataGrid中的项目时调用ReadAll方法时,提供的代码示例将抛出“'EditItem',此视图不允许”异常。 然而,如果我用注释掉的部分替换方法中的线条,它就能完美运行!

问题:

换句话说,它看起来像中继Linq.Enumerable.Where扩展方法而不是返回集合的IEnumerable版本从自定义集合中删除功能;如果它们都是IEnumerable,为什么会这样呢?

代码示例:

namespace MyCompany.Common.Abstracts
{
    using System;
    using System.Collections.Generic;
    using System.Collections.Specialized;
    using System.ComponentModel;
    using System.Data.Common;
    using System.Diagnostics;
    using System.Linq;
    using System.Linq.Expressions;
    using MyCompany.Common.Extensions;
    using MyCompany.Common.Utilities;


    public abstract class BaseRepository<TEntity> : IDisposable where TEntity : ViewModelBase
    {
        protected BaseRepository()
        {
            this.EntitySet = new NotifyingCollection<TEntity>();
            this.EntitySet.ItemEndEdit += new ViewModelBase.ItemEndEditEventHandler(ItemEndEdit);
            this.EntitySet.CollectionChanged += new NotifyCollectionChangedEventHandler(CollectionChanged);
        }

        protected abstract NotifyingCollection<TEntity> EntitySet { get; set; }

        protected virtual void PropertyChanged(object sender, PropertyChangedEventArgs e)
        {
            Debug.WriteLine(String.Format("Modify '{0}'", e.PropertyName), "PropertyChanged");
        }

        protected virtual void ItemEndEdit(IEditableObject sender)
        {
            this.Update(sender as TEntity);
        }

        protected virtual void CollectionChanged(object sender, NotifyCollectionChangedEventArgs e)
        {
            var collection = (e.Action == NotifyCollectionChangedAction.Remove) ?
                e.OldItems : e.NewItems;

            foreach (TEntity entity in collection)
            {
                switch (e.Action)
                {
                    case NotifyCollectionChangedAction.Add:
                        entity.PropertyChanged += this.PropertyChanged;
                        this.Create(entity);
                        break;

                    case NotifyCollectionChangedAction.Remove:
                        entity.PropertyChanged -= this.PropertyChanged;
                        this.Delete(entity);
                        break;

                    default:
                        Debug.WriteLine(String.Format("{0} '{1}'", e.Action.ToString(), entity.DisplayName), "CollectionChanged");
                        break;
                }
            }
        }

        public virtual bool Create(TEntity entity)
        {
            Debug.WriteLine(String.Format("Create '{0}'", entity.DisplayName), "CollectionChanged");
            return true;
        }

        public virtual IEnumerable<TEntity> Read(Expression<Func<TEntity, bool>> filter)
        {
            return this.EntitySet.Where(filter.Compile());
        }

        public virtual IEnumerable<TEntity> ReadAll()
        {
            return this.Read(x => true);
            //return this.EntitySet;
        }

        public virtual bool Update(TEntity entity)
        {
            Debug.WriteLine(String.Format("Update '{0}'", entity.DisplayName), "ItemEndEdit");
            return true;
        }

        public virtual bool Delete(TEntity entity)
        {
            Debug.WriteLine(String.Format("Delete '{0}'", entity.DisplayName), "CollectionChanged");
            return true;
        }

        public virtual IEnumerable<TColumn> GetColumn<TColumn>(string columnName)
        {
            var lookupTable = this.Read(x => x != null);

            List<TColumn> column = new List<TColumn>();
            foreach (TEntity record in lookupTable)
            {
                column.Add(record.GetPropValue<TColumn>(columnName));
            }
            var result = column.Distinct();

            foreach (TColumn element in result)
            {
                yield return element;
            }
        }

        public abstract int Commit();

        public abstract DbTransaction BeginTransaction();

        public abstract void Dispose();
    }
}

2 个答案:

答案 0 :(得分:5)

来自MichaelLPerry的博客:Common mistakes while using ObservableCollection

  

ObservableCollection势在必行。 Linq是声明性的。他们俩   没有额外帮助就不能一起使用。

     

命令式代码明确地对某些事情起作用。使用时   ObservableCollection,您显式调用Add,Remove等   更改集合的方法。你必须确切地确定何时和   如何采取这些行动。

     

声明性代码隐式生成某些内容。当你使用linq时   声明集合的样子,如何过滤,   映射和转换。您没有明确修改集合。

     

这两种范式不混合。一旦你使用linq   ObservableCollection,它不再是可观察的。有   像Bindable Linq这样的开源项目填补了这一空白。但   如果没有这些额外的帮助,当事情没有时,人们常常感到惊讶   工作

答案 1 :(得分:0)

EntitySet.Where方法的返回值将不再是原始NotifyingCollection<TEntity>