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

Question

有 3 种方法可以将项目添加到大多数列表...

• 通过直接公共 API 方法，通常
• 通过通用 IList.Add(T)Add(SomeType) >
• 通过非泛型 IList.Add(object) 接口方法

进行接口，您通常期望它们的行为或多或少相同。然而，LINQ 的 EntitySet 在 3.5 和 4.0 上都是特有的； IList API 不会将集合标记为“已分配” - 其他两种机制会< /strong> - 这听起来微不足道，但很重要，因为它严重影响样板代码中的序列化（即导致其被跳过）。

示例：

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

现在……这让我深感惊讶；如此之多以至于我花了两个多小时通过代码向上跟踪来隔离发生的事情。那么...

这有任何合理的理由吗？或者这只是一个错误？

（FWIW，3.5 中的 set.Assign(set) 也存在问题，但现在已在 4.0 中修复。）

Answer 1

有趣的是，现在多个版本都已经发现了这个问题（您说 3.5 的问题已在 4.0 中修复）。 这是 2007 年的帖子. 4.0 中的其余 IList 方法已正确绑定到 IList 方法。我认为有两种可能的解释（关于错误/功能多样性）：

1. 这是 Microsoft 尚未修复的实际错误。
2. 其他一些 Microsoft 代码利用利用此功能来添加项目，而无需设置HasLoadedOrAssignedValues。

这可能是两者兼而有之 - 框架内其他代码所指望的错误。听起来就像有人对自己说：

没有人会真正将其转换为 IList，然后调用 Add 方法，对吧？

Answer 2

令人惊讶的是，这种差异似乎根源于 IList.Add 和 IList.Add 方法实际上具有不同的语义

• ：如果要添加的实体已存在，则 code>IList.Add 方法会失败
• LIst.Add 方法会删除实体，然后重新添加已存在的实体

明显原因为了这区别在于 IList.Add 接口方法被定义为返回添加实体的索引，对于 IList.Add 的典型实现来说，该索引始终是 Count 添加之前集合的。

无论如何，由于这两个实现是有意不同的，因此作者似乎只是无意中省略了 IList.Add 版本中的 this.OnModified() 调用。

Answer 3

对我来说看起来像一个错误。 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 方法，这超出了我的理解。