EntitySet-是否有未设置IList.Add的合理原因？

问题描述

| 有3种方法可以将项目添加到大多数列表中... 通过直接的公共API方法，通常为“ 0” 通过通用的“ 1”接口 通过非通用的“ 2”接口方法 而且您通常希望它们的行为大致相同。但是，LINQ的

EntitySet<T>

在3.5和4.0上都是特有的。 “ 4” API不会将集合标记为“已分配”（其他两种机制会这样做），这听起来很琐碎，但重要的是它会严重影响样板代码中的序列化（即，导致序列化被跳过）。 例：

EntitySet<string> set1 = new EntitySet<string>();
set1.Add(\"abc\");
Debug.Assert(set1.Count == 1); // pass
Debug.Assert(set1.HasLoadedOrAssignedValues,\"direct\"); // pass

EntitySet<string> set2 = new EntitySet<string>();
IList<string> typedList = set2;
typedList.Add(\"abc\");
Debug.Assert(set2.Count == 1); // pass
Debug.Assert(set2.HasLoadedOrAssignedValues,\"typed list\"); // pass

EntitySet<string> set3 = new EntitySet<string>();
IList untypedList = set3;
untypedList.Add(\"abc\");
Debug.Assert(set3.Count == 1); // pass
Debug.Assert(set3.HasLoadedOrAssignedValues,\"untyped list\"); // FAIL

现在，这令我深感意外；如此之多，以至于我花了2个多小时来跟踪代码以隔离正在发生的事情。所以... 有什么理智的理由吗？还是这只是一个错误？ （FWIW，3.5中的

set.Assign(set)

也存在问题，但现在已在4.0中修复。）     

解决方法

        有趣的是，现在已经在多个版本中发现了这一点（您说过，在4.0中修复了3.5个问题）。这是2007年的帖子。4.0中的其余

IList

方法正确地与

IList<T>

方法联系在一起。我认为有两种可能的解释（关于错误/功能的变化）： 这是Microsoft尚未修复的实际错误。 此功能是其他一些Microsoft代码利用杠杆作用来添加项目而不设置ѭ9的功能。 可能两者兼有-框架内其他代码所依赖的bug。听起来好像有人对自己说：   没有人真正将其转换为IList然后调用Add方法，对吗？     ,        令人惊讶的是，这种差异似乎源于

IList.Add

和

IList<T>.Add

方法实际上具有不同的语义： 如果要添加的实体已经存在，则“ 10”方法将失败

LIst<T>.Add

方法将删除，然后重新添加一个已经存在的实体 造成这种差异的明显原因是，定义了“ 10”接口方法以返回添加的实体的索引，对于“ 10”的典型实现，该方法将始终是“ 17”之前的集合的“ 16”。 无论如何，由于两个实现是有意不同的，因此似乎作者偶然地忽略了

IList.Add

版本中的

this.OnModified()

调用。     ,        对我来说似乎是个虫子。 ILSpy显示了两种实现之间的区别：

int IList.Add(object value)
{
    TEntity tEntity = value as TEntity;
    if (tEntity == null || this.IndexOf(tEntity) >= 0)
    {
        throw Error.ArgumentOutOfRange(\"value\");
    }
    this.CheckModify();
    int count = this.entities.Count;
    this.entities.Add(tEntity);
    this.OnAdd(tEntity);
    return count;
}

// System.Data.Linq.EntitySet<TEntity>
/// <summary>Adds an entity.</summary>
/// <param name=\"entity\">The entity to add.</param>
public void Add(TEntity entity)
{
    if (entity == null)
    {
        throw Error.ArgumentNull(\"entity\");
    }
    if (entity != this.onAddEntity)
    {
        this.CheckModify();
        if (!this.entities.Contains(entity))
        {
            this.OnAdd(entity);
            if (this.HasSource)
            {
                this.removedEntities.Remove(entity);
            }
            this.entities.Add(entity);
            this.OnListChanged(ListChangedType.ItemAdded,this.entities.IndexOf(entity));
        }
        this.OnModified();
    }
}

看起来IList实现只是忽略了调用LINQ to SQL可能依赖以跟踪其更改的几个事件调用程序（

OnListChanged

和

OnModified

）。如果这是故意的，我希望他们也不要打给

OnAdd

。 为什么他们不简单地将

IList.Add

的值转换为

TEntity

并调用通用的

Add

方法，这超出了我的理解。